Problemas MongoDB con PHP

[dvex_92]

Saludos a todos! Soy Nuevo en este foro y a la vez soy principiante en utilizar MongoDB (NoSQL) ya q tengo un proyecto sobre esto. Cabe decir q en mi vida universitaria (actualmente) solo eh trabajado con DB's Relacionales.

Entre investigando en estos ultimos dias sobre PHP con MongoDB ya sé como hacer un "insert, delete y select" en la DB no relacional con PHP(ver. 5) (utilizando Netbeans).

Aqui os le pego el codigo que hize(inicialmente) para Testear todo lo que eh investigando:

Código PHP:

  $username = 'administrador'; 
            $userpass = 'administrador'; 
            $dataBase = 'Carrito'; 
 
            $conexion = new Mongo("mongodb://$username:$userpass@localhost/$dataBase"); 
 
            $collection = $conexion->selectCollection($dataBase,'Productos'); 
                                     
            $nuevoRegistro = array("nombre" => "El Arma Secreta", "Precio" => "S/. 65", "Cantidad" => "67 unid"); 
            $collection->insert($nuevoRegistro); 
             
            $collection->remove(array('nombre' => 'El Arma Secreta')); 
            $collection->remove(); 
             
            $cursor = $collection->find(array('Precio' => 'S/. 154')); 
            $cursor = $collection->find(); 
             
            foreach ($cursor as $prod) { 
                echo "El producto es: ".$prod["nombre"]." El precio es: ".$prod["Precio"]." La cantidad disponible es: ".$prod["Cantidad"]."\n"; 
            }  
   

Un Codigo pobre.. como principiante que soy. Aqui probe cada sentencia y me salio OK. Hacia todo en la DB
Ahora mi problema recae Cuando Quiero formalizar mas este codigo. Miren ahora:

Código PHP:

  class Persistencia { 
    /* Patron Singleton*/ 
    private static $_instancia; 
    public static function getInstancia() { 
        try { 
            if(self::$_instancia == null){ 
                self::$_instancia = new Persistencia(); 
            } 
        } catch (Exception $exc) { 
            throw $exc->getMessage(); 
        } 
    } 
    /*Fin del Patron */ 
 
    private $_cn; 
    private $_collection; 
 
 
    public function __construct() { 
        try { 
            $username = 'administrador'; 
            $userpass = 'administrador'; 
            $database = 'Carrito'; 
            $colecction = 'Productos'; 
             
            $this->_cn = new Mongo("mongodb://$username:$userpass@localhost/$database"); 
            $this->_collection = $this->_cn->selectCollection($database, $colecction); 
 
        } catch (Exception $exc) { 
            error_log($exc->getMessage()."\n", 3, "../log/Error.log"); 
        } 
    } 
     
    private function _executeQuery($query, $tipo) { 
        try{ 
            switch ($tipo) { 
                case 1 : 
                    $cursor = $this->_collection->find(); 
                    return $cursor; 
                    break; 
                case 2 : 
                    $this->_collection->insert($query); 
                    break; 
                case 3 : 
                    $this->_collection->remove(); 
                    break; 
            } 
        }  catch (Exception $exc){ 
            error_log($exc->getMessage()."\n", 3, "../log/Error.log"); 
            error_log("Query:$query\n", 3, "../log/Error.log"); 
        } 
    } 
 
 
    public function findAll() { 
        $lista = $this->_executeQuery("", 1); 
        return $lista; 
    } 
     
    public function addProduct($nombre, $precio, $cantidad) { 
        $query = array("nombre" => $nombre, "Precio" => $precio, "Cantidad" => $cantidad); 
        $this->_executeQuery($query, 2); 
    } 
     
    public function deleteAll() { 
        $this->_executeQuery("", 3); 
    } 
} 
 
Persistencia::getInstancia();  
   

Como se daran cuenta.. solo eh transladado todo el codigo anterior ah hacerlo en una clase.

Ahora.. quiero testear esto a ver si me sale tal y como lo hize en el 1er codigo y comenze con el FIND.

Código PHP:

  <?php 
require_once '../model/Persistencia.php'; 
 
$miInstancia = Persistencia::getInstancia(); 
$lista = $miInstancia->findAll(); 
var_dump($lista); 
?>   

Y ahora, nose que PASO que me sale "Fatal error: Call to a member function findAll() on a non-object in xampp\htdocs\ConnectMongoDB\test\Testear.php on line 6"
No entiendo...¿Que es lo que paso? Esa es mi duda. Trato de ver documentacion pero no encuentro el error.

Y una cosa mas.... ¿Puedo mejorar este codigo?.. Alguna mejora puedo hacerle?

Gracias!.. Espero con ansias sus respuestas.

[GatorV]

El problema es que tu Singleton esta mal, ya que no regresa nada, por eso te da ese error, también en el catch tienes esto:

Código PHP:

catch (Exception $exc) {
    throw $exc->getMessage();
}
 

Solamente puedes lanzar clases que sean Excepciones o derivadas de esta, por lo cual si vas a re-lanzar la excepción, entonces no la captures en el Singleton.

Saludos.

[dvex_92]

Aahh.. es verdad, no regresaba nada.. Tenia que hacerle:

Código PHP:

  return self::$_instancia;  
   

Y no entiendo a lo del CATCH.. A q te refieres con eso?

Y alguna recomendacion o algo que deba hacerle para mejorar mi codigo??

[GatorV]

Lo que estas haciendo en el catch es incorrecto, ya que lanzas el mensaje, y eso es un string y no puedes hacerlo.

Ya te comenté más arriba, si solo vas a cachar la excepción para lanzarla, pues no la caches y deja que fluya, o lanza la excepción:

Código PHP:

// mal
catch (Exception $ex) {
     throw $ex->getMessage();
}
 
// bien "a medias"
catch (Exception $ex) {
     throw $ex;
}
 

Saludos.