retour de resultat de fonction vers opener

Version imprimable

Bonjour,

je souhaiterais que ma fonction retourne une valeur vers une fenêtre parent (donc depuis un popup) lorsque le gentil utilisateur clique sur le bouton soumettre (il s'agit d'un formulaire).

Voici ce que j'ai fait:

la fonction:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<script type="text/javascript">
        function calculScore()
        {
                var scoreTotal;
                //calculer les sous-scores
                        //Physical function
                        var physicalFunctionScore = 10;
                        //pour chaque sous-question (a,b,etc.), regarder quelle case est cochee.
                        //si differents de une seule case cochee, afficher signal d'erreur.
                        if (document.Question3.aa3.checked==true){physicalFunctionScore += 1;}
                        else if (document.Question3.ba3.checked==true){physicalFunctionScore += 2;}
                        else if (document.Question3.ba3.checked==true){physicalFunctionScore += 3;}
                        else {document.write("error, check your answers in question 3.a");
                //additionner les sous-scores pour produire le score final
                return physicalFunctionScore;
        }
        //var score=45
        </script>

Le bouton:

Code:

<input type="button" onClick="window.opener.document.getElementById('scoreSF_36').value=calculScore()" value="Soumettre">

Je ne vois pas pourquoi ceci ne fonctionne pas... toute réponse est la bienvenue!!

Merci,

Stef

25/09/2010, 08h55
javatwister

parenthèse du document.write non fermée (et d'ailleurs, hein, tu sais ce qu'on en pense des document.write...)
25/09/2010, 21h12
Gualdim

Bien vu, il manque l'accolade en fait :) Je ne sais pas si ça règle mes problèmes par contre, je n'ai pas ma machine avec moi...

Non, que pense-t-on des document.write?
25/09/2010, 22h27
Auteur

bonjour,

document.write() remplace tout le code de la page par son argument lors de son appel.

Citation:

Non, que pense-t-on des document.write?

fais une recherche sur le forum avec ce mot clef :mrgreen:
26/09/2010, 13h17
Bovino

Citation:

Envoyé par Auteur

document.write() remplace tout le code de la page par son argument lors de son appel.

Plus précisément, le comportement de document.write() dépend de l'état du document.
Si celui-ci est ouvert (en gros, la page est en cours de création), alors document.write() insère le contenu de ses arguments à l'endroit où il est appelé.
Si le document est fermé (la page a fini de se charger), alors document.write() réouvre le document, insère le contenu de ses arguments puis referme le document, ce qui a pour effet concret de remplacer ce que contenait le document par le nouveau contenu comme l'indique Auteur.

Enfin, le principal autre reproche que l'on peut faire à document.write() est d'aller à l'encontre du principe de séparation des couches : (X)HTML pour le contenu, CSS pour la présentation et JavaScript pour le comportement.
26/09/2010, 13h21
SpaceFrog

moi aussi je me mets dans le "on" qui dit que document.write c'est tout null :mrgreen:

OK merci pour les explications :ccool:

Bon, d'abord désolé pour le manque d'accent: j'ai un clavier US, et ce matin après un update de Windows et pour un raison obscure, je ne peux pas switcher.

Ensuite, j'ai réalisé ce matin que je n'avais pas besoin de mon projet pour tester le couple popup-parent, voici ce que j'ai fait:
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 <script type="text/javascript"> function calculScore() { var scoreTotal; //calculer les sous-scores //Physical function var physicalFunctionScore = 10; //pour chaque sous-question (a,b,etc.), regarder quelle case est cochée. //si différents d'ne seule case cochée, afficher signal d'erreur. if (document.Question3.aa3.checked==true){physicalFunctionScore += 1;} else if (document.Question3.ba3.checked==true){physicalFunctionScore += 2;} else if (document.Question3.ca3.checked==true){physicalFunctionScore += 3;} //additionner les sous-scores pour produire le score final return physicalFunctionScore; } //var score=45 </script> <center> <input type="button" onClick="window.opener.document.getElementById('scoreSF_36').value=calculScore()" value="Soumettre"> <input type="button" onClick="window.opener.document.getElementById('scoreSF_36').value=null" value="Réinitialiser"> </center> <a href="#" onclick="window.close();">Fermer</a> <input type="text" id="scoreSF_36" /> <INPUT type="button" value="New Window!" onClick="window.open('SF36GOOD.html','mywindow','width=400,height=200')"> </body> </html>
Ça fonctionne, il semble donc que le problème était effectivement l'accolade... remarquez: document.write a disparu :lol:

Tiens j'y pense en postant mon code: quelqu'un saurait comment, avec Blogger, poster du code et faire en sorte que l'indentation soit respectée? Désolé ça n'a un peu rien avoir... :P

Bonjour,

je me demandais ce qui serait à utiliser à la place du document.write, si je veux signifier à l'utilisateur qu'il a fait une erreur à telle ou telle ligne?

Voici ma fonction Javascript pour le moment:

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
<script type="text/javascript">
	function calculScore()
	{
		var scoreTotal;
		//calculer les sous-scores
		//Physical function
		var physicalFunctionScore = 0;
		//pour chaque sous-question (a,b,etc.), regarder quelle case est cochee.
		//si differents de une seule case cochee, afficher signal d'erreur.
		if (document.Question3.aa3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.ba3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.ca3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ab3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bb3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cb3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ac3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bc3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cc3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ad3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bd3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cd3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ae3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.be3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.ce3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.af3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bf3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cf3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ag3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bg3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cg3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ah3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bh3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.ch3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.ai3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bi3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.ci3.checked==true){physicalFunctionScore += 3;}
 
		if (document.Question3.aj3.checked==true){physicalFunctionScore += 1;}
		else if (document.Question3.bj3.checked==true){physicalFunctionScore += 2;}
		else if (document.Question3.cj3.checked==true){physicalFunctionScore += 3;}
		//additionner les sous-scores pour produire le score final
		return physicalFunctionScore;
	}

L'utilisateur ne doit avoir la possibilité de cocher qu'une seule case par question, je ne sais pas encore vraiment comment implementer ceci.

La chose qui me dérange le plus dans ce code est sa longueur, est-ce que d'après vous il y aurait moyen de factoriser un peu?

Merci ;)

EDIT: J'ai pensé à utiliser alert(), mais pour peu que l'utilisateur soit... hum... un peu bourré, et suivant le nombre d'erreurs c'est autant d'alert qui vont s'ouvrir... c'est pas super propre...