Discussion :

[Mickael2604]

Bonjour à tous.

Dans le cadre de mon application, je dois vérifier les champs avec du javascript. Il y a quelques temps, tout marchait bien, mais aujourd'hui plus rien. Je ne comprends pas.Voici mon code :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
<SCRIPT >
function verif()
{
	alert("eh");
	if(document.formulaire.nom.value == ""){
		alert("Veuillez entrer un nom \n");
		return false;
		}
	if(document.formulaire.prenom.value == ""){
		alert("Veuillez entrer un prénom \n");
		return false;
		}
	if(document.formulaire.adresse.value == ""){
		alert("Veuillez entrer une adresse \n ");
		return false;
		}
	if((document.formulaire.codeP.value == "") && !(verifEntier(document.formulaire.codeP.value))){
		alert("Veuillez entrer un code postal valide");
		return false;
		}
	if(document.formulaire.ville.value == ""){
		alert("Veuillez entrer une ville");
		return false;
		}
	if((document.formulaire.conso.value == "") && !(verifEntier(document.formulaire.conso.value)))
	{
		alert("Veuillez entrer une conso valide ");
		return false;
	}
	if(!(verifEntier(document.formulaire.Jan.value)) || !(verifEntier(document.formulaire.Fev.value)) || !(verifEntier(document.formulaire.Mar.value)) || !(verifEntier(document.formulaire.Avr.value)) || !(verifEntier(document.formulaire.Mai.value)) || !(verifEntier(document.formulaire.Jun.value)) || !(verifEntier(document.formulaire.Jul.value)) || !(verifEntier(document.formulaire.Aou.value)) || !(verifEntier(document.formulaire.Sep.value)) || !(verifEntier(document.formulaire.Oct.value)) || !(verifEntier(document.formulaire.Nov.value)) || !(verifEntier(document.formulaire.Dec.value)))
	{
		alert("Veuillez entrer des pourcentages valides ");
		return false;
	}
	else
	{
		if((document.formulaire.Jan.value+document.formulaire.Fev.value+document.formulaire.Mar.value+document.formulaire.Avr.value+document.formulaire.Mai.value+document.formulaire.Jun.value+document.formulaire.Jul.value+document.formulaire.Aou.value+document.formulaire.Sep.value+document.formulaire.Oct.value+document.formulaire.Nov.value+document.formulaire.Dec.value != 100)
		{
			alert("La somme de vos poucentages n'est pas égale à 100");
		}
	}
	return true;
}
 
function verifEntier(e)
{
if(e.values == "" ) return false;
else
return e.match("^[0-9]{0,}$");
}
</SCRIPT>
 
 
<span><center>Ajouter client :</center></span><br />
<br /><br />
<form method="POST" action="#" name="formulaire"  onsubmit="return verif();">
<center>
<table>
	<tr>
	<td>
		<table>
			<tr> <td> Nom : </td> <td> <input type="text" name="nom"></td></tr>
			<tr> <td> Prenom : </td> <td> <input type="text" name="prenom"></td></tr>
			<tr> <td> Adresse : </td> <td> <input type="text" name="adresse"></td></tr>
			<tr> <td> Code postal : </td> <td> <input type="text" name="codeP" size="5"></td></tr>
			<tr> <td> Ville : </td> <td> <input type="text" name="ville"></td></tr>
			<tr> <td> Consommation : </td> <td> <input type="text" name="conso"></td></tr>
			<tr> <td> Type : </td> <td> 
			<select name="type">
				<option value="1">particulier</option>
				<option value="2">professionnel</option>
			</select></td></tr>
			<tr> <td> Type Citerne </td> <td> 
		</table>
	</td>
	<td>
		<table>
			<TR><center>Estimation Consommation</center></tr>
			<tr><td> Janvier </td><td><input type="text" name="Jan" size="2"> %  </td><td> Juillet </td><td><input type="text" name="Jul" size="2"> %</td></tr>
			<tr><td> Fevrier </td><td><input type="text" name="Fev" size="2"> %  </td><td> Août </td><td><input type="text" name="Aou" size="2"> %</td></tr>
			<tr><td> Mars </td><td><input type="text" name="Mar" size="2"> %  </td><td> Septembre </td><td><input type="text" name="Sep" size="2"> %</td></tr>
			<tr><td> Avril </td><td><input type="text" name="Avr" size="2"> %  </td><td> Octobre </td><td><input type="text" name="Oct" size="2"> %</td></tr>
			<tr><td> Mai </td><td><input type="text" name="Mai" size="2"> %  </td><td> Novembre </td><td><input type="text" name="Nov" size="2"> %</td></tr>
			<tr><td> Juin </td><td><input type="text" name="Jun" size="2"> %  </td><td> Decembre </td><td><input type="text" name="Dec" size="2"> %</td></tr>
		</table>
	</td>
	</tr>
</table>
</table>
	<br />
	<input type="submit" value="Valider" name="valid">
	</center>
</form>
 
<br />

Mon alert ne marche meme pas, il ne passe pas du tout dans le Javascript, help me please !

Merci

[jnore]

salut

pour commencer, tu as une parenthèse de trop ici:

Code :

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

1
2
3
 
 
if(document.formulaire.Jan.value+document.formulaire.Fev.value+document.formulaire.Mar.value+document.formulaire.Avr.value+document.formulaire.Mai.value+document.formulaire.Jun.value+document.formulaire.Jul.value+document.formulaire.Aou.value+document.formulaire.Sep.value+document.formulaire.Oct.value+document.formulaire.Nov.value+document.formulaire.Dec.value != 100)

Je l'ai retirée

[E.Bzz]

Bonjour,
+1 pour la remarque de jnore.

J'ajouterais que le test est un peu aléatoire dans la mesure où tu concatènes des chaînes de caractères (les .value) que tu compares au final à une valeurs numérique.
Pour additionner tes value, tu peux :

• soit multiplier chacune par 1 ce qui effectue la conversion

  Code :

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

  if(document.formulaire.Jan.value*1+...

• soit utiliser une fonction de conversion prévue par le langage

  Code :

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

  if(parseFloat(document.formulaire.Jan.value,10)+parseFloat(...

De plus, une syntaxe plus actuelle (et plus facile à utiliser) pour

Code :

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

document.formulaire.Jan.value

serait

Code :

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

document.forms['formulaire'].elements['Jan'].value

EDIT : par contre ton nav aurait dû te signaler l'erreur de la parenthèse (c'est sûrement elle qui plante le code de ta page) active la "notification des erreurs de scripts" (ou équivalent) dans les options de ton navigateur

A+