Discussion :

[Gualdim]

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 :

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
<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 :

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

<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

[javatwister]

parenthèse du document.write non fermée (et d'ailleurs, hein, tu sais ce qu'on en pense des document.write...)

[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?

[Auteur]

bonjour,

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

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

fais une recherche sur le forum avec ce mot clef

[Bovino]

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.

[SpaceFrog]

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

[Gualdim]

OK merci pour les explications

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 :

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
<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

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...

[Gualdim]

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 :

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
<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...