onsubmit avec trois fonctions

**silent_murder** · 10/05/2010, 20h49

Bonjour,

J'ai un soucis javascript. J'ai créé un tableau html avec des champs de texte dans chaque cellule (tableau généré en js à l'aide de deux champs de texte un pour les colonnes l'autre pour les lignes).
Dans ce tableau le header contient les quantités (produits) la colonne de gauche les valeurs (type de produit). Le reste sont les champs de texte correspondant au prix (q/valeur).
J'arrive à créer sans soucis ce tableau.

le problème se pose lors du fameux onsubmit. J'ai trois fonctions de contrôle de champs, une pour chaque type (quantité,valeur,prix).

Lorsque je mets les trois fonctions ensembles, cela me créer "une fin du monde" la soumission ne marche que pour une partie (ex:seulement la quantité) voire deux mais pas trois

Ce que je voudrais c'est si un champs de n'importe quelle partie n"est pas rempli la soumission soit avortée, bien sûr si un ou plusieurs champs des trois ne sont pas remplis la soumission ne pas non plus.

Voici les fonctions de contrôle :

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
function verifQ()
{ 
	var c =document.getElementById("col").value;	
	for(var i=2;i<=c;i++)
	{
 
		if (document.getElementById("Q"+i).value=="")
		{
			 return false;
		}		
	}
 
}
 
 
function vP()
{
	var col =parseInt(document.getElementById("col").value);
	var ligne =parseInt(document.getElementById("ligne").value);
	var cell=(col*ligne);
	var d = (col+2);
	var z=0; 
	for(var t=d;t<=cell;t++)
	{
		z++;			
 
		if(z%(col)==0)
		{
			z=0;t=t+1;	
		}
		if(document.getElementById("P"+t).value=="")
		{
			alert(2);
			return false; 				
		}
	}				
}
 
function verV()
{
	var lig =parseInt(document.getElementById("ligne").value);
	for(var n=1;n<=lig;n++)
	{
		if(document.getElementById("V"+n).value=="")
		{
			alert(1);
			return false;
			break;
		}
 
	}
 
}

voici le formulaire:

Code :

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

<form name="attributes" action="<?php echo tep_href_link(FILENAME_PRODUCTS_ATTRIBUTES, 'action=' . $form_action . '&option_page=' . $option_page . '&value_page=' . $value_page . '&attribute_page=' . $attribute_page); ?>" method="post" onsubmit="return(verifQ() && verV() && vP());">

J'ai essayé de bien résumer, dites moi si ce n'est pas clair.

PS: ne critiquez pas trop le code je débute dans la profession, sinon je vais débuter dans la dépression

**SpaceFrog** · 10/05/2010, 22h28

Pourquoi ne pas tout faire en une seule et même fonction de vérification ?

Sinon c'est bien de faire le return false quand c'est pas bon, mais:
1/ un return fait quitter la fonction
2/ faut pas oublier le return true quand c'est bon

ex:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function verifQ()
{ 
	var c =document.getElementById("col").value;	
        var ok=0
	for(var i=2;i<=c;i++)
	{
		if (document.getElementById("Q"+i).value=="")
		{
		  ok++;
		}		
	}
        return (ok==0);
 
}

**silent_murder** · 10/05/2010, 23h15

J'ai tenté en une fonction que voici :

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
function verif()
{ 
	var ok=0;
	var c =document.getElementById("col").value;	
	for(var i=2;i<=c;i++)
	{
 
		if (document.getElementById("Q"+i).value=="")
		{
			 ok++;
 
		}		
	}
 
 
	var col =parseInt(document.getElementById("col").value);
	var ligne =parseInt(document.getElementById("ligne").value);
	var cell=(col*ligne);
	var d = (col+2);
	var a=0; 
 
 
	for(var t=d;t<=cell;t++)
	{
		a++;			
 
		if(a%(col)==0)
		{
			a=0;
			t=t+1;	
		}
		if(document.getElementById("P"+t).value=="")
		{
			alert(document.getElementById("P"+t).name); 
			ok++;
 
		}
	}
 
	var ok2=0;
	var lig =parseInt(document.getElementById("ligne").value);
 
	for(var n=1;n<=lig;n++)
	{
		alert(1);
		if(document.getElementById("V"+n).value=="")
		{
			ok++;
 
 
 
		}
 
	}	
 
	return(ok==0);	
 
}

Cela ne marche absolument pas, lors du click la deuxième boucle fait le tour de tous ses champs vides jusqu'à valider le formulaire

, la troisième boucle quant à elle ne s'enclenche pas

**silent_murder** · 11/05/2010, 10h15

Bon je viens de changer une partie du 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
function verif()
{ 
	var ok=0;
	var col =parseInt(document.getElementById("col").value);
	var ligne =parseInt(document.getElementById("ligne").value);
	var cell=(col*ligne);
	var d = (col+2);
	var a=0; 
 
	for(var i=2;i<=col;i++)
	{
 
		if (document.getElementById("Q"+i).value=="")
		{
			 ok=ok+1; alert(3);
			 break;
		}		
	}
 
	for(var t=d;t<=cell;t++)
	{
		a++;			
 
		if(a%(col)==0)
		{
			a=0;
			t=t+1;	
		}
		if(document.getElementById("P"+t).value=="")
		{
			//	alert(document.getElementById("P"+t).name);  
			ok=ok+1; alert(2);
			break;				
		}
	}
 
 
	for(var n=1;n<=ligne;n++)
	{
 
		if(document.getElementById("V"+n).value=="")
		{
			ok=ok+1;alert(1);
			break;						
		}
 
	}	
 
	if(ok==0)
	{
		return true;
 
	}
	else
	{
		return false;	
 
	}
 
}

Toutes les boucles fonctionnent, déjà ça mais le formulaire se valide s'il n'y a pas de champs vide dans la première et troisième boucle alors que la deuxième boucle fait ok=ok+1; Le pire si je mets return false dans le if et dans le else le formulaire est quand même envoyé dans le cas prédemment expliqué.

Merci de m'aider ça me saoule.

Ps: dsl pour les admins je connaissais pas la balise code, je sais c'est chiant un n00b.