[SOLUCIONADO] Conexion a Base de Datos Orientada a Objetos

AnGuisi · #1 (**permalink**) 10/07/2013, 19:11

Tengo un problema al hacer la conexión a mi base de datos orientada a objetos. El mysqli_multi_query me arroja un error con respecto al parámetro. También quisiera utilizar esa misma conexión en otras funciones. He aquí el código.

Código PHP:

  <? 
 
 
class clssentenciasmysql { 
 
function conectar() { 
      $con=mysqli_connect("localhost","root","root123","soptecvnzla"); 
 
if (mysqli_connect_errno($con)) 
  { 
  echo "Error en Conexión: " . mysqli_connect_error(); 
  } 
} 
     
     
 
    function insertar () { 
 
   
  extract($_POST); 
   
   $sql = "INSERT INTO  `soptecvnzla`.`equipo` ( 
`codigo` , 
`responsable` , 
`ubicacion` 
) values ('$codigo', '$responsable', '$ubicacion');"; 
    
    
 
    
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_hardware` ( 
`marca` , 
`modelo` , 
`serial` , 
`observaciones` , 
`id_hardware`, 
`codigo` 
) values ('$marcadiscoduro',  
   '$modelodiscoduro', '$serialdiscoduro', '$obsdiscoduro', '1' ,'$codigo'),('$marcamemoria', '$modelomemoria', '$serialmemoria', '$obsmemoria', '2' ,'$codigo'),('$marcaprocesador', '$modeloprocesador', '$serialprocesador', '$obsprocesador', '3' ,'$codigo'), 
   ('$marcatarjvideo', '$modelotarjvideo', '$serialtarjvideo', '$obstarjvideo', '4' ,'$codigo'),('$marcatarjred', '$modelotarjred', '$serialtarjred', '$obstarjred', '5' ,'$codigo'),('$marcatarjinal','$modelotarjinal','$serialtarjinal','$obstarjinal','6' ,'$codigo'), 
   ('$marcatarjaudio','$modelotarjaudio','$serialtarjaudio','$obstarjaudio', '7' ,'$codigo'),('$marcalector','$modelolector','$seriallector','$obslector', '8' ,'$codigo');"; 
    
   
 
 
 
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_software` ( 
`licencia` , 
`version` , 
`fabricante` , 
`observaciones` , 
`id_software`, 
`codigo` 
) values ('$licenciaso', '$versionso', '$sofabricante', '$obsso','1' ,'$codigo'),('$licenciahoja','$versionhoja','$fabricantehoja','$obshoja','2' ,'$codigo'), 
   ('$licenciatexto','$versiontexto','$fabricantetexto','$obstexto','3' ,'$codigo'),('$licenciapresent','$versionpresent','$fabricantepresent','$obspresent','4' ,'$codigo'),('$licenciabd','$versionbd','$fabricantebd','$obsbd','5' ,'$codigo'),('$licencianav','$versionnav','$fabricantenav','$obsnav','6' ,'$codigo'), 
   ('$licencialeng','$versionleng','$fabricanteleng','$obsleng','7' ,'$codigo'),('$licenciaant','$versionant','$fabricanteant','$obsant','8' ,'$codigo'),('$licenciaotros','$versionotros','$fabricanteotros','$obsotros','9' ,'$codigo')";  
    
