INSERT - No consigo que funcione!

[daxxdj]

Código PHP:

  <?PHP

$name = $_REQUEST["name"];
$mail = $_REQUEST["mail"];
$coment = $_REQUEST["coment"];
$idimg = $_REQUEST["idimg"];

$intro = (NULL . ", " . $idimg . ", " . $name . ", " . $coment . ", " . $mail);

$conexion = mysql_connect("localhost", "XXXXXX", "XXXXXX");
mysql_select_db ("daxx13_lfdao");
$intro = mysql_query ("INSERT INTO comentarios (idcom, idimg, nombre, comentario, email) VALUES (" . $intro . ")") or die("error!");

?>   

He probado muchas combinaciones (llevo 2 horas de probar y probar) y no he conseguido insertar NADA. Una ayudita... :)

Gracias!

[pateketrueke]

Las cadenas de texto que insertas en la BD deben ir entre comillas.

VALUES('x', 'y', '...')

[daxxdj]

Iniciado por pateketrueke

Las cadenas de texto que insertas en la BD deben ir entre comillas.

Dobles o simples?

[TheM4ster]

Si guardas la coneccion en una variable, es porque lo vas a utilizar mas adelante,
y no veo q lo hagas,

$conexion = mysql_connect("localhost", "XXXXXX", "XXXXXX");

prueba asi,

$conexion = mysql_connect("localhost", "XXXXXX", "XXXXXX");
mysql_select_db ("daxx13_lfdao",$conexion);
$intro = mysql_query ("INSERT INTO comentarios (idcom, idimg, nombre, comentario, email) VALUES (" . $intro . ")",$conexion) or die("error!");

si no funciona, copia la estructura de la tabla ?

[loncho_rojas]

pregunto y de paso afirmo, jajaj.

No sera que si tengo 4 campos donde insertar, deben haber 4 variables para introducir?

Seria bueno que si te salta un error nos lo postees tambien

[cachusan]

Estimado, dos cosas:
- tomar los datos con $_REQUEST puede ser peligroso
- no sanitizarlos tambien, posible sql injection

Consejos:
- Especifica explicitamente de donde vienen los datos, $_GET o $_POST
- hace trim() a los valores
- mysql_real_escape_string($var) para sanitizar los valores
- Separa el codigo reutilizable lo mas posible, por ejemplo el primer bloque de codigo lo podes meter un archivo php y con un include('archivo.php') reutilizar la conexion.

Código PHP:

  
<?php
// conexion a servidor mysql 

/* Datos de xonexion */
$dbhost = 'localhost';
$dbuser = 'root';
$dbpass = 'password';

/* conecto */
$conn = mysql_connect($dbhost, $dbuser, $dbpass) or die ('Error conectando al servidor mysql');

/* seleccion de db */
$dbname = 'daxx13_lfdao';
mysql_select_db($dbname);
?>

<?php

// recibo los datos
$name = $_REQUEST["name"];
$mail = $_REQUEST["mail"];
$coment = $_REQUEST["coment"];
$idimg = $_REQUEST["idimg"];

// genero string para valores a ingresar
$values = "null, '{$idimg}', '{$name}', '{$coment}', '{$mail}' ";

// genero la query de insercion
$query = "INSERT INTO comentarios (idcom, idimg, nombre, comentario, email) VALUES ({$values})"

// ejecuto consulta de insercion
$result = mysql_query ($query) or die("error!");

?>   

[ecarlevaro]

Pero pone los cuatro campos, y los cuatro valores, el primero es NULL.

Hay dos errores de sintáxis en la formación de la consulta, $intro ya tiene paréntesis, y la concatenas dentro de un string con paréntesis:

Código PHP:

$intro = (NULL . ", " . $idimg . ", " . $name . ", " . $coment . ", " . $mail);
$intro = mysql_query ("INSERT INTO comentarios (idcom, idimg, nombre, comentario, email) VALUES (" . $intro . ")") or die("error!");
 

O quitas los paréntesis de $intro o quitas los paréntesis del string de mysql_query.

Y segundo, $intro = (NULL ¡ERROR! ¿y las comillas que indican que (NULL es una cadena?.

Consejo 1:
en el "or die('error!')" coloca la función mysql_error que te va a dar mas descripción durante la etapa de desarrollo de la aplicación y además concatenala con la consulta, que te imprima en pantalla la consulta SQL completa y ahi veras el error enseguida.

Consejo 2:
No te compliques con las comillas. Las comillas dobles PHP las interpreta hacia adentro reemplazando las variables por su valor (http://ar2.php.net/manual/es/language.types.string.php) por lo que tu $intro es mejor escribirla así:

Código PHP:

$intro = "(NULL,'$idimg','$name','$comment','$mail')";
 

Y listo. Recuerda quitar los dobles paréntesis.