0

En el primer ejemplo, me sale error... Le hice un var_dump a la variable y pues contiene todo, pero aún así da error. En el segundo ejemplo, quito la variable y pongo un texto cualquiera y sin problemas edita todo en la base de datos, entonces que puede estar pasando?

// CON ESTE EJEMPLO ME DA ERROR
if (strlen($message_by) > 0 && strlen($message_content) > 0) {
            
            $x = mysqli_query($base,"UPDATE $accounts SET message_content = '$message_content' WHERE message_by = '$message_by' OR message_destiny = '$message_by'"); 
            
            if (!$x) {
                
                echo "Error".var_dump($message_content);
                
            } else {
                
                echo "Editado";
                
            }
            
        }
        
// CON ESTE EJEMPLO ME MUESTRA EL 'EDITADO' Y EN LA BASE DE DATOS TODO CAMBIA
if (strlen($message_by) > 0 && strlen($message_content) > 0) {
            
            $x = mysqli_query($base,"UPDATE $accounts SET message_content = 'Ya no llamo la variable' WHERE message_by = '$message_by' OR message_destiny = '$message_by'"); 
            
            if (!$x) {
                
                echo "Error".var_dump($message_content);
                
            } else {
                
                echo "Editado";
                
            }
            
        }

4
  • ¿Cuál es el mensaje de error del primer script?
    – user128299
    Commented el 4 oct. 2021 a las 4:27
  • @BetaM El echo que tengo definido... Commented el 4 oct. 2021 a las 4:28
  • Revisa el ejemplo procedimental que ayuda a obtener el posible fallo en la query: php.net/manual/es/… una vez hecho eso por favor agrégalo a tu pregunta
    – user128299
    Commented el 4 oct. 2021 a las 4:31
  • @BetaM ErrorYou have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near Commented el 5 oct. 2021 a las 1:34

1 respuesta 1

1

Dos cosas:

  1. Falta el else de este bloque: if (strlen($message_by) > 0 && strlen($message_content) > 0) { Nunca dejes áreas sin cubrir en el flujo de tu código. Si hay una situación de else aquí lo único que el usuario tendrá es una pantalla en blanco.

  2. En fase de depuración muestra el error en sí mismo, así sabrás con más exactitud lo que estaría pasado. Uno de los posibles errores aquí es un duplicated key. Y, aparte de eso, en consultas de tipo UPDATE, INSERT o DELETE, convendría verificar la cantidad de filas afectadas. Una consulta de este tipo puede ser exitosa (no dar error) pero no modificar nada, porque el criterio WHERE no se cumple, o porque no hay nada que actualizar (en caso de mandar a actualizar con datos que son los mismos que ya existen).

Haciendo esto, deberías saber con exactitud lo que está ocurriendo:

// CON ESTE EJEMPLO YA NO ME DA ERROR
if (strlen($message_by) > 0 && strlen($message_content) > 0) {                
    $x = mysqli_query($base,"UPDATE $accounts SET message_content = '$message_content' WHERE message_by = '$message_by' OR message_destiny = '$message_by'");                
    if (!$x) {
        #Sólo en fase de depuración. Quitar luego mysqli_error
        echo "Error: ".mysqli_error($base);
    } else {
        echo "Filas editadas: ".mysqli_affected_rows($base);
    }
} else {
    echo "No se cumple la condición inicial";
}

Dos recomendaciones

  1. Tu código es altamente vulnerable a ataques de Inyección SQL. Considera usar consultas preparadas para corregir ese gravísimo problema de seguridad.
  2. Considera migrar tu código mysqli al estilo orientado a objetos. Es más claro y menos verboso que el estilo procedural.
10
  • Aparece: ErrorYou have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near Commented el 5 oct. 2021 a las 1:33
  • Ese error me sale junto a esta linea donde es un json que trato de guardar, creo que es por las comillas?: 'message_by':'[email protected]','message_content':'d','message_destiny':'loca@gmail' Commented el 5 oct. 2021 a las 2:16
  • @NuevoUsuario depura tu código. Escribe la instrucción SQL en una variable: $sql="UPDATE $accounts SET message_content = '$message_content' WHERE message_by = '$message_by' OR message_destiny = '$message_by'"; y luego imprímela y revísala: echo $sql; Si no logras ver el error muéstranos el contenido que tendría $sql. Es una cuestión básica de depuración. Si algo no funciona, revísalo, analízalo. Puede que en la variables que contiene la instrucción SQL haya datos extraños que rompan el código, creando un error de sintaxis.
    – A. Cedano
    Commented el 5 oct. 2021 a las 9:38
  • El error de sintaxis es lo menos grave que te podría pasar. Como señalé al final de la respuesta, tu código está expuesto a ataques de Inyección SQL, por tanto, la solución definitiva a esto será implementar consultas preparadas.
    – A. Cedano
    Commented el 5 oct. 2021 a las 9:51
  • No uso las consultas preparadas porque lo único que me hace falta es obtener el la información con while. Commented el 6 oct. 2021 a las 4:32

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.