[SOLUCIONADO] error con INSERT INTO

[guardarmicorreo]

He creado una clase para acceder a la base de datos de forma segura, la estoy probando en local y me da errores.

No encuentro dónde está el error :S

En el código (indico las líneas del error y más abajo dejo los errores devueltos)

Código PHP:

class Conectar
{
    
    public function conectar ( $string_consulta ) //línea 327
    {
        //establezco los datos de autenticación para acceder a la base de datos
        $IDConexion = mysqli_connect("localhost" , "root" , "" , "pruebas");
        
        //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
        mysqli_set_charset( $IDConexion, "utf8_spanish2_ci" );
        
        //escapo caracteres para evitar sql injection
        $Consulta = mysqli_escape_string( $IDConexion, $string_consulta ); //línea 336
        
        //conecto a la base de datos
        $IDConsulta = mysqli_query($IDConexion , $Consulta); //línea 339
        
        //retorno los resultados
        return $IDConsulta;
    }
}
 
$consulta = "INSERT INTO pruebas VALUES (null, 'pepe')";
 
$insertar = new Conectar(); //línea 348
 
$insertar->conectar($consulta);
 

Los errores

Warning: Missing argument 1 for Conectar::conectar(), called in C:\xampp\htdocs\xampp\web\ejemplo_formulario.php on line 348 and defined in C:\xampp\htdocs\xampp\web\ejemplo_formulario.php on line 327

Notice: Undefined variable: string_consulta in C:\xampp\htdocs\xampp\web\ejemplo_formulario.php on line 336

Warning: mysqli_query(): Empty query in C:\xampp\htdocs\xampp\web\ejemplo_formulario.php on line 339

¿Podeis ayudarme? :)

[jonni09lo]

No se que versión de PHP uses, pero por compatibilidad a versiones de PHP 4 los constructores de la clase se pueden llamar de la misma manera de la clase.

Me explico en PHP 4 para invocar al constructor se hacia de la manera

Código PHP:

Class Hola{
    function Hola(){
        echo "soy el constructor";
    }
}   
 
$HOLA = new Hola();
 
//imprime soy el constructor
 

PHP 5 por compatibilidad copia este comportamiento, así que te recomiendo que pruebes de esta manera:

Código PHP:

class Conectar
{
    public function __construct(){
        
    }
    
    public function conectar ( $string_consulta ) //línea 327
    {
        //establezco los datos de autenticación para acceder a la base de datos
        $IDConexion = mysqli_connect("localhost" , "root" , "" , "pruebas");
        
        //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
        mysqli_set_charset( $IDConexion, "utf8_spanish2_ci" );
        
        //escapo caracteres para evitar sql injection
        $Consulta = mysqli_escape_string( $IDConexion, $string_consulta ); //línea 336
        
        //conecto a la base de datos
        $IDConsulta = mysqli_query($IDConexion , $Consulta); //línea 339
        
        //retorno los resultados
        return $IDConsulta;
    }
}
 

Espero se entienda

Saludos

[guardarmicorreo]

Iniciado por jonni09lo

No se que versión de PHP uses, pero por compatibilidad a versiones de PHP 4 los constructores de la clase se pueden llamar de la misma manera de la clase.

Me explico en PHP 4 para invocar al constructor se hacia de la manera

Código PHP:

Class Hola{
    function Hola(){
        echo "soy el constructor";
    }
}   
 
$HOLA = new Hola();
 
//imprime soy el constructor
 

PHP 5 por compatibilidad copia este comportamiento, así que te recomiendo que pruebes de esta manera:

Código PHP:

class Conectar
{
    public function __construct(){
        
    }
    
    public function conectar ( $string_consulta ) //línea 327
    {
        //establezco los datos de autenticación para acceder a la base de datos
        $IDConexion = mysqli_connect("localhost" , "root" , "" , "pruebas");
        
        //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
        mysqli_set_charset( $IDConexion, "utf8_spanish2_ci" );
        
        //escapo caracteres para evitar sql injection
        $Consulta = mysqli_escape_string( $IDConexion, $string_consulta ); //línea 336
        
        //conecto a la base de datos
        $IDConsulta = mysqli_query($IDConexion , $Consulta); //línea 339
        
        //retorno los resultados
        return $IDConsulta;
    }
}
 

Espero se entienda

Saludos

Muchas gracias :D te entendí perfectamente.

He probado lo que me has dicho, ahora aunque no me arroja ningún error, sigue sin realizar la inserción.

Mi versión de PHP es la 5.4.19

Edito, mi tabla es muy sencilla, es la siguiente:

Nombre pruebas

campos id|nombre

He comprobado las credenciales para acceder y son correctas. Debería funcionar y no se por qué :S

[jonni09lo]

Base de datos, tipo de datos, y el sql que generas todo eso has revisado?

En alguno de estos puntos debe fallar, incluso para salir de dudas puedes cambiar tu nombre de la función

Saludos

[guardarmicorreo]

Iniciado por jonni09lo

Base de datos, tipo de datos, y el sql que generas todo eso has revisado?

En alguno de estos puntos debe fallar, incluso para salir de dudas puedes cambiar tu nombre de la función

Saludos

