Discussion :

[notar]

bonjour,
je débute en js et je suis nul de chez nul.
dans un form, j'ai fait çà:

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
 
<form action="" method="get" name="bord" onsubmit="window.print()" enctype="" target="">
<td width="10%"><input type="checkbox" name="numero" value="108"><br />n&deg; 108</td>
<td width="10%"><input type="checkbox" name="numero" value="107"><br />n&deg; 107</td>
 
<td width="10%"><input type="checkbox" name="numero" value="106"><br />n&deg; 106</td>
<td width="10%"><input type="checkbox" name="numero" value="105"><br />n&deg; 105</td>
<td width="10%"><input type="checkbox" name="numero" value="104"><br />n&deg; 104</td>
<td width="10%"><input type="checkbox" name="numero" value="103"><br />n&deg; 103</td>
<td width="10%"><input type="checkbox" name="numero" value="102"><br />n&deg; 102</td>
 
<td width="10%"><input type="checkbox" name="numero" value="101"><br />n&deg; 101</td>
<td width="10%"><input type="checkbox" name="numero" value="100"><br />n&deg; 100</td>
<td width="10%"><input type="checkbox" name="numero" value="99"><br />n&deg; 99</td>
 
//********************** et plus loin
 
<li>Rédigez votre chèque d'un montant de : 
	<SCRIPT language="JavaScript">
		function calc(z){
		var tb=0;
		var i;
                    for(i=0;<z;i++){
                        if(document.getElementsByName("numero")[i].checked==true){
                        tb++;
                     }
                 }
document.write(tb);
	</script>
 € à l'ordre de .......

après je l'appelle par:

Code :

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

1
2
3
 
 
<input type="button" value="Convertir" OnClick="calc(<?php print($Nb);?>)"/>

et cela ne me rend aucun résultat.

pouvez-vous me dire pourquoi cela ne fonctionne pas
d'avance merci

[vermine]

Bonjour,

Avant de regarder plus loin, il manque un élément dans le for:

Code :

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

1
2
 for(i=0;???<z;i++){

[notar]

autant pour moi! c'est une erreur de retranscription; il faut lire

Code :

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

1
2
 
for(i=0;i<z;i++){

[vermine]

A quoi sert à la variable tb ? Elle aura la même valeur que i, me semble-t-il:

Code :

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

1
2
3
4
 
document.write(i);
//voire peut-être avec une incrémentation, à vérifier
document.write(++i);

Sinon, ça bloque peut-être au onclick lorsque vous appelez une variable PhP comme paramètre de la fonction. Paramètre qui d'ailleurs est peut-être interprété comme une chaine de caractères dans votre for et non comme un nombre. :o

[notar]

le parametre récupéré de Php est un nombre ce qui donne z= 44
i est l'indice de comptage pour le for il va de 0 à 43
tb est le nombre de valeurs true

si je fais tout en Php, tout fonctionne et là, je merde complètement.

[vermine]

Excusez-moi, j'ai sauté une ligne dans ma lecture du code.

Par contre, je me concentrerai sur la valeur de "z". Vous avez mis un alert en début de fonction pour voir la valeur ? Et que donne un alert de:

Code :

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

1
2
 
alert(i<z);

[notar]

si je mets

Code :

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

1
2
3
4
 
function calc(z){
alert(z);
....//....}

je n'ai pas d'alerte qui s'affiche.
je comprends rien.

[E.Bzz]

Bonjour,
tu as oublié les quottes autour du paramètre de calc() généré par PHP
Du coup il est considéré comme une variable. Et comme cette variable n'existe pas ....

A+

[notar]

excusez-moi; il manquait une accolade pour fermer la fonction.
alert() affiche bien 44
mais tb ne renvoie rien.
ma condition

Code :

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

1
2
3
4
 
if(document.getElementsByName("numero")[i].checked==true){
tb++;
}

ne fonctionne pas!!

[notar]

si je fais

Code :

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

1
2
3
4
5
6
7
 
for(i=0;i<z;i++){
															if(document.getElementsByName("numero")[i].checked==1){
															alert(i);
															tb = tb+1;
															}
	}

alors alert(i) me renvoie bien les indices des cases cochées mais tb ne s'incrémente pas; pourtant il est bien déclaré plus haut

Code :

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

1
2
3
4
5
6
 
function calc(z){
//alert('attention'+z);
var tb = 0;
var i;
.............//..............}

comment faire pour qu'il s'incrémente?
merci

[E.Bzz]

Tu as essayé de faire un alert() au lieu du document.write() ?
De toutes façons, il faudra modifier ce dernier ...

A+

[notar]

si je fais

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 calc(z){
//alert('attention'+z);
var tb = 0;
var i;
for(i=0;i<z;i++){
if(document.getElementsByName("numero")[i].checked==1){
//alert(i);
tb = tb+1;
//alert(tb);
}
}
return tb;
}
document.write(tb);

mes alertes me retournent bien les valeurs, aussi bien pour le i que pour le tb mais je ne peux pas renvoyer mon tb pour le mettre dans document.write(tb)
comment faut-il faire ?

[E.Bzz]

Déjà s'agissant d'une variable locale, tu ne peux l'utiliser que dans la fonction.
Mais encore une fois, ce n'est pas un document.write() qu'il faut utiliser : teste d'abord avec un alert() ...

A+

[notar]

ok, j'ai sorti la déclaration de ma variable 'tb' de la fonction; donc normalement elle est globale

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
 
<head>
<SCRIPT language="Javascript">
var tb;
function calc(z){
//alert('attention'+z);
tb = 0;
var i;
for(i=0;i<z;i++){
if(document.getElementsByName("numero")[i].checked==1){
//alert(i);
tb = tb+1;
//alert(tb);
}
}
return tb;
}
</script>
</head>

et dans le <body>

Code :

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

1
2
3
4
 
<script>
document.write(tb);
</script>

le résultat me donne: undefined

ou est-ce que j'ai m....dé?

[E.Bzz]

le résultat me donne: undefined

ou est-ce que j'ai m....dé?

Mais encore une fois, ce n'est pas un document.write() qu'il faut utiliser : teste d'abord avec un alert() ...

A+

[E.Bzz]

Pour être un peu plus complet :
En dehors de la fonction, ton document.write() va s'exécuter au chargement de la page. A ce moment-là, tb n'est pas encore défini. C'est le message que tu obtiens.
Il suffirait donc de la mettre dans la fonction ?
Non plus, car dans ce cas là, le contenu de ton document (= de ta page) sera remplacé par le write(). Pas glop non plus.

Il faut que tu prévoit un <span> pour récupérer la valeur de tb.
Ainsi en fin de ta fonction tu pourras faire

Code :

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

document.getElementById('id_du_span').innerHTML = tb;

A+

[notar]

si je fais

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 calc(z){
//alert('attention'+z);
var tb = 0;
var i;
for(i=0;i<z;i++){
if(document.getElementsByName("numero")[i].checked==1){
//alert(i);
tb = tb+1;
//alert(tb);
}
}
return tb;
}
document.write(tb);

mes alertes me retournent bien les valeurs, aussi bien pour le i que pour le tb mais je ne peux pas renvoyer mon tb pour le mettre dans document.write(tb)
comment faut-il faire ?

alert() à la place de document.write() me donne aussi undefined

alors çà je l'ai règlé en mettant dans

Code :

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

1
2
 
<script>if(tb<>"")alert(tb);</script>

mais çà ne me dit toujours pas comment je peux renvoyer ma variable dans le script d'affichage.
merci pour votre aide

[notar]

merci de m'avoir donné la réponse.
mais comme je disais

bonjour,
je débute en js et je suis nul de chez nul.
dans un form, j'ai fait çà:

[vermine]

On voulait s'avoir si tb avait la bonne valeur en sortant de la fonction. Apparement, oui. Donc je remettrai la variable en local comme le dit E.Bzz et irai mettre cette valeur dans un élément html de la page (exemple du span précité) Donc, au lieu de faire un:

Code :

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

1
2
 
return tb;

faites plutôt un:

Code :

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

1
2
 
document.getElementById('id_du_span').innerHTML = tb;

avec (mais ce n'est un exemple et il utilise innerHTML) un span:

Code :

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

1
2
 
<span id='id_du_span'></span> <span> € à l'ordre de .......</span>

[notar]

ben oui, du coup voila le code complet de la fonction

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
 
<SCRIPT language="Javascript">
var tb;
function calc(z,price){
		//alert('attention'+z);
		tb = 0;
		var abon1=0;
		var abon2=0;
		var don=0;
		var TotalAboDon = 0;
		var TotalBulletin = 0;
		var i;
	for(i=0;i<z;i++){
		if(document.getElementsByName("numero")[i].checked==1){
		//alert(i);
		tb = tb+1;
		//alert(tb);
		}
	}
	if(document.getElementsByName("abonn")[0].checked==1){
	abon1 = document.getElementsByName("abonn")[0].value;
	//alert(abon1);
	}
	if(document.getElementsByName("abonn")[1].checked==1){
	abon2 = document.getElementsByName("abonn")[1].value;
	//alert(abon2);
	}
 
	don = document.getElementById('autredon').value;
	var reg=new RegExp("(,)", "g");
	don= don.replace(reg,".");
	//alert('don'+don);
 
document.getElementById('Recap').innerHTML = '<h3>RECAPITULATIF</H3><table width="50%"><tr><td width="70%" align="center">Cotisations et dons</td><td align="right">'+(Number(abon1)+Number(abon2)+Number(don))+' €</td></tr><tr><td>Commande de '+tb+' bulletins à '+price+' €</td><td align="right">'+Number(tb*price)+' €</td></tr><tr><td align="right"><b>Total</b></td><td align="right"><b>'+(Number(tb*price)+Number(abon1)+Number(abon2)+Number(don))+' €</b></td></tr></table>';
 
	if(document.getElementById('txtprenom').value==""){
	alert('prenom n\'est pas renseigné!');
	document.getElementById('prenom').innerHTML ='<input type="text" style="background-color: #FFE4C4;" name="prenom" id="txtprenom" size=14 maxlenght=30>';
	exit();
	}
	if(document.getElementById('txtnom').value==""){
	alert('nom n\'est pas renseigné!');
	document.getElementById('nom').innerHTML ='<input type="text" style="background-color: #FFE4C4;" name="nom" id="txtnom" size=20 maxlenght=40>';
	exit();
	}
	if(document.getElementById('txtvoie').value==""){
	alert('voie n\'est pas renseignée!');
	document.getElementById('voie').innerHTML ='<input type="text" style="background-color: #FFE4C4;" name="voie" id="txtvoie" size=48 maxlength=60>';
	exit();
	}
	if(document.getElementById('txtcp').value==""){
	alert('code postal n\'est pas renseigné!');
	document.getElementById('cp').innerHTML ='<input type="text" style="background-color: #FFE4C4;" name="cp" id="txtcp" size=5 maxlength=5>';
	exit();
	}
	if(document.getElementById('txtville').value==""){
	alert('ville n\'est pas renseignée!');
	document.getElementById('ville').innerHTML ='<input type="text" style="background-color: #FFE4C4;" name="ville" id="txtville" size=48 maxlength=60>';
	exit();
	}
	if((document.getElementById('txttel').value=="") && (document.getElementById('txtadremail').value=="")){
	alert('Donnez nous au moins un numéro de téléphone ou une adresse mail pour que nous puissions vous contacter!');
	exit();
	}
 
document.getElementById('Result').innerHTML = (Number(tb*price)+Number(abon1)+Number(abon2)+Number(don));
setTimeout("window.print()",2000);
//window.print();
}
</script>

avec un

Code :

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

1
2
 
<div id="Recap" align="center"></div>

et un appel par

Code :

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

1
2
 
<input type="button" class="imprim" value="Imprimer" OnClick="calc(<?php print($Nb);?>,<?php print($ancien_Numero); ?>)"/>

si j'avais su d'entré de jeu qu'il fallait employer un

Code :

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

1
2
 
document.getElementById('Id_du_span').innerHTML

aulieu de me faire tester des:

Envoyé par E.Bzz Voir le message
Mais encore une fois, ce n'est pas un document.write() qu'il faut utiliser : teste d'abord avec un alert() ...

j'aurais compris tout aussi bien et j'aurais gagné pas mal de temps.

dans tous les cas, je vous remercie pour l'aide précieuse apportée.