Paso de parámetros en PHP para borrar registros en MYSQL

leve91 · #1 (**permalink**) 12/07/2011, 19:46

Tengo problemas al borrar un solo registro de mi tabla de mysql. Estoy en un scrip de php que hace consulta a la DB y ordena los registros en filas cada una de color diferente, hasta allí todo bien pero al darle clic en el botón BORRAR que está a la derecha el archivo de borrar.php simplemente no hace nada. Tiene algo que ver con el enlace siguiente: echo ' <a href="borrar.php?type=movie & id = ' .$row['nombre'].'"> [BORRAR] </a> '; y que al pasar los parámetros al archivo de borrar simplemente no reconoce la parte el “id” que se refiere al nombre escrito en la tabla de mysql; lo había hecho funcionar de otra forma pero me borraba TODOS los registros de la tabla.
Dejo los dos archivos para que me digan en que estoy mal.
Hasta pronto.

Archivo de consultas:

Código PHP:

Ver original<?php
$db = mysql_connect('host', 'user', 'pass') or
die ('Unable to connect. Check your connection parameters.');
mysql_select_db('miDB', $db) or die(mysql_error($db));
?>
<html>
<head>
<title> Consulta de los depositos </title>
<style type="text/css">
th { background-color: #999;}
.odd_row { background-color: #EEE; }
.even_row { background-color: #FFF; }
</style>
</head>
<body>
<table style="width:100%;">
<tr>
<th colspan="2">Tabla 1</th>
</tr>
<tr>
<th colspan="8" >Encabezados</a> </th>
</tr>
<?php
$query = 'SELECT * FROM depositos';
$result = mysql_query($query, $db) or die (mysql_error($db));
$odd = true;
while ($row = mysql_fetch_assoc($result)) {
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Nombre: </b>';
echo $row['nombre'];
echo ' </td> <td> ';
echo ' <a href="borrar.php?type=movie & id='.$row['nombre'].'">
[EDITAR] </a> ';
echo ' <a href="borrar.php?type=movie & id = ' .$row['nombre'].'">
[BORRAR] </a> ';
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Apellido: </b>';
echo $row['apellido'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Dirección: </b>';
echo $row['direccion'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Estado: </b>';
echo $row['estado'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Municipio: </b>';
echo $row['municipio'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Ciudad: </b>';
echo $row['ciudad'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Deposito: </b>';
echo $row['deposito'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Cantidad: </b>';
echo $row['cantidad'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Día: </b>';
echo $row['dia'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Mes: </b>';
echo $row['mes'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Año: </b>';
echo $row['año'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Banco: </b>';
echo $row['banco'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';
$odd = !$odd;
echo ' <td style="width:75%;"> ';
echo '<b>Observaciones: </b>';
echo $row['observaciones'];
echo ' </td> <td> ';
 
echo ' </td> </tr> ';
echo'<th colspan="8" height="9"></th>';
}
?>
</table>
</body>
</html>

Archivo para borrar

Código PHP:

Ver original<?php
$db = mysql_connect('host', 'usuario', 'pass') or
die ('No se puede conectar. Revisar parámetros de conexión.');
mysql_select_db('midatabase', $db) or die(mysql_error($db));
 
if (!isset($_GET['do']) || $_GET['do'] != 1) {
($_GET['type']) {
'movie'
echo '¿Esta seguro de borrar este registro?<br/>';
}
echo '<a href="' . $_SERVER['REQUEST_URI'] . '&do=1">SI</a> ';
echo 'O <a href="consulta.php">NO</a>';
} else {
?>
<?php
$query = 'DELETE FROM registros
WHERE
nombre = ' .$_GET['id'];
$result = mysql_query($query, $db) or die(mysql_error($db));
?>
<p style="text-align: center;"> Su registro ha sido borrado.
<a href=" index.php"> Return to Index </a> </p>
<?php
}
?>

Uncontroled_Duck · #2 (**permalink**) 13/07/2011, 01:32

Hay que prestar especial atención a acotar entre comillas simples (') los valores literales (cadenas de caracteres). Te recomiendo que las consultas SQl las hagas siempre entre comillas dobles para evitar estos errores.

Así lo tienes:

Código PHP:

  $query = 'DELETE FROM registros 
WHERE 
nombre = ' .$_GET['id'];

Debería ser así:

Código PHP:

  $query = "DELETE FROM registros 
WHERE 
nombre = '" .$_GET['id']. "'";

Saludos,

leve91 · #3 (**permalink**) 13/07/2011, 13:10

Muchas gracias amigo, tu consejo me ha funcionado a la perfección, solo una última pregunta, mi programa esta diseñado para que me pregunte si deseo borrar un registro SI Ó NO, si le digo que sí lo borra y si le digo que no me regresa a consultas pero el registro lo borra de todas formas, no se que será lo que me falta.
Y nuevamente gracias por el aporte.

Uncontroled_Duck · #4 (**permalink**) 13/07/2011, 13:34

Cita:

Iniciado por leve91

Muchas gracias amigo, tu consejo me ha funcionado a la perfección, solo una última pregunta, mi programa esta diseñado para que me pregunte si deseo borrar un registro SI Ó NO, si le digo que sí lo borra y si le digo que no me regresa a consultas pero el registro lo borra de todas formas, no se que será lo que me falta.
Y nuevamente gracias por el aporte.

Imagino que será por el orden en que se ejecuta. Para evitarlo, más o menos sería en este orden.

Seudocódigo:

Código PHP:

  1 - Pulsas para borrar el registro 
2 - Preguntas si quieres borrarlo 
3 - Si la respuesta es si, envías el id del registro que deseas borrar 
if (compruebas la variable) { 
    Validas el id recibido 
    Ejecutas la query con respecto al id 
    Redireccionas 
}  
4 - Si la respuesta es no 
    El botón NO redirecciona

Saludos,

PIRRUMAN · #5 (**permalink**) 13/07/2011, 13:53

en etas partes no estoy seguro pero creo estan mal

Cita:

if (!isset($_GET['do']) || $_GET['do'] != 1) { //inicio del if

($_GET['type']) {//inicio de que???

'movie' ///se imprime??

echo '¿Esta seguro de borrar este registro?<br/>';

} ///fin de que??

echo '<a href="' . $_SERVER['REQUEST_URI'] . '&do=1">SI</a> '; ///aqui creo se pone ?id en vez de &id al menos que te de mas variables el ($_SERVER['REQUEST_URI'])ejemplo mi pagina.php?uno=1

echo 'O <a href="consulta.php">NO</a>';

} ///fin del if

else

podrias hacer purebas imprimiendo antes de ejecutar el delete el valor de id y hacer un exit: o comentar la linea donde esta $resulte imprimir $query para ver que te envia

ale_dla · #6 (**permalink**) 14/07/2011, 07:36

Tenes que borrarlo por el id

Código PHP:

  
<a href='borrar.php?id='.$row['id']."/>Borrar</a>

Y en borrar

Código PHP:

  $id = $_GET['id'];
$sql = "SELECT * FROM tutabla  WHERE id='$id' Limit 1";
$query = mysql_query($sql) or die(mysql_error);

Despues haces la query de borrar
saludos

xbquelle · #7 (**permalink**) 02/06/2015, 09:45

Gracias me has dado una pista y me ha servido de ayuda, al final lo he puesto así:

<div style="text-align:center;">
<table class="tabla_normal" align="center">
<tr style="background-color:#b08863; color:#FFF;">
<th>Borrar</th>
<th>Nombre máquina</th>
<th>Nº Serie</th>
<th>Fabricante</th>
</tr>

<?php
//Mostramos los registros
while ($row = mysql_fetch_array($resul))
{
?>
<form id="form_borra_maquinaria" action="borra_maquinaria_nuevo.php" method="post">
<tr><td><input type='hidden' name='id' value='<?php echo $row['numero_maquina']?>' />
<input class='buttonFormSmall' id='submit' type='submit' name='enviar' value='Borrar' title='Borrar'>
</td></form>
<?php
echo" <td align='center'>".$row['nombre_maquina']."</td>
<td align='center'>".$row['num_serie']."</td>
<td align='center'>".$row['fabricante']."</td>
</tr>";

}
mysql_free_result($resul);
?>
</table>

He puesto el <form> justo antes del botón de borrar el dato y el </form> justo después del botón. Y funciona perfecto. Muchas gracias