[DOM] DOM : accès à un élément

Version imprimable

Bonjour,

Ma fonction fait appel aux valeurs de deux éléments pour les comparer.
Ces éléments sont deux <select> dont les noms sont de la forme svX et spY où X et Y sont des nombres.
Je cherche à comparer svX et spY avec X=Y, en effet j'ai un nombre de <select> dans mon formulaire égal à val_de_inc.
J'ai donc essayé des "construire" les noms :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 function checkscript(){ flag_alert = false; flag = true; var ErrMsg; var counter; var svAv; var spAv; for (counter = 0; counter <= document.forms['venteform'].val_de_inc.value; counter++){ svAv = sv+counter; spAv = sp+counter; if(document.forms['venteform'].svAv.value>document.forms['venteform'].spAv.value){ flag-alert=true; break; } } if(flag_alert == true){ flag = false; ErrMsg = 'Les ventes ne peuvent pas être sauvegardés.'; alert(ErrMsg); } return flag; }
Et bien entendu ça ne fonctionne pas !
J'ai procédé de cette manière en m'inspirant d'un code que j'utilise en php qui fait la même chose.

Merci pour votre aide

09/06/2008, 17h25
Matthieu2000

Citation:

document.forms["formname"].elements["elementname"]
09/06/2008, 17h26
SpaceFrog

les value ne sont pas des numérique ...
un petit *1 ne leur ferait sans doute pas de mal ...
09/06/2008, 17h32
E.Bzz

Bonjour,
de plus tu affectes

Code:

flag-alert=true;

Mais tu as déclaré

Code:

flag_alert = false;

A+
09/06/2008, 17h44
SpaceFrog

oui un nom de variable ne peut contenir un opérateur ...