Vale, he probado a comentar el mysqli_set_charset() y el mysqli_real_escape_string(), cambiando el 2º parámetro de mysqli_query por $string_consulta y sí hace la inserción.

No entiendo por qué, si los he puesto siguiendo el manual de PHP.

El código quedó así

Código PHP:

class Conectar
{
    public function __construct()
    {
    }
    
    public function conectar ( $string_consulta )
    {
        //establezco los datos de autenticación para acceder a la base de datos
        $IDConexion = mysqli_connect("localhost" , "root" , "" , "pruebas");
        
        //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
        //mysqli_set_charset( $IDConexion, "utf8_spanish2_ci" );
        
        //escapo caracteres para evitar sql injection
        //$Consulta = mysqli_escape_string( $IDConexion, $string_consulta );
        
        //conecto a la base de datos
        $IDConsulta = mysqli_query($IDConexion , $string_consulta);
        
        //retorno los resultados
        return $IDConsulta;
    }
}
 
$consulta = "INSERT INTO pruebas VALUES (null, 'pepe')";
 
$insertar = new Conectar();
 
$insertar->conectar($consulta);
 

[jonni09lo]

Puede ser debido a lo que dice la documentación:

Precaución
Seguridad: el conjunto de caracteres predeterminado

El conjunto de caracteres debe ser establecido a nivel del servidor, o con la función mysqli_set_charset() de la API para que afecte a mysqli_real_escape_string(). Véase la sección de conceptos sobre conjuntos de caracteres para más información.

Saludos

[eits]

el problema esta en que cuando realizar el new Conectar(), se ejecuta el la funcionr Conectar( $string_consulta), es decir necesita un parametro lo que debes de hacer es

Código PHP:

$consulta = "INSERT INTO pruebas VALUES (null, 'pepe')";
$insertar = new Conectar($consulta); //línea 348
 

[guardarmicorreo]

Iniciado por eits

el problema esta en que cuando realizar el new Conectar(), se ejecuta el la funcionr Conectar( $string_consulta), es decir necesita un parametro lo que debes de hacer es

Código PHP:

$consulta = "INSERT INTO pruebas VALUES (null, 'pepe')";
$insertar = new Conectar($consulta); //línea 348
 

Gracias, pero no sirvió.

[xSkArx]

Por lo que veo tienes puesto mysqli_escape_string y segun el manual es mysqli_real_escape_string, ademas que hasta donde yo se mysqli_real_escape_string() es para los valores que van en la consulta, no para la cosulta propiamemte tal.

[eits]

te dejo este código no lo he podido verificar, pero te debe funcionar o por lo menos te dará una idea, aclaro, se puede mejorar.

Código PHP:

class Conectar
    {
        private $IDConexion='';//creamos una propiedad privada
       
        public function __construct()
        {
            //establezco los datos de autenticación para acceder a la base de datos
            $IDConexion = mysqli_connect("localhost" , "root" , "" , "pruebas");
            //establezco el conjunto de caracteres a nivel de servidor en tiempo de ejecucion
            mysqli_set_charset( $IDConexion, "utf8_spanish2_ci" );
        }
        public function consultar($string_consulta){
            //ejecuto la consulta
           if(mysqli_query($this->IDConexion , $string_consulta))
                return true;
            else
                die(mysqli_error());
        }
        public function escapar($string){
            //escapo la cadena de caracteres para evitar la inyección de sql
            return mysqli_escape_string($string);
        }
    }
     
    $nombre='pepe';//se asigna por $_POST[] o $_GET[]
    $insertar = new Conectar();//instanciamos la clase
    //creamos la consulta y escapamos
    $consulta = 'INSERT INTO pruebas VALUES (null, "'.$insertar->escapar($nombre).'")';  
    $insertar->consultar($consulta);
 

saludos.

[dagyus]

La consulta insert esta mal ya que es insert into pruebas(id,nombre) values ( NULL, 'pepe')

[guardarmicorreo]

Iniciado por SkAr88

Por lo que veo tienes puesto mysqli_escape_string y segun el manual es mysqli_real_escape_string, ademas que hasta donde yo se mysqli_real_escape_string() es para los valores que van en la consulta, no para la cosulta propiamemte tal.

Gracias!! exactamente fué ese mi error. Ahora sí funciona si solo escapo los valores.

Iniciado por dagyus

La consulta insert esta mal ya que es insert into pruebas(id,nombre) values ( NULL, 'pepe')

No, eso no es así. En la documentación de mysql dice

La lista de nombres de columna o la cláusula SET indican las columnas explícitamente.

Si no especifica la lista de columnas para INSERT ... VALUES o INSERT ... SELECT, los valores para cada columna en la tabla deben proporcionarse en la lista VALUES o por el SELECT. Si no sabe el orden de las columnas en la tabla, use DESCRIBE tbl_name para encontrarlo.

Gracias a todos por la ayuda ofrecida.

EDITO

Otra cosa que hay que tener en cuenta es el conjunto de caracteres soportados en la BD. En mi caso después de hacer toda la clase bien, depurando código me devolvía que el conjunto de caracteres utf8_spanish2_ci no estaba soportado por la BD, supongo que se refiriría a nivel global, porque en las tablas sí puedo establecer ese conjunto de caracteres.