Problema ó Duda al insertar un registro en una BD MySQL (no hace nada)

[yak_vi]

Buenas gente, estoy probando unas cosas en PHP y BD y resulta que estoy tratando de guardar en una base de datos MySQL dos registros, usuario y contraseña en texto plano (posteriormente veré como codificarlos) pero resulta que creo que estoy haciendo todo bien pero resulta que no guarda el registro en la base de datos pero tampoco da error. Ya he probado localmente con XAMPP en Windows y también en un hosting con linux y tampoco hace nada. Les pongo el código para que me digan si saben donde puede estar el error

Código PHP:

<?php
 
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);
 
$usuario = $_REQUEST['my_login'];
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";
echo $sql;
mysql_query($sql);
 
?>
 

P.D.: Este código ya lo tengo incrustado en otro script. Las variables de "my_pass" y "my_login" ya son definidas no están vacías.

Puse el echo para ver si me mostraba lo que me tomaba y en teoría está bien asi que no sé que puede ser lo que hago mal.

Desde ya muchas gracias!!!

Saludos!!

[Miguelithox]

Iniciado por yak_vi

Buenas gente, estoy probando unas cosas en PHP y BD y resulta que estoy tratando de guardar en una base de datos MySQL dos registros, usuario y contraseña en texto plano (posteriormente veré como codificarlos) pero resulta que creo que estoy haciendo todo bien pero resulta que no guarda el registro en la base de datos pero tampoco da error. Ya he probado localmente con XAMPP en Windows y también en un hosting con linux y tampoco hace nada. Les pongo el código para que me digan si saben donde puede estar el error

Código PHP:

<?php
 
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);
 
$usuario = $_REQUEST['my_login'];
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";
echo $sql;
mysql_query($sql);
 
?>
 

P.D.: Este código ya lo tengo incrustado en otro script. Las variables de "my_pass" y "my_login" ya son definidas no están vacías.

Puse el echo para ver si me mostraba lo que me tomaba y en teoría está bien asi que no sé que puede ser lo que hago mal.

Desde ya muchas gracias!!!

Saludos!!

Tu código anterior (malo):

Código PHP:

  <?php
 
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);
 
$usuario = $_REQUEST['my_login'];
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . "[B];[/B])";
echo $sql;
mysql_query($sql);
 
?>   

El código correcto:

Código PHP:

  <?php
 
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);
 
$usuario = $_REQUEST['my_login'];
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ")";
echo $sql;
mysql_query($sql);
 
?>   

Si no te diste cuenta, colocaste un ; punto y coma en la consulta MySQL. ;)

[yak_vi]

Iniciado por Miguelithox

Tu código anterior (malo):



El código correcto:

Código PHP:

  <?php
 
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);
 
$usuario = $_REQUEST['my_login'];
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ")";
echo $sql;
mysql_query($sql);
 
?>   

Si no te diste cuenta, colocaste un ; punto y coma en la consulta MySQL. ;)

Muchas gracias, ya voy a probar, No, no me había dado cuenta de ese ";"!

Ya cuento que pasó :)

Saludos!!

[mariomon17]

Obviemos a ciertos tipos, nos ira mejor.
....
Lo primero que veo, es lo siguiente

Código PHP:

mysql_select_db("dbpruebas",$conexion)
 

Para seleccionar una DB se utilizan comillas simples.

Lo siguiente:

Código PHP:

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";
 

Values, te faltan comillas simples, seria asi:

Código PHP:

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES ('" . $usuario . "','" . $passwrd . "';)";
 

Por ultimo

Código PHP:

mysql_query($sql)
 

En la ejecucion de la consulta, hay que poner el parametro de conexion:

Código PHP:

mysql_query($sql,$conexion)
 

Un saludo!

[Miguelithox]

Iniciado por yak_vi

Muchas gracias, ya voy a probar, No, no me había dado cuenta de ese ";"!

Ya cuento que pasó :)

Saludos!!

Si te sirvió, dejame un karma (Y)

[Miguelithox]

Iniciado por mariomon17

Obviemos a ciertos tipos, nos ira mejor.
....
Lo primero que veo, es lo siguiente

Código PHP:

mysql_select_db("dbpruebas",$conexion)
 

Para seleccionar una DB se utilizan comillas simples.

Lo siguiente:

Código PHP:

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";
 

Values, te faltan comillas simples, seria asi:

Código PHP:

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES ('" . $usuario . "','" . $passwrd . "';)";
 

Por ultimo

Código PHP:

mysql_query($sql)
 

En la ejecucion de la consulta, hay que poner el parametro de conexion:

Código PHP:

mysql_query($sql,$conexion)
 

Un saludo!

También lo de las comillas simples cuando ingresas, se me había escapado. Como dijo mariomon17.

[yak_vi]

Solucionado, lo dejé asi (por si a alguien le sirve):

Código PHP:

<?php
// Datos de la BD 
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia"); 
mysql_select_db("dbpruebas",$conexion);
 
