Discussion :

[trx337]

bonjour tout le monde
suite à mon besoin d'éviter des emails frauduleux , j'ai écrit ce code qui fonctionne bien sur vos conseils antérieurs mais je pense qu'une des lignes du codes peut etre amélioré car passer en javascript pour écrire en php n'est pas logique :
c'est cette ligne ci :

Code :

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

echo "<script language='JavaScript'>document.getElementById('h1glu').innerHTML= ('ne pas envoyer d\'adresse:  non envoyé') ;  </script>";

Code html :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
<div id="contenu">
<h1 id="h1glu">Merci pour votre envoi</h1>
</div>
 
<?php
 
$regex = '/http|www|href|src|open/i';
 
 
 
if (
 
(preg_match($regex, $_POST['messagelivre'])  )||
(preg_match($regex, $_POST['prenom'])  )||
(preg_match($regex, $_POST['nom'])  )||
(preg_match($regex, $_POST['email'])  )
 
 
 
 
)
        
 
{ 
 
 
 
echo "<script language='JavaScript'>alert  ('ne pas envoyer d\'adresse non envoyé clon');  </script>"; 
#document.getElementById("h1glu").innerHTML= ('fgh')  ;
 
echo "<script language='JavaScript'>document.getElementById('h1glu').innerHTML= ('ne pas envoyer d\'adresse:  non envoyé') ;  </script>"; 
 
 
 
return false;
 }

[trx337]

en fait ma question se résume à

peut on se passer de javascript pour écrire ceci ? :

Code :

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

echo "<script language='JavaScript'>document.getElementById('h1glu').innerHTML= ('ne pas envoyer d\'adresse:  non envoyé') ;  </script>";

[Invité]

Bonjour,

oui :

1- On enregistre le message dans une variable.

Code :

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

$msg = 'ne pas envoyer d\'adresse:  non envoyé';

2- On affiche le message au bon endroit :

Code :

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

<h1 id="h1glu"><?php echo $msg; ?></h1>

Important : généralement, le traitement PHP s'effectue AVANT l'affichage HTML (contrairement à ce que tu as fait).

[trx337]

j'ai essayé mais j'ai l'impression que javascript permet dans mon cas de remplacer au bon endroit le
"Merci pour votre envoi"
par
"ne pas envoyer d\'adresse: non envoyé"

donc dans mon cas , javascript fait mieux que php grance au code :

Code :

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

document.getElementById('h1glu').innerHTML

[Invité]

Normal : ton code est mal structuré.
L'ajout de JavaScript ici en est justement le signe.

(et si l'utilisateur a désactivé JavaScript sur son navigateur, ça ne s'affichera pas...)

Je répète :

• on fait d'abord le traitement PHP
• ensuite seulement, on fait l'affichage HTML

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
$msg = 'Merci pour votre envoi'; // message par défaut
$regex = '/http|www|href|src|open/i';
if (
	(preg_match($regex, $_POST['messagelivre']) )||
	(preg_match($regex, $_POST['prenom']) )||
	(preg_match($regex, $_POST['nom']) )||
	(preg_match($regex, $_POST['email']) )
){ 
	echo "<script>alert('ne pas envoyer d\'adresse non envoyé clon');</script>"; 
 
	$msg = 'ne pas envoyer d\'adresse:  non envoyé';
	//return false;
}
?>
<div id="contenu">
	<h1 id="h1glu"><?php echo $msg; ?></h1>
</div>

N.B. A quoi sert le return false; ??
(à priori à rien, non ?)

[trx337]

d'accord je comprends bien
le return false permettait de couper la fin du script qui ne servait plus puisque c'était false

autrement on pouvait mettre un else comme ceci :

Code :

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

1
2
3
4
5
else
 
{
echo  "    <h1 align='center'>  Merci pour votre envoi</h1>                 " ;
}

mais ok ton script est bien

[trx337]

bonjour

cela fait un petit temps que c'était résolu mais je reçois des emails ne contenant rien
Donc j'aimerais savoir comment ajouter l'ensemble vide à ma variable regex :
serait ce ceci : (deux barres avec du vide) ?

Code :

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

$regex = '/http|www|href|src|open||/i';

merci pour votre réponse

Code :

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

1
2
3
4
5
6
7
8
9
$regex = '/http|www|href|src|open/i';
 
if (
 
(preg_match($regex, $_POST['messagelivre'])  )||
(preg_match($regex, $_POST['prenom'])  )||
(preg_match($regex, $_POST['nom'])  )||
(preg_match($regex, $_POST['email'])  )
)

[Invité]

Bonjour,

Une regex n'est pas une baguette magique.

Là, il suffit de tests "normaux", pour vérifier qu'un champ est bien rempli :

// champs obligatoires

Code :

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

if( !empty($_POST['messagelivre']) && !empty($_POST['nom']) ...........)

[trx337]

bonjour et merci
bon je préfère vérifier que je n'ai pas fait d'erreurs :

est ce que j'ai bon ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
if (
 
(preg_match($regex, $_POST['messagelivre'])  )||
(preg_match($regex, $_POST['prenom'])  )||
(preg_match($regex, $_POST['nom'])  )||
(preg_match($regex, $_POST['email'])  )||
 
(empty($_POST['messagelivre']) && empty($_POST['prenom'])  && empty($_POST['nom']) && empty($_POST['email'])  )
 
 
)
 
{ 
 
 
echo "<script language='JavaScript'>alert  ('ne pas envoyer d\'adresse non envoyé' );  </script>"; 
 
 
echo "<script language='JavaScript'>document.getElementById('h1glu').innerHTML= ('ne pas envoyer d\'adresse:  non envoyé') ;  </script>"; 
 
 
 
 
 
 
return false;
 } 
 
 
?>