error de novato.. borrar un registro, ayudadme por favor

[asitudela]

Buenas

Me gustaría borrar un registro de mi base de datos, pero me da el siguiente error:

Parse error: parse error in /home/www/recoin.com/fventas/baja2.php on line 18

yo lo que hago es pasar desde un formulario la variable $email, para poder dar de baja a ese email en la bd para que no reciba más boletines..

Código PHP:

  <?php  
 
  //--------------Conectamos con la bd  
  require("conex_.php");  
    
$busqueda= @mysql_query("SELECT email,id FROM suscriptor");  
  if (!$busqueda)  
  {  
    echo ("<br>Error al seleccionar los elementos de la base de datos, Inténtalo más tarde");  
    exit;  
  }  
  while($row = mysql_fetch_array($busqueda))  
  {  
        if ($row["email"] == $email)  
      {  
        $borrar=$row["id"];  
        DELETE FROM suscriptor where id=$borrar; //linea 18  
      }  
  }  
?>   

También había intentado así pero me da error

Código PHP:

  <?php  
 
  //--------------Conectamos con la bd  
  require("conex_.php");  
 
  $busqueda= @mysql_query("SELECT email FROM suscriptor");  
  if (!$busqueda)  
  {  
    echo ("<br>Error al seleccionar los elementos de la base de datos, Inténtalo más tarde");  
    exit;  
  }  
  while($row = mysql_fetch_array($busqueda))  
  {  
        if ($row["email"] == $email)  
      {  
        DELETE FROM suscriptor where email=$email;  
      }  
  }  
?>   

[Appletalk]

DELETE FROM suscriptor where id=$borrar; //linea 18

No podes poner la consulta asi nomas
Tendrias que almacenarla en una variable
$hola = "DELETE FROM suscriptor where id=$borrar";

Y despues hacer

mysql_query($hola);

[asitudela]

Código PHP:

  <?php 
 
  //--------------Conectamos con la bd 
  require("conex_.php"); 
 
  $busqueda= @mysql_query("SELECT email,id FROM suscriptor"); 
  if (!$busqueda) 
  {  
    echo ("<br>Error al seleccionar los elementos de la base de datos, Inténtalo más tarde"); 
    exit; 
  } 
  while($row = mysql_fetch_array($busqueda)) 
  { 
        if ($row["email"] == $email_borrar) 
      { 
        $borrar=$row["id"]; 
                          $comando = "DELETE FROM suscriptor where id=$borrar"; 
        mysql_query($comando); 
      } 
  } 
?>   

[asitudela]

Gracias Appletalk por tu ayuda, ya he encontrado el problema que tenía, y que también tenías en tu código, era el puto ;

Por lo tanto quedaría así:

$hola = "DELETE FROM suscriptor where id=$borrar;";

[sir_notos]

te recomiendo que uses comillas simples cuando no uses variables en tus strings

PE.

Código PHP:

  echo 'Hola este es un string sin variables no tiene caso usar comillas dobles :-)'; 
echo "Hola este es un string con variables y hoy es $var en estamos en $var2"; 
 
$comando['sql'] = 'SELECT * FROM table'; // este esta bien 
$comando[sql] = 'SELECT * FROM table'; // este no 
$comando["sql"] = 'SELECT * FROM table'; // este si pero es mejor el primer :) 
 
 
 
// cuando usas comillas simples php no busca variables los strings :-) 
// es mejos usar ' si no tienes variables  
   

Saludos

[asitudela]

Gracias por el consejo sir_notos, que como no llevo más q dos meses dándole un poco al php no sabía eso.

Salu2

[xavivars]

Yo lo haria así, es lo mismo, pero un poco mas rapido

Código PHP:

  <?php 
 
  //--------------Conectamos con la bd 
  require("conex_.php"); 
 
$correcto= @mysql_query("DELETE FROM suscriptor where email=$email_borrar;"); 
 
if($correcto) 
     echo 'Se han borrado '.mysql_affected_rows().' lineas de la BD'; 
else 
     echo 'No existe ese email en la base de datos'; 
?>   

Espero que te sirva ;)