Discussion :

[zozoman]

Bonjour à tous,

j'ai créé une fonction javascript pour vérifier la pertinence de mes champs. Les champs se colorent en rouge s'ils sont erronés mais la couleur disparait au bout d'une seconde et le message d'alerte ne s'affiche pas

fonction javascript :

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
// Vérification de la pertinence des champs 
function verifierCreerFAJ()
{
    var erreur=0;
    for(var i=0;i<10;i++)
    {
        var inputNomSel=[];
        var inputFajSel=[];
        inputNomSel[i]=document.forms["formCreerFAJ"].elements["substance["+i+"]"].value;
        inputFajSel[i]=document.forms["formCreerFAJ"].elements["faj["+i+"]"].value;
 
        if( (inputNomSel[i]==="" && inputFajSel[i]!=="") || (inputNomSel[i]==="Substance "+i && inputFajSel[i]!=="") )
        {
            document.forms["formCreerFAJ"].elements["substance["+i+"]"].style.backgroundColor = "#FF0000";
            erreur++;
        }
        else
        {
            inputNomSel.innerHTML=document.forms["formCreerFAJ"].elements["substance["+i+"]"].value;
            document.forms["formCreerFAJ"].elements["substance["+i+"]"].style.backgroundColor = "#FFFFFF";
        }
 
    }
 
    if(erreur===0)
    {
        return (true);
    }
    else
    {
        alert("Certains champs sont incomplets ou erronés");
        return (false);
    }
}

code HTML / php

Code php :

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
echo'<form id="formCreerFAJ" action="" method="POST" onsubmit="return verifierCreerFAJ();">'."\n";
            echo'   <table class="faj1">'."\n";
            echo'       <caption>Stock de sels de métaux précieux</caption>'."\n";
            echo'       <thead bgcolor="'.$tableau20['couleurFAJ'].'">'."\n";
            echo'           <tr>'."\n";
            echo'               <th rowspan="2" class="sel">Sel</th>'."\n";
            echo'               <th colspan="4">Pré-coffre (kg)</th>'."\n";
            echo'               <th colspan="4">Consigne (kg)</th>'."\n";
            echo'               <th colspan="4">Coffre (kg)</th>'."\n";
            echo'               <th rowspan="2">Total (kg)</th>'."\n";
            echo'           </tr>'."\n";
 
            echo'           <tr>'."\n";
            echo'               <th>J-1</th>'."\n";
            echo'               <th>Entrées</th>'."\n";
            echo'               <th>Sorties</th>'."\n";
            echo'               <th>Total</th>'."\n";
            echo'               <th>J-1</th>'."\n";
            echo'               <th>Entrées</th>'."\n";
            echo'               <th>Sorties</th>'."\n";
            echo'               <th>Total</th>'."\n";
            echo'               <th>J-1</th>'."\n";
            echo'               <th>Entrées</th>'."\n";
            echo'               <th>Sorties</th>'."\n";
            echo'               <th>Total</th>'."\n";
            echo'           </tr>'."\n";
            echo'       </thead>'."\n";
 
            echo'       <tbody bgcolor="'.$tableau20['couleurFAJ'].'">'."\n";
            $nbColonnes=13;
            for($i=0;$i<5;$i++)
            {
                echo'           <tr>'."\n";
                echo'               <td><input type="text" name="substance['.$i.']" id="substance['.$i.']" value="Substance '.$i.'" size="12" maxlength="20" />'."\n";
                echo'                   <select name="faj['.$i.']" id="faj['.$i.']">'."\n";
                echo'                       <option value=""></option>'."\n";
 
                $txtRequete11='SELECT * FROM substancefaj WHERE idFAJ="1" OR idFAJ="2" OR idFAJ="3" OR idFAJ="5" ORDER BY idFAJ';
                $requete11=mysql_query($txtRequete11);
 
                while($tableau11=mysql_fetch_assoc($requete11))
                {
                    echo'                       <option value="'.$tableau11['idFAJ'].'">'.$tableau11['compositionPrincipaleFAJ'].'</option>'."\n";
                }
                echo'                   </select>'."\n";
                echo'               </td>'."\n";
          /*******************************************************************************************/               
                for($j=0;$j<$nbColonnes;$j++)
                {
                    echo'               <td></td>'."\n"; 
                }
                echo'           </tr>'."\n";
            }
            echo'       </tbody>'."\n";
            echo'   </table>'."\n";
 
            // Création du formulaire de validation de faj1
            echo'   <table class="formSimple">'."\n";
            echo'       <tr>'."\n";
            echo'           <td><input type="submit" value="Valider" name="valider0" /></td>'."\n";
            echo'           <td><input type="reset" value="Effacer" name="effacer" /></td>'."\n";
            echo'       </tr>'."\n";
            echo'   </table>'."\n";
            echo'   <input type="hidden" name="idMetal" value="'.$_POST['idMetal'].'" />'."\n";
            echo'</form>'."\n";

Pourriez vous m'aider ?

Merci d'avance

[Bovino]

Merci de lire et appliquer : Important : Les règles incontournables d'utilisation de ce forum.
Le code JavaScript s'applique sur un document HTML. Le PHP n'est donc d'aucune utilité et rend la lecture plus compliquée.
Merci donc de poster le code HTML généré (celui visible en affichant la source depuis le navigateur) et non le PHP.

[SpaceFrog]

a vue de nez ça vient de tes elements['nom[indexe]']ça retourne une collection ...

le premier à pour indexe 0 ...

Code :

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

elements['nom[indexe]'][0]

[Bovino]

