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; 
      } 
  } 
?>   

[peloteitor]

No recuerdo muy bien el php pero a ver si esta pregunta t sirve de ayuda:
¿Por qué haces la primera consulta de la forma: @mysql_query("SELECT...."); y la segunda la metes a fuego en el código php?

[asitudela]

"¿Por qué haces la primera consulta de la forma: @mysql_query("SELECT...."); "

Lo hago para que recoja todos los emails de bd

"y la segunda la metes a fuego en el código php?"

nose que me quieres decir con esto.


A mi lo que me importa realmente es saber si el comando para borrar de la bd está bien:

DELETE FROM suscriptor where email=$email;

De todas formas gracias por contestar

[peloteitor]

El error que te da significa que el compilador de php no entiende esa línea. Eso es porque las palabras DELETE, FROM, etc ni son vbles ni son palabras reservadas de php.
Lo que quise decir antes es que no tendrias que hacer algo así como:
$borrar = @mysql_query("DELETE...");

[asitudela]

Al parecer no había que ponerlo directamente, ya no me da error pero tampoco me borra el registro

Código PHP:

  <?php 
  
  //--------------Conectamos con la bd 
  require("conex_.php"); 
  //--------------Comprobamos  que el em@il no se ha introducido antes 
  $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); 
      } 
  } 
?>   

[eqsoftware]

Estimado asitudela:
Tu error es el siguiente, cuando pones DELETE FROM suscriptor where email=$email; Php intenta interpretar esa linea, al encontrar una palabra que no conoce (como DELETE), tira un error.
Para poder eliminar un registro utlizando php, tenes que usar la api correspondiente al motor de base de datos que estas usando. Por lo visto estas utilizando MySql, por lo tanto con la siguiente linea te bastara para lograr tu objetivo.
$query = "DELETE FROM suscriptor WHERE email = '$email' ";
mysql_query($query);

Nota: Puse la consulta dentro de una variable, porque de esta forma es mas facil hacer un debug de la misma.

[asitudela]

Puff me sigue sin funcionar

Código PHP:

  <?php 
  //--------------Conectamos con la bd 
  require("conex_.php"); 
  //--------------Comprobamos  que el em@il no se ha introducido antes 
  $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_borrar) 
      { 
        $query = "DELETE FROM suscriptor WHERE email = '$email_borrar' ";  
        mysql_query($query); 
      } 
  } 
?>   

[peloteitor]

Intenta esto:
$query = "DELETE FROM suscriptor WHERE email = ".$email;
mysql_query($query);

[asitudela]

GRACIAS eqsoftware y peloteitor

Ya lo he solucionado, lo he hecho como me dijistes, aunque no tenías en cuenta una cosa muy importante (un descuido lo tiene cualquiera) que era el ; que hacía que no se ejecutara el query

Entonces quedaría así el asunto:

$query = "DELETE FROM suscriptor WHERE email = '$email_borrar'; ";
mysql_query($query);

Muchas gracias de nuevo