Discussion :

[dark_vidor]

Bonjour,

J'ai créer un formulaire basic ... (issue d'une génération en php, utilisation de template...)

http://www.team-gt5net.com/~championnat/admin/team.php

Pour l'instant il n'est pas protégé, et il n'as aucune action derière juste un affichage des valeurs ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<form name="add_team" method="post" action="team.php?submit=1" onSubmit="javascript:return valider_form(this);">
  <fieldset>
  <legend>Ajouter une &eacute;quipe</legend>
  Tag :  <input type="text" name="tag_team" />
  Site : <input type="text" name="site_team" />
  Manager : <select name="manager_team">
    <option value="2">dark_vidor</option>
    <option value="1">Invité</option>
  </select>
  <input type="submit" value="Envoyer">
  </fieldset>
</form>

et ma fonction pour vérifier mon formulaire est la suivante :

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
function isEmpty( str )
{
	return ( str == null || str == "" ) ? true : false ;
}
 
 
function valider_form(form)
{
	//On regarde si tous les champs sont bien remplis
	for( var i = 0 ; i < form.elements.length ; i++ )
	{
		if( isEmpty( form.elements[i].value ) )
			alert( form.elements[i].name+" est vide");
	}
}

ce que je n'arrive pas a comprendre c'est que si je soumet mon formulaire vide j'ai undefined est vide, tag_team est vide, site_team est vide ... comme si form.elements[0].value n'exitait pas ? or à l'index 0 on devrait avoir tag_team

je pourrait mettre un for i = 0 to i = form.elements.length mais avant j'aimerais comprendre pourquoi

merci

[Jumano]

Bonjour,
Le "undefined" doit correspondre à "<fieldset>" ....
Donc tu peux modifier ta fonction comme ceci :

Code :

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

for( var i = 1 ; i < form.elements.length ; i++ )

Si biensûr tous tes formulaires contiennent ce premier élément avant les "inputs"

[dark_vidor]

ha ok

[SpaceFrog]

Je pense qu'il y a une fonction générique proposée dans les contibutions ...

[dark_vidor]

ba vous pouvez mettre celle la ... elle est simple et permet de voir si tout les champs sont remplis pour n'importe quel formulaire ... ça peut donner une base pour d'autre

[Bovino]

Code :

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

return ( str == null || str == "" ) ? true : false ;

Tu peux remplacer par

Code :

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

return !str;