moins de lignes de code

Version imprimable

[RESOLU] moins de lignes de code

Bonjour , j'ai fait un petit script avec des radios boutons, mais il me semble que cela fait beaucoup de code pour faire cela.

Y a t'il moyen de racourcir le code et comment

voici mon code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
function change_couleur_police(radio_couleur_police)
{
if (radio_couleur_police[0].checked) { document.getElementById('textearea_affichage_texte').style.color="#ffffff";}
if (radio_couleur_police[1].checked) { document.getElementById('textearea_affichage_texte').style.color="#cccccc";}
if (radio_couleur_police[2].checked) { document.getElementById('textearea_affichage_texte').style.color="#666666";}
if (radio_couleur_police[3].checked) { document.getElementById('textearea_affichage_texte').style.color="#000000";}
if (radio_couleur_police[4].checked) { document.getElementById('textearea_affichage_texte').style.color="#0066ff";}
if (radio_couleur_police[5].checked) { document.getElementById('textearea_affichage_texte').style.color="#0000cc";}
if (radio_couleur_police[6].checked) { document.getElementById('textearea_affichage_texte').style.color="#000066";}
if (radio_couleur_police[7].checked) { document.getElementById('textearea_affichage_texte').style.color="#ffffcc";}
if (radio_couleur_police[8].checked) { document.getElementById('textearea_affichage_texte').style.color="#ffff00";}
if (radio_couleur_police[9].checked) { document.getElementById('textearea_affichage_texte').style.color="#ff9900";}
if (radio_couleur_police[10].checked) { document.getElementById('textearea_affichage_texte').style.color="#ff0000";}
if (radio_couleur_police[11].checked) { document.getElementById('textearea_affichage_texte').style.color="#cc0000";}
if (radio_couleur_police[12].checked) { document.getElementById('textearea_affichage_texte').style.color="#990066";}
if (radio_couleur_police[13].checked) { document.getElementById('textearea_affichage_texte').style.color="#993300";}
if (radio_couleur_police[14].checked) { document.getElementById('textearea_affichage_texte').style.color="#00cc00";}
if (radio_couleur_police[15].checked) { document.getElementById('textearea_affichage_texte').style.color="#009900";}
if (radio_couleur_police[16].checked) { document.getElementById('textearea_affichage_texte').style.color="#006600";}
if (radio_couleur_police[17].checked) { document.getElementById('textearea_affichage_texte').style.color="#003300";}
 
}

merci pour votre aide

Code:

1
2
3
4
5
6
7
function change_couleur_police(radio_couleur_police) 
{ 
for (i=0; i<=17; i++) {
var color = new Array("#ffffff", "#cccccc", "#666666", "#000000", "#0066ff", "#0000cc", "#000066", "#ffffcc", "#ffff00", "#ff9900", "#ff0000", "#cc0000", "#990066", "#993300", "#00cc00", "#006600", "#003300");
if (radio_couleur_police[i].checked) { document.getElementById('textearea_affichage_texte').style.color=color[i];} 
 }
}

Corrigez moi si le Array() n'est pas bon.

29/03/2005, 20h19
cedjy93

je dirais qu'il faut mettre la declaration du tableau avant la fonction, sinon on redeclare a chaque fois un nouveau tableau, ce qui est inutile

alternative:

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
52
53
54
55
<html> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
 
<title>...</title> 
</head> 
 
<body>
 
<form id="f" action="">
 
<p>
<textarea id="affichage" cols="50" rows="10"></textarea>
</p>
 
<div>
<select id="s">
<option value="#ffffff">#ffffff</option>
<option value="#cccccc">#cccccc</option>
<option value="#666666">#666666</option>
<option value="#000000">#000000</option>
<option value="#0066ff">#0066ff</option>
<option value="#0000cc">#0000cc</option>
<option value="#000066">#000066</option>
<option value="#ffffcc">#ffffcc</option>
<option value="#ffff00">#ffff00</option>
<option value="#ff9900">#ff9900</option>
<option value="#ff0000">#ff0000</option>
<option value="#cc0000">#cc0000</option>
</select>
 
<em>Choisissez une couleur</em>
</div>
 
</form>
 
 
 
 
<script type="text/javascript"> 
 
var s=document.getElementById('s');
var t=document.getElementById('affichage');
 
for(i=0;i<s.length;i++){
s.options[i].style.backgroundColor=s.options[i].value};
 
s.onchange=function(){t.style.color=s.value;t.focus()}
 
 
</script> 
 
 
</body> 
</html>