Muy básico pero no doy con el error

mordvinia · #1 (**permalink**) 04/01/2015, 18:21

Saludos. En la línea 9 tengo un error pero sinceramente no lo veo. Agradezco su ayuda.

Código PHP:

 
<html> 
<head> 
<title>Ingresar</title> 
</head> 
<body> 
<?php 
$conexion = mysql_connect("localhost", "root", "") or die("Problemas en la conexion"); 
mysql_select_db("base", $conexion) or die("Problemas en la seleccion de la base de datos"); 
mysql_query (''INSERT INTO datos (cedula, apellido, nombre) VALUES ('$_POST[cedula]','$_POST[apellido]','$_POST[nombre]')'' $conexion) or die("Problemas en el select" . mysql_error()); 
mysql_close($conexion); 
echo "Registro exitoso"; 
?> 
</body> 
</html>

gnzsoloyo · #2 (**permalink**) 04/01/2015, 18:23

Creo que estás confundiendo doble apóstrofe (') con comillas ("). No es lo mismo...

Código PHP:

Ver originalmysql_query ("INSERT INTO datos (cedula, apellido, nombre) 
VALUES ('$_POST[cedula]','$_POST[apellido]','$_POST[nombre]')", $conexion) 
or die("Error en ejecución de consulta: " . mysql_error());

acopantepuy · #3 (**permalink**) 04/01/2015, 18:29

Debes cambiar ' ' por " y colocar la coma (,) despues de la sentencia SQL:

Código PHP:

Ver originalmysql_query ("INSERT INTO datos (cedula, apellido, nombre) VALUES ('$_POST[cedula]','$_POST[apellido]','$_POST[nombre]')", $conexion) or die("Problemas en el select" . mysql_error());

moginn · #4 (**permalink**) 04/01/2015, 18:34

además de las comillas dobles, realmente se puede poner como ha puesto el amigo?

Yo siempre lo hago así

Código PHP:

Ver originalmysql_query ("INSERT INTO datos (cedula, apellido, nombre) VALUES ('" . $_POST[cedula] . "','" . $_POST[apellido] . "','" . $_POST[nombre] . "'");

mordvinia · #5 (**permalink**) 04/01/2015, 18:37

Gracias gnzsoloyo y acopantepuy ya me funcionó ese código.

Dalam · #6 (**permalink**) 04/01/2015, 19:59

Cita:

Iniciado por moginn

además de las comillas dobles, realmente se puede poner como ha puesto el amigo?

Yo siempre lo hago así

Código PHP:

Ver originalmysql_query ("INSERT INTO datos (cedula, apellido, nombre) VALUES ('" . $_POST[cedula] . "','" . $_POST[apellido] . "','" . $_POST[nombre] . "'");

En optimizacion de codigo como nota informativa dire que la diferencia entre usar comillas dobles y simples es grande, debido a que cuando se usan comillas dobles php hace dos pasadas, una para sustitucion de variables y otra para la salida.
El uso que estas poniendo esta mal de perfomance, es mejor que uses comillas simples y que escapes las que usa la sentencia de mysql.
Por orden de perfomance seria de esta forma
1.Comillas simples y concatenacion.
2.Comillas dobles.
3.Comillas dobles y concatenacion.

No obstante esto apartir de PHP5 te da pocos milisegundos de optimizacion, y a no ser que tu programa sea extenso, ni lo notaras.
Una ventaja del uso de las comillas dobles es la concatenacion mas legible.

Cita:

$texto = " $uno $dos {$tres} {$cuatro} ";

Es mas legible queeste ejemplo de como seria con tu modo

Cita:

$texto = " " . $uno . " " . $dos . " " . $three . " " $four . " ";

moginn · #7 (**permalink**) 04/01/2015, 21:53

Cita:

Iniciado por Dalam

En optimizacion de codigo como nota informativa dire que la diferencia entre usar comillas dobles y simples es grande, debido a que cuando se usan comillas dobles php hace dos pasadas, una para sustitucion de variables y otra para la salida.
El uso que estas poniendo esta mal de perfomance, es mejor que uses comillas simples y que escapes las que usa la sentencia de mysql.

Pues no sabía que había tanta diferencia entre ambas, yo siempre las he usado indistintamente. Gracias por tu información!

Rodrigoespoz · #8 (**permalink**) 05/01/2015, 09:48

espero que te sirva

Código PHP:

  <?php  
$conexion = mysql_connect("localhost", "root", "") or die("Problemas en la conexion");  
mysql_select_db("base", $conexion) or die("Problemas en la seleccion de la base de datos");  
if(isset($_POST['cedula'])){ 
$ced=$_POST[cedula]; 
$app=$_POST[apellido]; 
$nom=$_POST[nombre]; 
mysql_query (''INSERT INTO datos (cedula, apellido, nombre) VALUES ('$ced','$app','$nom')'' $conexion) or die("Problemas en el select" . mysql_error());  
mysql_close($conexion);  
echo "Registro exitoso"; } 
?>

replica · #9 (**permalink**) 05/01/2015, 10:39

Cita:

Iniciado por Rodrigoespoz

espero que te sirva

Código PHP:

  <?php 
$conexion = mysql_connect("localhost", "root", "") or die("Problemas en la conexion"); 
mysql_select_db("base", $conexion) or die("Problemas en la seleccion de la base de datos"); 
if(isset($_POST['cedula'])){
$ced=mysqli_real_escape_string($_POST['cedula']);
$app=mysqli_real_escape_string($_POST['apellido']);
$nom=mysqli_real_escape_string($_POST['nombre']);
mysql_query ("INSERT INTO datos (cedula, apellido, nombre) VALUES ($ced,$app,$nom)", $conexion) or die("Problemas en el select" . mysql_error()); 
mysql_close($conexion); 
echo "Registro exitoso"; }
?>

Ya que creas las variables para utilizarlas en la query, no las asignes directamente, pásales el real escape string por si acaso.

Edito: y las comillas de la query, que no las has corregido, y seguían siendo 2 comillas simples seguidas.
Aunque en realidad no sé qué resultado pueden dar esas comillas simples seguidas.

Dalam · #10 (**permalink**) 05/01/2015, 19:58

No esta mas decir que se migren a una clase de MySQL para alargar la vida de los scripts

Código PHP:

Ver original<?php
$mysqli = new mysqli('host','username','password','database_name');
if ($mysqli->connect_error) {
    die('Error : ('. $mysqli->connect_errno .') '. $mysqli->connect_error);
}
if(isset($_POST['cedula'])){
    $ced = $mysqli->real_escape_string($_POST['cedula']);
    $app = $mysqli->real_escape_string($_POST['apellido']);
    $nom = $mysqli->real_escape_string($_POST['nombre']);
    if ($mysqli->query("INSERT INTO datos (cedula, apellido, nombre) VALUES ($ced,$app,$nom)")) {
        echo "Registro exitoso";
    }
}
$mysqli->close();