Validation formulaire en JS avant envoi par appui entrer [Résolu]

[darkterreur]

Bonjour,

J'ai fait un code tout simple avec en formulaire et un champs de saisie et une validation par la touche entrer.
Le but et de renommer un fichier en respectant un model.

Cependant je en rentre pas dans ma fonction JS, je ne trouve pas mon erreur, pourtant tout a l'air correcte.

Mon JavaScript:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
<script language="javascript">
	ControleChamp(var id){
		alert("je suis appeller");
		var regex = /[0-9]{2}\s\d\s[0-9]{2}\s\d\w\s[0-9]{3}\s[0-9]{2}/;
 
		if(regex.test(id.value))
		{
			alert("true");
			return true;
		}
		else
		{
			alert("false");
			id.value = "__ _ __ __ ___ __";
			id.style="border-color:red";
			return false;
		}
	}
 
 
</script>

Mon formulaire:

Code :

1
2
3
4
 
<form method="post" <?php echo"action='traitement.php?var=".$NameFile."'";?> onBlur="ControleChamp(this)">
				<label>Num&eacute;ro BL</label> : <input id='NumBl' type="text" name="NumBl"/>
				</form>

Aucun alert() ne s'affiche donc je ne rentre même pas dans la fonction en revanche le formulaire envoi bien les deux variable et je récupère bien de l'autre coté.

Quel erreur ai je faite?

Je vous remercie par avance pour vos lumières

[hariman]

Bonjour,

Au lieu de onblur, utilise :

Code :

onsubmit="return ControleChamp(this)"

[darkterreur]

Bonjour,

merci de votre réponse rapide,
malheureusement cela ne fonctionne pas non plus.

J'avais tester avant, mais je viens de tester avec votre code des fois que ...
je copie la modification au cas ou:

Code :

1
2
3
4
 
<form method="post" <?php echo"action='traitement.php?var=".$NameFile."'";?> onsubmit="return ControleChamp(this)">
				<label>Num&eacute;ro BL</label> : <input id='NumBl' type="text" name="NumBl"/>
				</form>

Merci d'avance pour vos lumières

[Bovino]

Code :

ControleChamp(var id)

Ca sort d'où cette syntaxe ?

D'autre part, si j'en crois ton code, le paramètre id (qui n'est au passage pas un nom de paramètre particulièrement judicieux) correspond à l'objet formulaire. Je vois donc mal ce que tu attends de

Code :

id.value

[darkterreur]

Bonjour Bovino,

Mhhh visiblement faire une déclaration de variable dans les parametre attendu semble etre une abération, je corrige cela de suite.

Mais meme corriger je devrais avoir le premier alert ().

Pour le :

Code :

id.value

cela serai t'il plus judicieux?

Code :

document.getElementById('id').value

Je vous remercie d'avance pour vos lumières.

[Bovino]

Il y a beaucoup de fondamentaux que tu ne sembles pas maitriser...

Pour déclarer une fonction, ça se fait avec le mot clé function ; un formulaire ne possède pas de propriété value, ce sont les champs qui en ont.

[darkterreur]

Merci Bovino,

Tes parole bien que dur n'est reste pas moins juste. Ne pratiquent pas régulierement j'ai fait des erreurs idiotes.

En tout cas cela fonctionne maintenant
je colle le code complet des fois que cela serve:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function ControleChamp(){
			alert("je suis appeller");
			var regex = /[0-9]{2}\s\d\s[0-9]{2}\s\d\w\s[0-9]{3}\s[0-9]{2}/;
 
			if(regex.test(document.getElementById('NumBl').value))
			{
				alert("true");
				return true;
			}
			else
			{
				alert("false");
				document.getElementById('NumBl').value = "__ _ __ __ ___ __";
				document.getElementById('NumBl').style="border-color:red";
				return false;
			}
		}

Code :

1
2
3
<form method="post" <?php echo"action='traitement.php?var=".$NameFile."'";?> onsubmit="return ControleChamp()">
				<label>Num&eacute;ro BL</label> : <input id='NumBl' type="text" name="NumBl"/>
				</form>

Puis je me permettre de profiter de mon post pour voir si l'on améliorer ma regex qui n'est fonctionne pas exactement comme je le voudrai ou dois je ouvrir un second post?

[Bovino]

Tes parole bien que dur n'est reste pas moins juste.

Il ne faut pas s'en formaliser, il n'y a rien de méchant

Puis je me permettre de profiter de mon post pour voir si l'on améliorer ma regex

Il est préférable d'ouvrir une nouvelle discussion.

Dernières petites précisions.
Dans ton code, tu utilises plusieurs fois document.getElementById('NumBl'), dans ce cas, il est préférable de le stocker dans une variable, ça évitera à JavaScript d'avoir à faire la recherche à chaque fois.

Code :

document.getElementById('NumBl').style="border-color:red";

Là, tu redéfinis l'ensemble des styles de l'élément, il est préférable de passer par la propriété spécifique.

Ton code rectifié :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function ControleChamp(){
	alert("je suis appeller");
	var elem = document.getElementById('NumBl');
	var regex = /[0-9]{2}\s\d\s[0-9]{2}\s\d\w\s[0-9]{3}\s[0-9]{2}/;
	if(regex.test(elem.value))
	{
		alert("true");
		return true;
	}
	else
	{
		alert("false");
		elem.value = "__ _ __ __ ___ __";
		elem.style.borderColor="red";
		return false;
	}
}

[darkterreur]

Merci beaucoup pour ces indications ainsi que votre aide.