Modification non demandée ^^

**47-47** · 11/09/2010, 22h23

Bonjour tout le monde!

Je suis actuellement sur un code ou il me faut un petit peu de javascript... Bien qu'appréciant moyennement le javascript, je m'efforce d'en faire proprement... si possible...

Mais voilà, je me suis fait un petit code qui me modifie la couleur du texte lorsque l'on clique sur une checkbox... Alors cet élément fonctionne, mais l'élément imprévu, c'est que ca me modifie le fond de la div... (ayant un fond "aléatoire", mais de couleurs unies, tout devient de la meme couleurs alors que ca ne devrait pas... ^^)

voici mon code, si vous pourriez m'aider... ^^

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
 
function mafonction(base,a,b)
{
	for(var i=0;i<b;i++)
	{
		if(document.getElementById(base+a).checked == true)
		{
			if(i==a)
				document.getElementById('resultat'+base+i).style="color: #009900;";
			else
				document.getElementById('resultat'+base+i).style="color: #000000;";
 
		}
		else
			document.getElementById('resultat'+base+i).style="color: #000000;";
	}
}

Et l'appel se fait ainsi:

Code :

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

1
2
 
<label for="_mini_0" name="Nom_Bidon">Nom_Bidon:<div id="resultat_mini_0" style="width:100%;" class="ma_classe">6</div></label><input type="checkbox" name="check[]" value="0" id="_mini_0" class="une_classe" onClick="javascript:mafonction('_mini_',0,8);" />

Bon, certaines parties vous paraitront peut-être inutile... mais elles ne se sont surement pas... ^^ Le seul problème que j'ai, vient vraiment du fait que ca ne change pas uniquement ce que je demande... ^^

**SpaceFrog** · 12/09/2010, 09h51

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
function mafonction(base,a,b)
{
    for(var i=0;i<b;i++)
    {
        if(document.getElementById(base+a).checked)
        {
            if(i==a)
                document.getElementById('resultat'+base+i).style.color=(i==a)?"#009900":"#000000";
 
        }
        else
            document.getElementById('resultat'+base+i).style.color="#000000";
    }
}
</script>
</head>
 
<body>
<label for="_mini_0" name="Nom_Bidon">Nom_Bidon:<label><div id="resultat_mini_0" style="width:100%;" class="ma_classe">6</div>
<input type="checkbox" name="check[]" value="0" id="_mini_0" class="une_classe" onclick="mafonction('_mini_',0,8);" />

**47-47** · 12/09/2010, 11h46

Merci pour ta réponse! Ca marche niquel!

Au fait, je pose juste la question, car je suis pas adepte des variantes du if "raccourci"... Mais dans le premier if, lorsqu'il y a:

Code :

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

1
2
 if(i==a)
                document.getElementById('resultat'+base+i).style.color=(i==a)?"#009900":"#000000";

Est-ce que le double appel de la condition est nécessaire?!

Code :

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

if(i==a) et (i==a)?...

**dukej** · 12/09/2010, 13h07

Envoyé par 47-47

Merci pour ta réponse! Ca marche niquel!

Au fait, je pose juste la question, car je suis pas adepte des variantes du if "raccourci"... Mais dans le premier if, lorsqu'il y a:

Code :

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

1
2
 if(i==a)
                document.getElementById('resultat'+base+i).style.color=(i==a)?"#009900":"#000000";

Est-ce que le double appel de la condition est nécessaire?!

Code :

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

if(i==a) et (i==a)?...

spacefrog s'est trompé il a laissé le if(i==a) tu peux l'enlever

**SpaceFrog** · 12/09/2010, 17h38

oui désolé erreur de copier coller

Code :

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

document.getElementById('resultat'+base+i).style.color=(i==a)?"#009900":"#000000";

est en fait le raccourci de

Code :

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

1
2
3
4
5
6
7
 
if(i==a){
   document.getElementById('resultat'+base+i).style.color="#009900";
   }
esle{
   document.getElementById('resultat'+base+i).style.color="#000000";
   }

**47-47** · 12/09/2010, 18h59

tout à fait, mais n'étant pas habitué à ce raccourci, je pose la question afin d'être sur que j'avais bien compris la chose...

Ceci dit, c'est tout bête l'erreur que j'avais faite... passage du color en argument du style au lieu de le mettre comme élément... ^^ Et résultat des courses, ca change tout!

En tout cas, mille fois merci pour cette correction, j'y avais pas pensé!

Modification non demandée ^^

JavaScript

Discussions similaires

Partager

Partager