Discussion :

[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 :

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
 
<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 :

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

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 :

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

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 :

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

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 :

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

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 :

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

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 :

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

id.value

cela serai t'il plus judicieux?

Code :

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

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 :

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
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 :

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

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 :

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

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 :

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
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.