   $insertar=mysqli_multi_query($this->conectar(),$sql); 
   if(!$insertar) {  
        
        echo ("<script language='javascript'> 
        alert('Error'); 
        </script>"); 
   } 
   else { 
   echo ("<script language='javascript'> 
        alert('Registro Incluido Exitosamente'); 
        </script>"); 
   } 
   
   } 
    
    
   
  }

carlos_belisario · #2 (**permalink**) 10/07/2013, 19:17

Si estas utilizando OOP deberías de sacar provecho a mysqli que tiene soporte para objetos,
además de esto, si haces un llamado al método conectar() una sola vez ya tienes establecida la conexión no tienes que estar haciendolo cada vez que haces el query para el insert, pero para responder tu duda

que error te da??

y para que la depuración sea mejor deberías de usar
http://www.php.net/manual/es/mysqli.error.php

saludos

Italico76 · #3 (**permalink**) 10/07/2013, 19:20

Jamas, por seguridad, hagas esto:

Cita:

extract($_POST);

Y en todo caso NO se hace DENTRO del Modelo.

Por otro lado deberias implementar alguno de los patrones de diseño para bases de datos: ActiveRecord o DataMapper

AnGuisi · #4 (**permalink**) 10/07/2013, 19:27

Cita:

Iniciado por Italico76

Jamas, por seguridad, hagas esto:

Y en todo caso NO se hace DENTRO del Modelo.

Por otro lado deberias implementar alguno de los patrones de diseño para bases de datos: ActiveRecord o DataMapper

:O Porque no debo usarla, hasta ahora me funciona a la perfección.

AnGuisi · #5 (**permalink**) 10/07/2013, 19:28

Cita:

Iniciado por carlos_belisario

Si estas utilizando OOP deberías de sacar provecho a mysqli que tiene soporte para objetos,
además de esto, si haces un llamado al método conectar() una sola vez ya tienes establecida la conexión no tienes que estar haciendolo cada vez que haces el query para el insert, pero para responder tu duda

que error te da??

y para que la depuración sea mejor deberías de usar
[url]http://www.php.net/manual/es/mysqli.error.php[/url]

saludos

El error esta en la llamada al metodo conectar. Parece que como parametro a mysqli_multi_query esta mal.

Italico76 · #6 (**permalink**) 10/07/2013, 19:29

Si usas extract() sobre $_POST o $_GET, etc... corres muchisimo riesgo de INJECTIONS o sea que alguien te envie algo que tu no esperas y se te convierta en variable local y dependiendo en que parte de tu codigo hagas ese extract() puede ser catastrofico

PD: lo que te dice @carlos_belisario es cierto, la extension MySql esta OBSOLETA y puede sacar ventajas OO si usas PDO o mysqli

carlos_belisario · #7 (**permalink**) 10/07/2013, 19:36

http://www.php.net/manual/es/function.extract.php el manual es tu amigo, ahí te da la sugerencia que te esta diciendo italico76

en teoría deberías de hacer algo así

Código PHP:

Ver originalclass clssentenciasmysql 
{ 
    private $con;
    public function conectar() { 
        $this->con=mysqli_connect("localhost","root","root123","soptecvnzla"); 
        if (mysqli_connect_errno($this->con)) 
       { 
           echo "Error en Conexión: " . mysqli_connect_error(); 
       }
    public insertar() 
    {
         //todo lo que tienes de tu metodo insert
         $insertar=mysqli_multi_query($this->con,$sql);  
        //lo que continua
    }
}

pero si te soy sincero el diseño de la clase no esta muy bien hecho (critica constructiva aveces cuando se escribe no se ve bien el tono), en el foro hay varios ejemplos de este tipo de clases, pudieras buscarlas para tomar ejemplo, saludos

AnGuisi · #8 (**permalink**) 10/07/2013, 19:47

Okey tomando en consideración algunas de las criticas he reestructurado el codigo de la siguiente manera.

Código PHP:

  <? 
 
 
class clssentenciasmysql { 
 
    function conectar () { 
              $mysqli=new mysqli_connect("localhost","root","root123","soptecvnzla"); 
 
if ($mysqli->connect_errno) 
  { 
  echo "Error en Conexión: " . $mysqli->connect_error; 
  } 
    } 
 
    function insertar () { 
 
   
  extract($_POST); 
   
   $sql = "INSERT INTO  `soptecvnzla`.`equipo` ( 
`codigo` , 
`responsable` , 
`ubicacion` 
) values ('$codigo', '$responsable', '$ubicacion');"; 
    
    
 
    
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_hardware` ( 
`marca` , 
`modelo` , 
`serial` , 
`observaciones` , 
`id_hardware`, 
`codigo` 
) values ('$marcadiscoduro',  
   '$modelodiscoduro', '$serialdiscoduro', '$obsdiscoduro', '1' ,'$codigo'),('$marcamemoria', '$modelomemoria', '$serialmemoria', '$obsmemoria', '2' ,'$codigo'),('$marcaprocesador', '$modeloprocesador', '$serialprocesador', '$obsprocesador', '3' ,'$codigo'), 
   ('$marcatarjvideo', '$modelotarjvideo', '$serialtarjvideo', '$obstarjvideo', '4' ,'$codigo'),('$marcatarjred', '$modelotarjred', '$serialtarjred', '$obstarjred', '5' ,'$codigo'),('$marcatarjinal','$modelotarjinal','$serialtarjinal','$obstarjinal','6' ,'$codigo'), 
   ('$marcatarjaudio','$modelotarjaudio','$serialtarjaudio','$obstarjaudio', '7' ,'$codigo'),('$marcalector','$modelolector','$seriallector','$obslector', '8' ,'$codigo');"; 
    
   
 
 
 
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_software` ( 
`licencia` , 
`version` , 
`fabricante` , 
`observaciones` , 
`id_software`, 
`codigo` 
) values ('$licenciaso', '$versionso', '$sofabricante', '$obsso','1' ,'$codigo'),('$licenciahoja','$versionhoja','$fabricantehoja','$obshoja','2' ,'$codigo'), 
   ('$licenciatexto','$versiontexto','$fabricantetexto','$obstexto','3' ,'$codigo'),('$licenciapresent','$versionpresent','$fabricantepresent','$obspresent','4' ,'$codigo'),('$licenciabd','$versionbd','$fabricantebd','$obsbd','5' ,'$codigo'),('$licencianav','$versionnav','$fabricantenav','$obsnav','6' ,'$codigo'), 
   ('$licencialeng','$versionleng','$fabricanteleng','$obsleng','7' ,'$codigo'),('$licenciaant','$versionant','$fabricanteant','$obsant','8' ,'$codigo'),('$licenciaotros','$versionotros','$fabricanteotros','$obsotros','9' ,'$codigo')";  
    
   $insertar=$mysqli->multi_query($sql); 
   if(!$insertar) {  
        
        echo ("<script language='javascript'> 
        alert('Error'); 
        location.href='registro.html'; 
        </script>"); 
   } 
   else { 
   echo ("<script language='javascript'> 
        alert('Registro Incluido Exitosamente'); 
        location.href='registro.html'; 
        </script>"); 
   } 
   
   } 
    
   function consultar() { 
        
   
       extract($_POST); 
 
 
            $sql= "SELECT equipo.codigo,   
equipo.responsable,   
equipo.ubicacion,   
caracteristicas_hardware.marca,   
caracteristicas_hardware.observaciones,   
caracteristicas_hardware.serial,   
caracteristicas_hardware.modelo,   
caracteristicas_software.fabricante,   
caracteristicas_software.licencia,   
caracteristicas_software.observaciones,   
caracteristicas_software.version,   
software.tipo,   
hardware.tipo FROM    
hardware,   
caracteristicas_hardware,   
software,   
caracteristicas_software,   
equipo   
WHERE   
equipo.codigo = caracteristicas_hardware.codigo   
AND   
caracteristicas_hardware.id_hardware = hardware.id_hardware   
AND   
equipo.codigo = caracteristicas_software.codigo   
AND   
caracteristicas_software.id_software = software.id_software 
AND 
'$codigo' = `equipo`.`codigo`";   
 
$result=$mysqli->query($sql);   
 
if (!$result) { 
 
 echo("Error al consultar" . $mysqli->error); 
} 
 
while ($row = mysqli_fetch_array($result)) { 
       
      extract ($row); 
 
} 
 
  } 
   
  }

Me arroja el siguiente error.

Call to a member function query() on a non-object.

Y segun el manual esa es la forma adecuada de hacer el llamado.

carlos_belisario · #9 (**permalink**) 10/07/2013, 20:05

Ok, digamos que deberías de elegir una sola opción o usas el método funcional

Código PHP:

Ver originalmysqli_connect();
mysqli_query();
...

o usas el método orientado a objetos

Código PHP:

Ver original$mysqli = new mysqli('localhost', 'mi_usuario', 'mi_contraseña', 'mi_bd');
$mysqli->query();
...

pero usar ambos se ve mal en la organización del código

Cita:

Código PHP:

  $result=$mysqli->query($sql);

te da error porque simplemente no encuentra $mysqli como un objeto y esto porque?? porque simplemente no lo has instanciado.

Como te dije tienes que acomodar mejor el diseño de tu clase, ver un poco más de ejemplos de OOP, porque entiendo que estas comenzando con este paradigma pero no es simplemente llevar funciones a objetos, es ver bien que es un objeto cual es su responsabilidad a nivel de la aplicación hacer uso de la herencia, etc.

Disculpa si sueno rudo (espero que no), pero el objetivo no es decepcionarte ni desmotivarte sino mas bien motivarte a hacer las cosas bien, saludos

AnGuisi · #10 (**permalink**) 10/07/2013, 21:49

Por el contrario te agradezco mucho tus sugerencias. Donde podría ver ejemplos de los que hablas?

AnGuisi · #11 (**permalink**) 10/07/2013, 22:10

Esto hasta ahora funciona bien.

Código PHP:

  <? 
 
 
class clssentenciasmysql { 
 
     
  
 
    function insertar () { 
 
         $mysqli = new mysqli('localhost', 'root', 'root123', 'soptecvnzla'); 
 
if ($mysqli->connect_errno) 
  { 
  echo "Error en Conexión: " . $mysqli->connect_error; 
  } 
 
   
  extract($_POST); 
   
   $sql = "INSERT INTO  `soptecvnzla`.`equipo` ( 
`codigo` , 
`responsable` , 
`ubicacion` 
) values ('$codigo', '$responsable', '$ubicacion');"; 
    
    
 
    
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_hardware` ( 
`marca` , 
`modelo` , 
`serial` , 
`observaciones` , 
`id_hardware`, 
`codigo` 
) values ('$marcadiscoduro',  
   '$modelodiscoduro', '$serialdiscoduro', '$obsdiscoduro', '1' ,'$codigo'),('$marcamemoria', '$modelomemoria', '$serialmemoria', '$obsmemoria', '2' ,'$codigo'),('$marcaprocesador', '$modeloprocesador', '$serialprocesador', '$obsprocesador', '3' ,'$codigo'), 
   ('$marcatarjvideo', '$modelotarjvideo', '$serialtarjvideo', '$obstarjvideo', '4' ,'$codigo'),('$marcatarjred', '$modelotarjred', '$serialtarjred', '$obstarjred', '5' ,'$codigo'),('$marcatarjinal','$modelotarjinal','$serialtarjinal','$obstarjinal','6' ,'$codigo'), 
   ('$marcatarjaudio','$modelotarjaudio','$serialtarjaudio','$obstarjaudio', '7' ,'$codigo'),('$marcalector','$modelolector','$seriallector','$obslector', '8' ,'$codigo');"; 
    
   
 
 
 
   $sql .= "INSERT INTO  `soptecvnzla`.`caracteristicas_software` ( 
`licencia` , 
`version` , 
`fabricante` , 
`observaciones` , 
`id_software`, 
`codigo` 
) values ('$licenciaso', '$versionso', '$sofabricante', '$obsso','1' ,'$codigo'),('$licenciahoja','$versionhoja','$fabricantehoja','$obshoja','2' ,'$codigo'), 
   ('$licenciatexto','$versiontexto','$fabricantetexto','$obstexto','3' ,'$codigo'),('$licenciapresent','$versionpresent','$fabricantepresent','$obspresent','4' ,'$codigo'),('$licenciabd','$versionbd','$fabricantebd','$obsbd','5' ,'$codigo'),('$licencianav','$versionnav','$fabricantenav','$obsnav','6' ,'$codigo'), 
   ('$licencialeng','$versionleng','$fabricanteleng','$obsleng','7' ,'$codigo'),('$licenciaant','$versionant','$fabricanteant','$obsant','8' ,'$codigo'),('$licenciaotros','$versionotros','$fabricanteotros','$obsotros','9' ,'$codigo')";  
    
    
    
    
   $insertar=$mysqli->multi_query($sql); 
   if(!$insertar) {  
        
        echo ("<script language='javascript'> 
        alert('Error'); 
        location.href='registro.html'; 
        </script>"); 
   } 
   else { 
   echo ("<script language='javascript'> 
        alert('Registro Incluido Exitosamente'); 
        location.href='registro.html'; 
        </script>"); 
   } 
   
   } 
    
   function consultar() { 
                $mysqli = new mysqli('localhost', 'root', 'root123', 'soptecvnzla'); 
 
if ($mysqli->connect_errno) 
  { 
  echo "Error en Conexión: " . $mysqli->connect_error; 
  } 
   
       extract($_POST); 
 
 
            $sql= "SELECT equipo.codigo,   
equipo.responsable,   
equipo.ubicacion,   
caracteristicas_hardware.marca,   
caracteristicas_hardware.observaciones,   
caracteristicas_hardware.serial,   
caracteristicas_hardware.modelo,   
caracteristicas_software.fabricante,   
caracteristicas_software.licencia,   
caracteristicas_software.observaciones,   
caracteristicas_software.version,   
software.tipo,   
hardware.tipo FROM    
hardware,   
caracteristicas_hardware,   
software,   
caracteristicas_software,   
equipo   
WHERE   
equipo.codigo = caracteristicas_hardware.codigo   
AND   
caracteristicas_hardware.id_hardware = hardware.id_hardware   
AND   
equipo.codigo = caracteristicas_software.codigo   
AND   
caracteristicas_software.id_software = software.id_software 
AND 
'$codigo' = `equipo`.`codigo`";   
 
$result=$mysqli->query($sql); 
 
if (!$result) { 
 
 echo("Error al consultar" . $mysqli->error); 
} 
 
while ($row = $result->fetch_array()) { 
       
      extract ($row); 
 
} 
echo $responsable; 
echo $ubicacion; 
  } 
   
  }

Mi pregunta ahora seria ¿Como hago la conexion desde otra funcion y poder utilizarla en las demas? Siguiendo la misma metodologia.

carlos_belisario · #12 (**permalink**) 11/07/2013, 06:12

Cita:

Iniciado por AnGuisi

Por el contrario te agradezco mucho tus sugerencias. Donde podría ver ejemplos de los que hablas?

puedes comenzar por acá http://php.net/manual/es/language.oop5.php

y de ahí te aseguro que la segunda duda se te resuelve sola, saludos