¿Que estoy haciendo mal?... (Sistema newsletter, problema al conectar a la DB)

Iniciado por Cergath, 23 Septiembre 2009, 18:35 PM

0 Miembros y 1 Visitante están viendo este tema.

Cergath

Pues miren:

Empiezo por crear la DB en phpmyadmin normal, le puse "Suscritos" y ejecuté la consulta:

CitarCREATE TABLE `tabla_boletin` (
`id` bigint(20) NOT NULL auto_increment,
`email` varchar(100) NOT NULL default '',
`fecha` datetime NOT NULL default '0000-00-00 00:00:00',
PRIMARY KEY (`id`),
UNIQUE KEY `email` (`email`)
) TYPE=MyISAM AUTO_INCREMENT=1;

Todo bien, ahora la página para suscribir, que es un cuadro tonto que pide el email y en seguida te pone en la DB, o bien te elimina poniendo el correo.

Código (php) [Seleccionar]
<?php 
if($seguro_del) { 
include(
"conexion.php");
$result = @mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` LIKE ' $email '"); 
if(
mysql_num_rows($result) != 0) { 
$sql = @mysql_query("DELETE FROM `tabla_boletin` WHERE `email`=' $email '"); 
$msg " El correo electr?nico se ha removido, no recibir? nuestras novedades. "
} else { 
$msg " El email que solicitaste para darlo de baja no existe en la base de datos. "

} } elseif(
$seguro_ins) { 
include(
"conexion.php"); 
$sql "SELECT email FROM `tabla_boletin` WHERE `email` LIKE ' $email '"
$result mysql_query($sql); 
if(
mysql_num_rows($result) != 0) { $msg " El email que indicaste para la suscripci?n ya se encuentra registrado en la base de datos. "; } else { 
if (!
ereg("^[^@]+@([a-zA-Z0-9\-]+\.)+[a-zA-Z0-9]{2,}$"$email)) { $msg =" Tu correo electr?nico no existe o esta mal escrito. "; } else { 
$fecha date("Y/n/d H:i:s"); 
$sql "INSERT INTO `tabla_boletin` SET `email`=' $email ', `fecha`=' $fecha '"
$result mysql_query($sql); 
$msg " La suscripci?n fue un ?xito, recibir?s las novedades de nuestro sitio. "
} } } 
?>

<html>
<head>
<title>Suscribir al bolet?n</title>
</head>
<body>
<div align="center">
<?php if($msg) { ?>
<table width="100%" border="0" cellspacing="5" cellpadding="0">
<tr>
<td><div align="center" class="texto_gral"><?=$msg;?></div></td>
</tr>
<tr>
<td><div align="center"><a href="javascript:window.close()"><strong>cerrar ventana</strong></a></div></td>
</tr>
</table>
<? } else { ?>
<form action="suscripcion.php" method="post" name="" id="">
<table width="200" border="0" cellspacing="2" cellpadding="0">
<tr>
<td><input name="email" type="text" id="email" value="tu correo electr&oacute;nico" size="25">
</td>
</tr>
<tr>
<td> <div align="right">
<input name="seguro_ins" type="submit" id="seguro_ins" value="suscr?bete">
<input name="seguro_del" type="submit" id="seguro_del" value="desinscribir">
</div></td>
</tr>
</table>
</form>
<? } ?>
</div>
</body>
</html>


¿Va bien verdad?,
ahora fíjense en la parte de Include (al principio), pide el archivo Conexion.php que lo tengo así:

Código (php) [Seleccionar]
<?
$link=mysql_connect("localhost", "root", "lalolanda");
mysql_select_db("suscritos",$link) OR DIE ("Error: Imposible Conectar");
?>


¿Esta bien no?... cuando corro el script no me sale error, pero a la hora de poner el email y dar en Suscribir, puf, no pasa nada en la DB, no queda el registro....






Adjunto el fichero para enviar el email en html:
Código (php) [Seleccionar]
<html>
<head>
<title>Sistema de bolet?n :: Env?o de emails</title>
</head>
<body>
<?
if($seguro_env) {
include("conexion.php");
$sql = mysql_query("SELECT * FROM `tabla_boletin`");
$n = 1;
while($row = mysql_fetch_array($sql)) {
$asunto ="Bolet?n NO.1";
$headers = "MIME-Version: 1.0\r \n";
$headers .= "Content-type: text/html; charset=iso-8859-1\r \n";
$headers .= "From: \"Bolet?n de Dise-web\" <boletin@dise-web.com>\r \n";
if(!$mensaje) { echo "No haz escrito el bolet?n!"; } else {
mail("".$row["email"]."","$asunto","$mensaje\n\n","$headers");
echo "<div>".$n++.". Enviado email a <b>".$row["email"]."</b><br></div>";
}
}
} else {
?>
<form action="enviarboletin.php" method="post" name="" id="">
<table width="100%" border="0" cellspacing="1" cellpadding="0">
<tr>
<td><font size="4" face="tahoma, Arial, verdana"><strong>Puedes usar HTML
para el bolet&iacute;n.</strong></font></td>
</tr>
<tr>
<td><textarea name="mensaje" cols="50" rows="5" id="mensaje"></textarea></td>
</tr>
<tr>
<td><input name="seguro_env" type="submit" id="seguro_env" value="Enviar"></td>
</tr>
</table>
</form>
<? } ?>
</body>
</html>


Muchas gracias, espero ayuda...

WHK

El problema lo tienes en la sentencia LIKE, fijate que no es lo mismo poner
Código (php) [Seleccionar]
$result = @mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` LIKE ' $email '");

a poner
Código (php) [Seleccionar]
$result = @mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` = '$email'");

like da una aproximación o sea algo que contenga eso pero puede retornar muchos resultados, si quieres algo exacto usa el signo igual.

Otro detalle es que si usas like de seguro no tienes ningún email que comienze con un espacio en blanco ni termmine con uno también asi que o le sacas esos espacios en blanco en like o le pones el signo porcentage %palabra% ya que porcentage es comodín y significaría que pueda comenzar por lo que sea y termine en lo que sea pero que contenga palabra, si le pones espacio en blanco + palabra + espacio en blanco estarás buscando todos los mails que comienzen y terminen en un espacio en blanco y que además puedan contener la palabra y por eso no te devuelve resultados.

Fijate en tu phpmyadmin y fabrica la query así:
$sql = 'select * from ....'
después vas y haces un echo $sql; tomas lo que te muestre y lo pegas en tu phpmyadmin y verás el problema.

También puedes sacarle las arrobas para que te muestre si hay algún error en la función porque así vas a ciegas.

Otra cosa, utiliza http://cl2.php.net/mysql_real_escape_string para evitar las inyecciones sql
Código (php) [Seleccionar]
if(!$result = mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` = '".mysql_real_escape_string($email)."'")
die('Error en la query.');


Ahora, si quieres saber la cantidad de mails que concuerden puedes usar count()

Código (php) [Seleccionar]
if(!$result = mysql_query("SELECT count(id) FROM `tabla_boletin` WHERE `email` = '".mysql_real_escape_string($email)."'")
die('Error en la query.');


Te devolverá una columna llamada 'count(id)' con la cantidad de mails que concuerden con el criterio.

Cergath

Gracias por los consejos!, ya los implementé.

Sin embargo el script nada que me pone los registros en la base de datos, colocas el email, clic en suscribir y nada... :-\

miguel86

Vete paso por paso, comprobando cada consulta si da resultado, fijate en el fondo de tu página ya que el mensaje en el die si tienes un fondo oscuro no lo verás, coge tu php copia la consulta y la pegas en la parte sql del Mysql (Phpmyadmin) para comprobar que no tiene errores de sintaxis y si funcionan dilo y te miramos la semántica (pero antes mira la sintaxis ya que eso corre de tu cuenta no quieras que te miramos todas las sentencias y te las corrijamos).
Cuando lo hayas hecho y hayas comprobado de arriba a abajo cada sentencia habrás dado con la sentencia que te da el problema, colócala y te la corregimos. O al menos pon la salida que te da de colocar las sentencias de la primera a la última en el orden que van al probarlas en el phpmyadmin.
Salu2

^Tifa^

No puedo ayudarte en abundancia, ya que el error puede ser variado... haz considerado revisar los logs que genera Apache respecto a esto? generalmente cuando tengo un fallo en algo de php o un cgi de perl o viceversa y no se donde esta el error, voy a los logs de error de Apache y alli me instruyo mas sobre que esta pasando  ;)  mira ver si te funciona.

Filibustero de bolsillo

Cergath, es simple:

No hace el registro porque la acción del formulario es redirigirse a si mismo más no a las variables, no es lo mismo:
<form action="suscripcion.php" method="post" name="" id="">

Que:
<form action="<?php $_SERVER['PHP_SELF']?>" method="post" name="" id="">

Saludos!

Edit PD: de igual manera haz lo que te dijo WHK, más ahora si te hará el registro en la DB.

Absence makes the heart grow fonder.

Eru Iluvatar

Cita de: WHK en 23 Septiembre 2009, 20:45 PM
El problema lo tienes en la sentencia LIKE, fijate que no es lo mismo poner
Código (php) [Seleccionar]
$result = @mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` LIKE ' $email '");

a poner
Código (php) [Seleccionar]
$result = @mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` = '$email'");

like da una aproximación o sea algo que contenga eso pero puede retornar muchos resultados, si quieres algo exacto usa el signo igual.

Otro detalle es que si usas like de seguro no tienes ningún email que comienze con un espacio en blanco ni termmine con uno también asi que o le sacas esos espacios en blanco en like o le pones el signo porcentage %palabra% ya que porcentage es comodín y significaría que pueda comenzar por lo que sea y termine en lo que sea pero que contenga palabra, si le pones espacio en blanco + palabra + espacio en blanco estarás buscando todos los mails que comienzen y terminen en un espacio en blanco y que además puedan contener la palabra y por eso no te devuelve resultados.

Fijate en tu phpmyadmin y fabrica la query así:
$sql = 'select * from ....'
después vas y haces un echo $sql; tomas lo que te muestre y lo pegas en tu phpmyadmin y verás el problema.

También puedes sacarle las arrobas para que te muestre si hay algún error en la función porque así vas a ciegas.

Otra cosa, utiliza http://cl2.php.net/mysql_real_escape_string para evitar las inyecciones sql
Código (php) [Seleccionar]
if(!$result = mysql_query("SELECT email FROM `tabla_boletin` WHERE `email` = '".mysql_real_escape_string($email)."'")
die('Error en la query.');


Ahora, si quieres saber la cantidad de mails que concuerden puedes usar count()

Código (php) [Seleccionar]
if(!$result = mysql_query("SELECT count(id) FROM `tabla_boletin` WHERE `email` = '".mysql_real_escape_string($email)."'")
die('Error en la query.');


Te devolverá una columna llamada 'count(id)' con la cantidad de mails que concuerden con el criterio.

cergath, necesito un sistema como estos, espero que no te moleste si lo uso!...
oye,
y esto donde lo coloco?
Código (php) [Seleccionar]
if(!$result = mysql_query("SELECT count(id) FROM `tabla_boletin` WHERE `email` = '".mysql_real_escape_string($email)."'")
die('Error en la query.');