Script qui ne s'execute pas comme voulu

Salut tout le monde !

Voici le 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
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
function CalculateMaskClass()
{
    var Guerrier = document.getElementById('Guerrier');
    var Paladin = document.getElementById('Paladin');
    var Chasseur = document.getElementById('Chasseur');
    var Voleur = document.getElementById('Voleur');
    var Pretre = document.getElementById('Pretre');
    var DK = document.getElementById('DK');
    var Chaman = document.getElementById('Chaman');
    var Mage = document.getElementById('Mage');
    var Demoniste = document.getElementById('Demoniste');
    var Druide = document.getElementById('Druide');
 
    var MGuerrier = 1;
    var MPaladin = 2;
    var MChasseur = 4;
    var MVoleur = 8;
    var MPretre = 16;
    var MDK = 32;
    var MChaman = 64;
    var MMage = 128;
    var MDemoniste = 256;
    var MDruide = 1024;
 
    var Mask = 0;
 
    if(Guerrier.getAttribute('checked'))
        { Mask = Mask + MGuerrier; }
    if(Paladin.getAttribute('checked'))
        { Mask = Mask + MPaladin; }
    if(Chasseur.getAttribute('checked'))
        { Mask = Mask + MChasseur; }
    if(Voleur.getAttribute('checked'))
        { Mask = Mask + MVoleur; }
    if(Pretre.getAttribute('checked'))
        { Mask = Mask + MPretre; }
    if(DK.getAttribute('checked'))
        { Mask = Mask + MDK; }
    if(Chaman.getAttribute('checked'))
        { Mask = Mask + MChaman; }
    if(Mage.getAttribute('checked'))
        { Mask = Mask + MMage; }
    if(Demoniste.getAttribute('checked'))
        { Mask = Mask + MDemoniste; }
    if(Druide.getAttribute('checked'))
        { Mask = Mask + MDruide; }
 
    var Final = document.getElementById('AllowableClass');
    Final.setAttribute('value', Mask);
 
    }

Et voici l'appel :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<label for="AllowableClass">Mask :</label> <input type="text" name="AllowableClass" id="AllowableClass" value="-1" /> <b><a onclick="CalculateMaskClass()">Calculer Mask</a></b><br/>
<br/>
<label for="Guerrier">Gerrier :</label> <input class="CheckBox" type="checkbox" id="Guerrier" checked="false" /><br/>
<label for="Paladin">Paladin :</label> <input class="CheckBox" type="checkbox" id="Paladin" checked="false" /><br/>
<label for="Chasseur">Chasseur :</label> <input class="CheckBox" type="checkbox" id="Chasseur" checked="false" /><br/>
<label for="Voleur">Voleur :</label> <input class="CheckBox" type="checkbox" id="Voleur" checked="false" /><br/>
<label for="Pretre">Prêtre :</label> <input class="CheckBox" type="checkbox" id="Pretre" checked="false" /><br/>
<label for="DK">Chevalier de la Mort :</label> <input class="CheckBox" type="checkbox" id="DK" checked="false" /><br/>
<label for="Chaman">Chaman :</label> <input class="CheckBox" type="checkbox" id="Chaman" checked="false" /><br/>
<label for="Mage">Mage :</label> <input class="CheckBox" type="checkbox" id="Mage" checked="false" /><br/>
<label for="Demoniste">Démoniste :</label> <input class="CheckBox" type="checkbox" id="Demoniste" checked="false" /><br/>
<label for="Druide">Druide :</label> <input class="CheckBox" type="checkbox" id="Druide" checked="false" /><br/>

Le problème :

Quand je clique sur le lien, le code ne s'execute qu'une fois, et ne prend pas en compte le fait que les Checkbox soit checked ou non.

Une solution svp ? Merci d'avance.

Code:

if(Guerrier.getAttribute('checked'))

Cette condition est incomplète, tu récupères l'attribut mais ce n'est pas un booléen. Là tu récupères "false" sous forme de String, donc sans plus d'informations cette String est évaluée à true en booléen. Pour récupérer l'état de la checkbox sous forme de booléen, c'est la propriété checked :

Code:

if(Guerrier.checked)

Attention, tu commets un contresens sur tes input !

Code:

<input class="CheckBox" type="checkbox" id="Guerrier" checked="false" />

L'attribut HTML checked est un attribut de type booléen (d'ailleurs, sa seule valeur possible est supposée être checked).
Cela signifie que ce n'est pas la valeur de l'attribut qui va déterminer l'état de la checkbox, mais uniquement sa présence et ce quelle que soit sa valeur !
Donc dans ton cas, même si tu as mis "false", il sera considéré comme coché.

Merci beaucoup ! Je pensais que la première version donnait une réponse en bool mais non.

Merci beaucoup !