Discussion :

[schats]

bonjour

j'ai un première problème suite a problème de spam incessant j'ai décider de changer le système

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
elseif((preg_match("#viagra|^cialis$|cialis</a>|levitra|<a | http:// | www |<img #i" , $message)) == true)
   {
   $erreur[] = 'spam interdit';
   }

maitenant si je met une adresse donc http ou www ben le code ne réagit pas :s

un autre problème le membre peu s'il veut laisser sont adresse mail
mais avec la conditions

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
elseif(isset($email)) 
 {
 if(strchr($email,"@") == FALSE)
 {
		$erreur[] = 'mettre une vrai adresse mail';

la nous somme obliger de mettre une adresse mail je ne comprend pas pourquoi ???

merci de m'aider schats

[Nicolas N.]

Bonjour,

Je pense que vous avez simplement oublié les anti-slash ainsi que les paranthèses. J'ai testé le code suivant, et il fonctionne :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
elseif((preg_match("#(viagra|^cialis$|cialis<\/a>|levitra|<a|http:\/\/|www|<img)#i" , $message)) == true)
   {
   $erreur[] = 'spam interdit';
   }

Pour votre deuxième problème, le script vérifie la présence du '@' dans le cas où le champs 'email' est rempli grâce à la fonction isset(); donc si le champs 'email' n'est pas rempli, il ne vérifie rien .

Nico.

[schats]

bonjours ,

Pour votre deuxième problème, le script vérifie la présence du '@' dans le cas où le champs 'email' est rempli grâce à la fonction isset(); donc si le champs 'email' n'est pas rempli, il ne vérifie rien .

voila c'est ca que je veut mais la il me met

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

$erreur[] = 'mettre une vrai adresse mail';

même si on a pas mit d'adresse mail

[julp]

Je pense que vous avez simplement oublié les anti-slash ainsi que les paranthèses. J'ai testé le code suivant, et il fonctionne

Les parenthèses sont inutiles ici car elles n'ont aucun rôle à jouer, que ce soit pour la capture ou dans l'expression. Les slashs n'ont pas non plus besoin d'être échappés pour la simple et bonne raison que ce ne sont ni des métacaractères ni utilisés comme séparateur.

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
elseif((preg_match("#viagra|^cialis$|cialis</a>|levitra|<a | http:// | www |<img #i" , $message)) == true)
   {
   $erreur[] = 'spam interdit';
   }

Les espaces sont volontaires ?

un autre problème le membre peu s'il veut laisser sont adresse mail
mais avec la conditions

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
elseif(isset($email)) 
 {
 if(strchr($email,"@") == FALSE)
 {
		$erreur[] = 'mettre une vrai adresse mail';

La variable $email elle sort d'où ? N'y aurait-il pas confusion avec la fonction empty ? Je pense qu'on peut également trouver mieux que strchr (une expression régulière par exemple) ...

[schats]

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
elseif((preg_match("#viagra|^cialis$|cialis</a>|levitra|<a | http:// | www |<img #i" , $message)) == true)
   {
   $erreur[] = 'spam interdit';
   }

non les espace sont volontaire enfin c'est pour voir un peu clair :s

la variable email est

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
if(!empty($_POST) ) // le form a été soumis
{
  // servira à enregistrer les messages pour les erreurs rencontrées
   $_POST = array_map('trim', $_POST);
   $erreur = array();
 
   $pseudo = mysql_real_escape_string($_POST['pseudo']);
   $email = mysql_real_escape_string($_POST['email']);
   $message = mysql_real_escape_string($_POST['message']);

[julp]

non les espace sont volontaire enfin c'est pour voir un peu clair :s

Justement sans l'option x vos espaces seront considérés comme faisant partie du masque !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

elseif(isset($email))

sera toujours vrai à partir du moment où le formulaire où est défini le champ email est soumis ...

[schats]

euh alors comment faire ???

[julp]

euh alors comment faire ???

Je vous l'ai dit

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
elseif((preg_match("#viagra|^cialis$|cialis</a>|levitra|<a | http:// | www |<img #i" , $message)) == true)
   {
   $erreur[] = 'spam interdit';
   }

vous supprimez les espaces sans signification ou vous utilisez l'option x.

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

elseif(isset($email))

a priori vous devez remplacer isset par !empty

[Korko Fain]

a partir du moment ou tu fais un
$toto = quelquechose
$toto sera considéré comme défini. Meme si ton quelque chose est une chaine vide, NULL ou FALSE.

pour tester la chaine vide, utilise empty()
pour tester NULL utilise is_null()
et pour false, un if suffit (puisque c'est déja un booléen).