0

Miren, tengo el siguiente error

Column count doesn't match value count at row 1

acá el código del mysql

        $sql = "INSERT INTO 
                    topics(topic_subject,
                           topic_date,
                           topic_cat,
                           topic_by)
               VALUES('" . mysqli_real_escape_string($con, $_POST['topic_subject']) . ",
                           NOW(),
                           " . mysqli_real_escape_string($con, $_POST['topic_cat']) . ",
                           " . $_SESSION['user_id'] . "
                           ')";

        $result = mysqli_query($con, $sql);
        if(!$result)
        {
            //something went wrong, display the error
            echo '' . mysqli_error($con) . '';
            die();
        }

Y tengo una duda, es posible preparar el mysql sin necesidad de cambiar todo el código ?

7
  • el código completo va en la pregunta, edita tu duda y colócalo aquí
    – user75901
    Commented el 11 feb. 2019 a las 5:20
  • Hola, por favor edita tu pregunta y coloca el código aquí, no en un sitio externo.
    – RockoDev
    Commented el 11 feb. 2019 a las 5:23
  • el código es largo, ese es el problema, 126 líneas, pondré el del mysql que es usado Commented el 11 feb. 2019 a las 5:36
  • Pareciera un error de concatenación, al campo topic_cat le hace falta añadir las comillas simples como tiene para el campo topic_subject, si es que se tratase de un valor cadena. Igual debe revisar y elegir utilizar sentencias preparadas por la seguridad de su código.
    – Dev. Joel
    Commented el 11 feb. 2019 a las 5:53
  • me acabo de dar cuenta que agregé una comilla que no tendría que estar, la voy a quitar a ver si se soluciona Commented el 11 feb. 2019 a las 5:59

1 respuesta 1

2

Prueba esto:

$sql = "INSERT INTO 
                    topics(topic_subject,
                           topic_date,
                           topic_cat,
                           topic_by)
               VALUES('" . mysqli_real_escape_string($con, $_POST['topic_subject']) . "' ,
                           NOW(),
                           '" . mysqli_real_escape_string($con, $_POST['topic_cat']) . "',
                           " . $_SESSION['user_id'] . "
                           )";

Tienes un lío de comillas...

2
  • En lugar de mysqli_real_escape_string es mejor usar consultas preparadas.
    – A. Cedano
    Commented el 11 feb. 2019 a las 10:06
  • Tienes razón pero yo solamente cambié la consulta original para igualar las comillas Commented el 11 feb. 2019 a las 12:42

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.