Ceci dit, mettre l'index dans le nom n'a aucun intérêt... Si ce n'est de compliquer les choses.

D'autre part, il y a d'autres erreurs de conception, notamment la redéfinition de tableaux de valeurs à chaque itération, tableaux qui ne servent d'ailleurs à rien. On te l'avais déjà indiqué il me semble.

[zozoman]

J'ai ajouté un indice de tableau mais le javascript ne fonctionne toujours pas. D'ailleurs, pourquoi ajouter un indice de tableau ?

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
function verifierCreerFAJ()
{
    var erreur=0;
    for(var i=0;i<10;i++)
    {
        var inputNomSel=[];
        var inputFajSel=[];
        inputNomSel[i]=document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].value;
        inputFajSel[i]=document.forms["formCreerFAJ"].elements["faj["+i+"]"][0].value;
 
        if( (inputNomSel[i]==="" && inputFajSel[i]!=="") || (inputNomSel[i]==="Substance "+i && inputFajSel[i]!=="") )
        {
            document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].style.backgroundColor = "#FF0000";
            erreur++;
        }
        else
        {
            inputNomSel.innerHTML=document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].value;
            document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].style.backgroundColor = "#FFFFFF";
        }
 
    }
 
    if(erreur===0)
    {
        return (true);
    }
    else
    {
        alert("Certains champs sont incomplets ou erronés");
        // erreur3.innerHTML="Certains champs sont incomplets ou erronés";
        return (false);
    }
}

Code html :

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
<form id="formCreerFAJ" action="" method="POST" onsubmit="return verifierCreerFAJ();">
   <table class="faj1">
       <caption>Stock de sels de métaux précieux</caption>
       <thead bgcolor="#DAA520">
           <tr>
               <th rowspan="2" class="sel">Sel</th>
               <th colspan="4">Pré-coffre (kg)</th>
               <th colspan="4">Consigne (kg)</th>
               <th colspan="4">Coffre (kg)</th>
               <th rowspan="2">Total (kg)</th>
           </tr>
           <tr>
               <th>J-1</th>
               <th>Entrées</th>
               <th>Sorties</th>
               <th>Total</th>
               <th>J-1</th>
               <th>Entrées</th>
               <th>Sorties</th>
               <th>Total</th>
               <th>J-1</th>
               <th>Entrées</th>
               <th>Sorties</th>
               <th>Total</th>
           </tr>
       </thead>
       <tbody bgcolor="#DAA520">
           <tr>
               <td><input type="text" name="substance[0]" id="substance[0]" value="Substance 0" size="12" maxlength="20" />
                   <select name="faj[0]" id="faj[0]">
                       <option value=""></option>
                       <option value="1"></option>
                       <option value="2">Ag</option>
                       <option value="3">Au</option>
                       <option value="5">Pd/Ni</option>
                   </select>
               </td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
           </tr>
           <tr>
               <td><input type="text" name="substance[1]" id="substance[1]" value="Substance 1" size="12" maxlength="20" />
                   <select name="faj[1]" id="faj[1]">
                       <option value=""></option>
                       <option value="1"></option>
                       <option value="2">Ag</option>
                       <option value="3">Au</option>
                       <option value="5">Pd/Ni</option>
                   </select>
               </td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
           </tr>
           <tr>
               <td><input type="text" name="substance[2]" id="substance[2]" value="Substance 2" size="12" maxlength="20" />
                   <select name="faj[2]" id="faj[2]">
                       <option value=""></option>
                       <option value="1"></option>
                       <option value="2">Ag</option>
                       <option value="3">Au</option>
                       <option value="5">Pd/Ni</option>
                   </select>
               </td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
           </tr>
           <tr>
               <td><input type="text" name="substance[3]" id="substance[3]" value="Substance 3" size="12" maxlength="20" />
                   <select name="faj[3]" id="faj[3]">
                       <option value=""></option>
                       <option value="1"></option>
                       <option value="2">Ag</option>
                       <option value="3">Au</option>
                       <option value="5">Pd/Ni</option>
                   </select>
               </td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
           </tr>
           <tr>
               <td><input type="text" name="substance[4]" id="substance[4]" value="Substance 4" size="12" maxlength="20" />
                   <select name="faj[4]" id="faj[4]">
                       <option value=""></option>
                       <option value="1"></option>
                       <option value="2">Ag</option>
                       <option value="3">Au</option>
                       <option value="5">Pd/Ni</option>
                   </select>
               </td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
               <td></td>
           </tr>
       </tbody>
   </table>
   <table class="formSimple">
       <tr>
           <td><input type="submit" value="Valider" name="valider0" /></td>
           <td><input type="reset" value="Effacer" name="effacer" /></td>
       </tr>
   </table>
   <input type="hidden" name="idMetal" value="3" />

[andry.aime]

D'autre part, il y a d'autres erreurs de conception, notamment la redéfinition de tableaux de valeurs à chaque itération, tableaux qui ne servent d'ailleurs à rien. On te l'avais déjà indiqué il me semble.

J'ai ajouté un indice de tableau mais le javascript ne fonctionne toujours pas. D'ailleurs, pourquoi ajouter un indice de tableau ?

Code :

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

1
2
3
4
5
6
7
function verifierCreerFAJ()
{
    var erreur=0;
    for(var i=0;i<10;i++)
    {
        var inputNomSel=[];
        var inputFajSel=[];

Code :

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

1
2
3
4
inputNomSel[i]=document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].value;
..................
inputNomSel.innerHTML=document.forms["formCreerFAJ"].elements["substance["+i+"]"][0].value;
        }

Un array n'as pas d'attribut innerHTML,
...