Discussion :

[stealth35]

bon

- les $vop_truc, utilise un tableau
- le str_replace, prend en charge aussi les tableaux pas besion de le faire plein de fois, de plus c'est strip_tags que tu fais, et faire, trime + stripslshes + htmlspecialchars, la chaine ne ressemble plus a rien
- ton if($color) ne sert a rien non plus, fait un tableau
- le isset($_POST['op_truc']), pareil, utilise un array_filter
- $regex_mail pareil, fait un filter_var
- ne jamais faire confiance au HTTP_REFERER, moi j'en envoie pas
- les X empty() ne sert a rien, fait un array_filter
- tout ça pour envoyer un Mail ?, utilise SwiftMailer

[dancom5]

Bonjour.

J'ai réglé le problème en changeant l'endroit où placer le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
if(strlen($message) > 19 || strlen($op_z) != 0 ) {
 
  if (mail($to, $sujet, $msg, $headers)){
    $alert = 'E-mail envoyé avec succès';
 
    unset($_POST);
 
  }
  else{ $alert = 'Erreur d\'envoi de l\'e-mail'; }
 
}else{ $alert = 'Sélectionner une ou plusieurs options et/ou laisser un message'; }

A l'endroit, où il était placé mon code, strlen indiquait tout le temps 1 ou 0.
C'est pour cette raison que ça ne réussissait pas.

Donc, j'ai simplifié en ne mettant que :

Code :

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

1
2
3
if(strlen($message) > 19 || strlen($op_z) != 0 ) {
...mail(...)...
}else{ $alert = 'Sélectionner une ou plusieurs options et/ou laisser un message'; }

J'ai opté pour soit un ou l'autre, ou bien les deux.
Si je sélectionne une option, on peut laisser moins de 20 caractères;
si non, il faut laisser plus de caractères.

Pour la réponse de stealth35 me donnant des pistes de corrections du code,
je vais regarder ça aussi. Y'a des choses qui proviennent du tuto d'ici.

Je ne suis pas très familier avec les "tableaux" d'une manière avancée. Alors,
j'aurais besoin d'aide. Ça sera dans un autre post.

Merci tout le monde pour l'aide que vous m'avez donnée.