PHP

Correction

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Correction

snarky

Code :

...
else // Il s'agit d'une désinscription.
{
include('sql.php');
$search_mail_exist = mysql_query("SELECT newsletter_id, newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail' AND newsletter_send='1'" ) or die("erreur lors de la vérification" );
$mail_exist = mysql_fetch_row($search_mail_exist);
if($mail_exist[0] != 0) //on regarde si le mail existe.
{
// Si oui :
echo $mail_exist[0], ' le mail existe et est activé.<br />';
$search_mail_exist = mysql_query("SELECT newsletter_id, newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail' AND newsletter_send='1'" ) or die("erreur lors de la vérification" );
while($newsletter_id = mysql_fetch_array($search_mail_exist))
{
$unregister = "REPLACE newsletter(`newsletter_id`, `newsletter_mail`, `newsletter_send`) VALUES(".$newsletter_id['newsletter_id'].", '$newsletter_mail', '0')";
mysql_query($unregister) or die("echec lors de la désinscription à la newsletter" );
//header("location: http://sn4rky.free.fr/index.php" );
}
}
else
{
// Si non :
echo $newsletter_mail, ' n\'est pas utilisé';
}
}

lorsqu'un mail existe, je lui demande de me l'indiquer puis d'effectue une action.
il m'indique parfaitement ce que je lui demande, mais bloque au niveau du while.
pourquoi n'arrive t'il pas a m'afficher la suite ?

si je rajoute

Code :

$search_mail_exist = mysql_query("SELECT newsletter_id, newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail' AND newsletter_send='1'" ) or die("erreur lors de la vérification" );

au dessus du while ça marche parfaitement
serait un probleme de portée de ma variable ?

Message édité par snarky le 07-07-2005 à 02:16:08

Publicité

The-Shadow

Développeur
T'as été voir dans ton profil?

Ce serait pas plutot ?
$newsletter_id == mysql_fetch_array($search_mail_exist)

Message édité par The-Shadow le 06-07-2005 à 18:18:44

snarky

The-Shadow a écrit :

Ce serait pas plutot ?
$newsletter_id == mysql_fetch_array($search_mail_exist)

en tps normal avec un = ça marche tres bien

The-Shadow

Développeur
T'as été voir dans ton profil?

Je n'affirmais rien, j'utilise jamais while, trop évasif à mon gout, je préfère un foreach, mais je te faisais une suggestion qui me semblait plus logique, ça se trouve, je t'ai dit n'importe quoi, enfin, tu peux essayer quand même, tu ne risques pas grand chose

esox_ch

Oui desolé Shadow mais tu as faux . Le but de la syntaxe avec = est que le tableau retournera false une fois arrivé a la fin = >

while($var = FALSE) => Arret

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

dwogsi

Défaillance cérébrale...

Non non c'est bien ca :
$newsletter_id = mysql_fetch_array($search_mail_exist)

Parce qu'ici il s'agit de répéter les instruction tant que l'attirubtion d'une valeur à $newsletter_id par mysql_fetch_assoc() et non pas si les deux on le même valeur.

Edit:
Posté en même temp qu esox_ch... Pour dire la même chose...

Message édité par dwogsi le 06-07-2005 à 18:27:38

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

snarky