mysql_query("INSERT INTO accounts (usuario,contrasenia) values ('" . $_REQUEST['my_login'] . "','". $_REQUEST['my_pass'] . "')") or die("error en la conexion al servidor SV1");
?>
 

Y funcionó bien :)

Saludos y muchas gracias desde ya!

[memoadian]

Iniciado por cocacola1

Tu codigo es una porqueria !!

¿Por que no te vas de aqui?

a ver Yak_vi
unas sugernecias

si recibes los valores por un formulario, quizas sea mejor usar $_POST
y no $_REQUEST, de este modo tendrás mayor control de lo que pasa, bueno solo es una sugerencia. luego analicemos tu código

Código PHP:

<?php
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);//aqui solo debes poner la tabla
 
$usuario = $_REQUEST['my_login'];//como dije mejor POST si es por un form
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";//en este caso usa la sintaxis de punto Ej 'BD.tabla'
echo $sql;//aqui te falta algo asi $res = mysql_query($sql,$con);
mysql_query($sql);
?>
 

ahora veamos un ejemplo funcional

Código PHP:

<?php
$con = mysql_connect("localhost","user","contraseña");
$bd = mysql_select_db("tabla") or die ("Verifique la Base de Datos");
//conectamos y con die verificamos errores de conexion
 
$usuario = $_POST['my_login'];//como dije mejor POST si es por un form
$passwrd = $_POST['my_pass'];
 
$sql = "INSERT INTO nom_bd.accounts (usuario, contrasenia) VALUES (".$usuario.", ".$passwrd.");";
//nom_bd es el nombre de tu base de datos
//no dejes espacios al concatenar
//el punto y coma va hasta al final de la cadena no antes del parentesis
$res = mysql_query($sql,$con); //realizamos la insercion
//aunque deberia funcionar sin el $res XD
mysql_query($sql,$con);
// LISTO???
?>
 

[yak_vi]

Iniciado por memoadian

¿Por que no te vas de aqui?

a ver Yak_vi
unas sugernecias

si recibes los valores por un formulario, quizas sea mejor usar $_POST
y no $_REQUEST, de este modo tendrás mayor control de lo que pasa, bueno solo es una sugerencia. luego analicemos tu código

Código PHP:

<?php
// Datos de la BD
$conexion = mysql_connect("localhost" , "usuario" , "contrasenia");
mysql_select_db("dbpruebas",$conexion);//aqui solo debes poner la tabla
 
$usuario = $_REQUEST['my_login'];//como dije mejor POST si es por un form
$passwrd = $_REQUEST['my_pass'];
 
$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";//en este caso usa la sintaxis de punto Ej 'BD.tabla'
echo $sql;//aqui te falta algo asi $res = mysql_query($sql,$con);
mysql_query($sql);
?>
 
ahora veamos un ejemplo funcional
 
<?php
$con = mysql_connect("localhost","user","contraseña");
$bd = mysql_select_db("tabla") or die ("Verifique la Base de Datos");
//conectamos y con die verificamos errores de conexion
 
$usuario = $_POST['my_login'];//como dije mejor POST si es por un form
$passwrd = $_POST['my_pass'];
 
$sql = "INSERT INTO nom_bd.accounts (usuario, contrasenia) VALUES (".$usuario.", ".$passwrd.");";
//nom_bd es el nombre de tu base de datos
//no dejes espacios al concatenar
//el punto y coma va hasta al final de la cadena no antes del parentesis
$res = mysql_query($sql,$con); //realizamos la insercion
//aunque deberia funcionar sin el $res XD
mysql_query($sql,$con);
// LISTO???
?>
 

Ya funcionó perfecto! Lo dejé como puse mas arriba y andubo de diez! Ahora, con respecto al $_REQUEST es porque en el script ya está implementado así y cumple bien su función por eso lo dejé asi y anda de diez!

Saludos!!

[memoadian]

DIOS hubo como 500 respuestas antes de que pudiera postear XD

[Miguelithox]

Iniciado por memoadian

DIOS hubo como 500 respuestas antes de que pudiera postear XD

Jajaja, tienes que ser más rápido con los dedos en el teclado. xD

[memoadian]

Iniciado por Miguelithox

Jajaja, tienes que ser más rápido con los dedos en el teclado. xD

Nahh fui por unos nachos al refri XD

[egepe]

Para mi si no me perdí leyendo la respuesta de toda la gente es que tu insert tiene un "pto y coma " (;) que está mal ubicado

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";

Bueno de verdad q no sé si eso era lo que querías....

[yak_vi]

Iniciado por egepe

Para mi si no me perdí leyendo la respuesta de toda la gente es que tu insert tiene un "pto y coma " (;) que está mal ubicado

$sql = "INSERT INTO accounts (usuario, contrasenia) VALUES (" . $usuario . "," . $passwrd . ";)";

Bueno de verdad q no sé si eso era lo que querías....

Ya lo solucioné gracias por responder también!