Merci,
J'ai donc corrigé. (\" car stocké dans une variable php)
Code:

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 function checkscript(){ flag_alert = false; flag = true; var ErrMsg; var counter; var svAv; var spAv; for (counter = 0; counter <= document.forms[\"venteform\"].elements[\"sval_de_inc\"].value*1; counter++){ svAv = sv+counter; spAv = sp+counter; if(document.forms[\"venteform\"].elements[\"svAv\"].value*1>document.forms[\"venteform\"].elements[\"spAv\"].value*1){ flag_alert=true; break; } } if(flag_alert == true){ flag = false; ErrMsg = 'Les ventes ne peuvent pas être sauvegardés\nLa vente d'un produit est supérieure au stock lui étant associée.'; alert(ErrMsg); } return flag; }
ça ne fonctionne pas. Est ce que j'emploie le *1 comme il faut où est ce que vous pensez que dans

Code:

document.forms[\"venteform\"].elements[\"svAv\"].value*1

svAv ne prend pas la valeur sv19 (pour exemple).

Sinon dans le reste de l'application, dont j'ai repris le développement, tous les éléments sont récupérés avec document.forms['nomduform'].nomdelelement et ca fontionne. Cela pose t'il un problème. C'est une application à usage interne , à utiliser uniquement avec firefox.

09/06/2008, 17h57
E.Bzz

Citation:

Envoyé par PKO06

J'ai donc corrigé. (\" car stocké dans une variable php)

Oui mais on ne devrait pas retrouver ici les caractères de protection (\) puisque ça semble être le code généré :?

A+
09/06/2008, 18h03
PKO06

Non vraiment aucun problème de se coté là .. en php :

j'ai une class page.

$page = new page();

<script type="text/javascript" language="JavaScript">

</script>

et dans le fichier

$page->javascirpt .= " ... ";

C'est très peu clair, assez complexe mais ça fonctionne très bien.
09/06/2008, 18h15
Matthieu2000

Citation:

svAv = "sv"+counter;
spAv = "sp"+counter;

encore des quotes...
ps 1 : regarde la console et corrige. :aie:
ps 2 : la variable svAv n'a pas besoin de quote mais les caines de caractères oui.
09/06/2008, 18h30
PKO06

Ok merci, corrigé à nouveau ... fonctionnel toujours pas.
Le problème c'est que je code avec eclipse+phpeclipse. Comme mon code javascript est stocké dans une variable php ... je n'ai aucune coloration syntaxique.
10/06/2008, 08h39
SpaceFrog

mets nous le code généré ...

et pour les " c'est que tu code mal ton php ...
une variable ph doit être entre ' ' ...

Code:

$variable= 'document.forms["venteform"].elements["svAv"].value*1'

comme ça la syntaxe html / javascript est respectée

Je vous donne mon code généré :

Code:

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
 
function checkscript(){
	flag_alert = false;
	flag = true;
	var ErrMsg;
	var counter;
	var svAv;
	var spAv;		
 
	for (counter = 0; counter <= (document.forms["venteform"].elements["val_de_inc"].value)*1; counter++){
		svAv = "sv"+counter;
		spAv = "sp"+counter;
		if(document.forms["venteform"].elements["svAv"].value*1>document.forms["venteform"].elements["spAv"].value*1){
			flag_alert=true;
			break;
		}
	}
 
	if(flag_alert == true){
		flag = false;
		ErrMsg = 'Les ventes ne peuvent pas être sauvegardées.';
		alert(ErrMsg);
	}
 
	return flag;
}

Merci pour votre aide.

10/06/2008, 16h15
Matthieu2000

Citation:

if(document.forms["venteform"].elements[svAv].value*1>document.forms["venteform"].elements[spAv].value*1){

sans quote!

Sans les quotes cela revient au même ... rien :(
J'ai utilisé une méthode bourrin.

J'ai généré mon code javascript en php :

Code:

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
 
$scriptJS = '
 function checkscript(){
		flag_alert = false;
		flag = true;
		var ErrMsg;
		var svAv;
		var spAv;
		var stop = 1;
		while (stop > 0){
			';
			for ($i=0 ; $i <= $inc; $i++){
				$svAv = "sv".$i;
				$spAv = "sp".$i;
				$verifScript .='	if((document.forms["venteform"].elements["'.$svAv.'"].value)*1>(document.forms["venteform"].elements["'.$spAv.'"].value)*1){
			flag_alert=true;
			break;
		}
	';
			}
	$verifScript .='
		stop = 1;
		}
		if(flag_alert == true){
			flag = false;
			ErrMsg = "Les ventes ne peuvent pas être sauvegardées.";
			alert(ErrMsg);
		}
 
		return flag;
	}
	</script>
	';

Un extrait du code généré :

Code:

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
 
function checkscript(){
		flag_alert = false;
		flag = true;
		var ErrMsg;
		var svAv;
		var spAv;
		var stop = 1;
		while (stop > 0){
				if((document.forms["venteform"].elements["sv0"].value)*1>(document.forms["venteform"].elements["sp0"].value)*1){
			flag_alert=true;
			break;
		}
		if((document.forms["venteform"].elements["sv1"].value)*1>(document.forms["venteform"].elements["sp1"].value)*1){
			flag_alert=true;
			break;
		}
		if((document.forms["venteform"].elements["sv2"].value)*1>(document.forms["venteform"].elements["sp2"].value)*1){
			flag_alert=true;
			break;
		}
		if((document.forms["venteform"].elements["sv3"].value)*1>(document.forms["venteform"].elements["sp3"].value)*1){
			flag_alert=true;
			break;
		}
                ...
                stop = 1;
           }
          if(flag_alert == true){
			flag = false;
			ErrMsg = "Les ventes ne peuvent pas être sauvegardées.";
			alert(ErrMsg);
		}
 
		return flag;
	}

C'est pas beau mais ça fonctionne.

11/06/2008, 09h20
E.Bzz

Citation:

Envoyé par PKO06

Sans les quotes cela revient au même ... rien :(

C'est peut-être que ton code HTML ne correspond pas à ce que laisse supposer ton code JS ....

A+
11/06/2008, 13h02
PKO06

Effectivement.
Je colle un petit morceau de mon code.
C'est un tableau ... c'est très laid étant donnée que mon php me génère un tableau d'environ 35*3 sur une ligne :oops:

Code:

<tr><td>ADAM</td><input type="hidden" name="cp3" value="161-10"><td><select name="sp3"><option value="0">0</option><option selected value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option><option value="11">11</option><option value="12">12</option><option value="13">13</option><option value="14">14</option><option value="15">15</option><option value="16">16</option><option value="17">17</option><option value="18">18</option><option value="19">19</option><option value="20">20</option><option value="21">21</option><option value="22">22</option><option value="23">23</option><option value="24">24</option><option value="25">25</option><option value="26">26</option><option value="27">27</option><option value="28">28</option><option value="29">29</option><option value="30">30</option><option value="31">31</option><option value="32">32</option><option value="33">33</option><option value="34">34</option><option value="35">35</option><option value="36">36</option><option value="37">37</option><option value="38">38</option><option value="39">39</option></select><select name="sv3"><option selected value="0">0</option></select> com : 0  Montant vente : 0</td><input type="hidden" name="cp4" value="161-11"><td>...
11/06/2008, 13h26
E.Bzz

Tu n'as bien qu'un seul "cp1", un seul "cp2" etc.
Ou bien un par ligne du tableau ?

A+
11/06/2008, 14h13
PKO06

Je n'en ai qu'un seul pour chaque cp, sp, et sv.
Je les ai numéroté pour pouvoir les récupérer après un POST.
11/06/2008, 14h24
E.Bzz

Alors d'après ton code, avec la dernière correction donnée par Matthieu2000, ça aurait dû être bon :?

Tu es sûr de n'avoir enlever les quottes qu'au bon endroit (par exemple pas celles de "venteform") ?

PS : tu es sûr aussi que la ligne générée n'est pas "trop longue" ?
y a une limite, je crois (mais laquelle ?) ....

A+
11/06/2008, 15h05
PKO06

Merci pour l'aide ça fait plaisir.
Pour la ligne a priori pas de problème puisque j'arrive à sauvegarder toutes les valeurs dans la base.
Je vais réessayer pour m'assurer de ne pas avoir laissé passer une faute d'inattention (de plus).

Je vais commencer à vous lasser. Mes débuts en Javascript sont laborieux.
Mon code si vous aimez trouver les erreurs, je me contenterais de la solution alternative postée plus haut.

Le code généré :

Code:

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
function checkscript(){
	flag_alert = false;
	flag = true;
	var ErrMsg;
	var counter;
	var svAv;
	var spAv;		
 
	for (counter = 0; counter <= document.forms["venteform"].elements["val_de_inc"].value*1; counter++){
		svAv = "sv"+counter;
		spAv = "sp"+counter;
		if(document.forms["venteform"].elements[svAv].value*1>document.forms["venteform"].elements[spAv].value*1){
			flag_alert=true;
			break;
		}
	}
 
	if(flag_alert == true){
		flag = false;
		ErrMsg = "Les ventes ne peuvent pas être sauvegardés. La quantité vendue est supérieure au stock.";
		alert(ErrMsg);
	}
 
	return flag;
}

11/06/2008, 16h15
E.Bzz

Je crains qu'il ne faille aussi le code HTML généré (même long) :aie:

A+
11/06/2008, 16h34
PKO06

J'envoie carrément la page.
:oops: je penserais à faire mes <select> en AJAX un jour.

http://dl.free.fr/mSnSZxWyC/pageHtml.rar

Citation:

Envoyé par PKO06

Code:

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
function checkscript(){
	flag_alert = false;
	flag = true;
	var ErrMsg;
	var counter;
	var svAv;
	var spAv;		
 
	for (counter = 0; counter <= document.forms["venteform"].elements["val_de_inc"].value*1; counter++){
		svAv = "sv"+counter;
		spAv = "sp"+counter;
		if(document.forms["venteform"].elements[svAv].value*1>document.forms["venteform"].elements[spAv].value*1){
			flag_alert=true;
			break;
		}
	}
 
	if(flag_alert == true){
		flag = false;
		ErrMsg = "Les ventes ne peuvent pas être sauvegardés. La quantité vendue est supérieure au stock.";
		alert(ErrMsg);
	}
 
	return flag;
}

semble marcher sous ie !lol

11/06/2008, 17h44
PKO06

... je t'ai envoyé la page avec l'autre script (généré en php pour chaque champs).
J'ai quand même testé sous ie avec celui que tu as cité ... pas de résultat non plus.

Comme t'as du le voir c'est une appli interne, utilisé seulement avec ff.
J'abandonne pour le moment, le principal c'est que ça fonctionne.

@+