on donne des cours a shadow et moua dans tout ça d'ou viens mon probleme ? :'(

The-Shadow

Développeur
T'as été voir dans ton profil?

esox_ch a écrit :

Oui desolé Shadow mais tu as faux . Le but de la syntaxe avec = est que le tableau retournera false une fois arrivé a la fin = >

while($var = FALSE) => Arret

Bah je le saurais comme ça, pas de quoi être désolé, c'était autant une question qu'une réponse.
Perso, dans ton cas, je ferais un tout bête mysql_num_rows et une boucle for, ça te permet de choper direct là ou ça merde sans avoir à refaire ton code.

The-Shadow

Développeur
T'as été voir dans ton profil?

Euh, comme j'ai dis, j'utilise pas While, j'aime pas, mais c'est normal que la fermeture de la connexion se retrouve dans la boucle ?
Encore une fois, c'est autant une question qu'une réponse.

sielfried

snarky a écrit :

Tu fais déjà un mysql_fetch_row sur ton résultat, donc forcément. Je suppose que ton truc retourne qu'une seule ligne, que capte mysql_fetch_row, et ensuite mysql_fetch_array cherche les lignes d'après et n'en trouve pas.

---------------
StarCraft Professional Gaming Database | [Ze Topic] Starcraft/BroodWar

Publicité

snarky

sielfried a écrit :

en effet ptite erreur, j'avais carrement pas fait attention a ça.
je vais corriger sa merci :jap:

snarky

j'ai tjr le meme soucis par contre
si j'enleve

* $search_mail_exist = mysql_query("SELECT newsletter_id, newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail' AND newsletter_send='1'" ) or die("erreur lors de la vérification" );

du dessus du while, il n'arrive pas a m'affiche la valeur de id du mail ($newsletter_id)

esox_ch

The-Shadow a écrit :

Euh, comme j'ai dis, j'utilise pas While, j'aime pas, mais c'est normal que la fermeture de la connexion se retrouve dans la boucle ?
Encore une fois, c'est autant une question qu'une réponse.

non pas normal

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

sielfried

Tu fais des trucs un peu affreux aussi...

$newsletter_id = $newsletter_id['newsletter_id'];

[:mlc]

---------------
StarCraft Professional Gaming Database | [Ze Topic] Starcraft/BroodWar

snarky

esox_ch a écrit :

non pas normal

mauvais copier/coller :ange:

snarky

sielfried a écrit :

Tu fais des trucs un peu affreux aussi...

$newsletter_id = $newsletter_id['newsletter_id'];

[:mlc]

oui pour l'instant c'est un peu affreux :whistle:
mais sa devrait tout de meme marcher non ?

Message édité par snarky le 06-07-2005 à 19:11:46

esox_ch

aaaa non pas marcher

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

snarky

bon ben, en faite, j'avais utiliser mysql_fetch_row au mieux de mysql_num_rows

The-Shadow

Développeur
T'as été voir dans ton profil?

snarky a écrit :

mauvais copier/coller :ange:

Bah quand tu cites un code sur un forum, la plupart du temps, l'Elite HFR te trouve le point virgule en trop, alors si toi tu déplaces carrément des lignes entières pendant ton copier/coller, ça va pas être facile de t'aider.

snarky

ça demontre que j'ai pas de soucis avec les ";"
et puis mon soucis venait en fin de compte du resultat que j'attendais et non de la maniere dont je voulais obtenir ce resultat, d'où mon erreur d'emploi de la mauvaise fonction :whitsle:

et puis comme ça, ça m'aura permis de m'auto-corrigé tout en ameliorant mon code grâce aux réflexions des gens ^^

esox_ch

Oui mais si tu nous donnes un code juste on pourra corriger des choses qui sonte effectuvement fausses dasn ton code

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

snarky

esox_ch a écrit :

Oui mais si tu nous donnes un code juste on pourra corriger des choses qui sonte effectuvement fausses dasn ton code

pas fausses mais pas correctes :lol:

Code :

<?php
//récupération des variables :
include('sql.php');
$newsletter_mail = htmlentities(trim($_POST['newsletter_mail']),ENT_QUOTES,'UTF-8');
if($_POST['newsletter_abo'] == 1) // Il s'agit d'une inscription.
{
if(isset($newsletter_mail))
{
if(!eregi("^[-._a-z0-9]*@[a-z0-9]{3,}\.[a-z]{2,6}$",$newsletter_mail)) // vérification du mail :
{
// Email invalide :
echo'L\'adresse E-mail que vous avez inscrit n\'est pas valide : ', $newsletter_mail;
}
else
{
// Email valide :
$search_mail_exist = mysql_query("SELECT newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail'" );
$exist = mysql_num_rows($search_mail_exist);
// On verifie si le mail existe déja ou pas.
if($exist)
{
// Si oui :
$search_mail_send = mysql_query("SELECT newsletter_send FROM newsletter WHERE newsletter_mail='$newsletter_mail'" );
$send = mysql_fetch_array($search_mail_send);
if($send['newsletter_send'] == 1)
{
// Si oui : Le mail est déja utilisé.
echo $newsletter_mail ,' est déjà utilisé.';
}
else
{
//Si non : On procède à une réinscription.
$search_mail_send = mysql_query("SELECT newsletter_id FROM newsletter WHERE newsletter_mail='$newsletter_mail'" );
while($newsletter_id = mysql_fetch_array($search_mail_send))
{
$register = "REPLACE newsletter(`newsletter_id`, `newsletter_mail`, `newsletter_send`) VALUES(".$newsletter_id['newsletter_id'].", '$newsletter_mail', '1')";
mysql_query($register);
header("location: http://sn4rky.free.fr/index.php" );
}
}
}
else
{
// Si non : On procède à une inscription.
$register = "INSERT INTO newsletter(`newsletter_mail`) VALUES('$newsletter_mail')";
mysql_query($register);
header("location: http://sn4rky.free.fr/index.php" );
}
}
}
}
else // Il s'agit d'une désinscription.
{
$search_mail_exist = mysql_query("SELECT newsletter_id, newsletter_mail FROM newsletter WHERE newsletter_mail='$newsletter_mail' AND newsletter_send='1'" );
$exist = mysql_num_rows($search_mail_exist);
if($exist) //on regarde si le mail existe.
{
// Si oui :
while($newsletter_id = mysql_fetch_array($search_mail_exist))
{
$unregister = "REPLACE newsletter(`newsletter_id`, `newsletter_mail`, `newsletter_send`) VALUES(".$newsletter_id['newsletter_id'].", '$newsletter_mail', '0')";
mysql_query($unregister);
header("location: http://sn4rky.free.fr/index.php" );
}
}
else
{
// Si non :
echo $newsletter_mail, ' n\'est pas utilisé';
}
}
?>

voici mon code final
j'ai mis en place les htmlentities mais je suis pas sur de la manière donc il faut les mettres :whistle:

Message édité par snarky le 07-07-2005 à 02:28:49

Profil supprimé

-mysql_close() ne te sers à rien, php ferme la connexion automatiquement à la fin du script
-pour l'htmlentities, regarde la doc php du l'utilise mal et ne sert à rien pour une variable numérique
- if($exist === 1) : if($exist) suffit
- si ton include ne contient pas de sortie html, mets le en debut de script une fois pour toute
- ta verification de mail est mauvaise : une adresse en toto@titi.com est valide, mais toto@toto.tutu.com non
- les die() de partout c'est horrible
- y a surement d'autres truc, la j'ai regardé en diagonale

snarky

oki oki merchi bien, je vais faire les arrangements necessaires :hello:

Message édité par snarky le 07-07-2005 à 02:16:59

snarky

rho@bond.edu.au est un mail valide non ?

gatsusat

oué

gatsusat

il existe une regexp qui fait 200 lignes et elle valide tout type d'adresse email donc là pour ton cas, c'st de verifier kil y a au moins un . dans le domaine de l'adresse mail

par exemple un mail peut etre :

president.jacques.chirac@elysee.gouv.fr

snarky

Code :

...
$exist = mysql_num_rows($search_mail_exist);
if($exist)
{
...

est ce que ça serait pas mieux, si je m'étais directement la valeur de $exist dans le if() ?

Code :

...
if(mysql_num_rows($search_mail_exist))
{
...

Message édité par snarky le 07-07-2005 à 14:17:49

sielfried

Si t'as plus besoin du flag après, alors c'est un peu plus rapide oui. [:spamafote]

---------------
StarCraft Professional Gaming Database | [Ze Topic] Starcraft/BroodWar

snarky

bah vu que c'est une variable créée a l'occasion pour faire le test. alors autant l'enlever ^^ merci

Publicité

FORUM HardWare.fr

Programmation

PHP

Correction

Sujets relatifs
[Résolu] Correction Feuille de Style CSS Firefox > IE	perceptron - correction d'erreur et descente du gradient
controles des champ et correction du formulaire PHP	Algorithme / méthode de correction ortographique ?
correction please	jtree correction
Code de correction d'erreur	Correction de mon code...
Correction dans la saisie	VB6 : Cette fonction a besoin d'une correction.. help !
Plus de sujets relatifs à : Correction

Page générée en 0.121 secondes