Discussion :

[kOrt3x]

Bonjour,

J'aimerai savoir s'il n'est pas possible de simplifier ces deux "fonctions" :

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
                    $("#boutonOk").click(function() { 
                        if ( !$("#Q5R3").val() && $("#Q5R2").attr("checked") ) {
                            window.alert("Merci de remplir le champ");
                            $("#Q5R3").focus();
                            return false;
                        }
                    });
 
                    $("#boutonOk").click(function() { 
                        if ( !$("#Q5R5").val() && $("#Q5R4").attr("checked") ) {
                            window.alert("Merci de remplir le champ");
                            $("#Q5R5").focus();
                            return false;
                        }
                    });

N'est-il pas possible de faire ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
                    $("#boutonOk").click(function() { 
                        if ( !$("#Q5R3").val() && $("#Q5R2").attr("checked") ) {
                            window.alert("Merci de remplir le champ");
                            $("#Q5R3").focus();
                            return false;
                        }
                        if ( !$("#Q5R5").val() && $("#Q5R4").attr("checked") ) {
                            window.alert("Merci de remplir le champ");
                            $("#Q5R5").focus();
                            return false;
                        }
                    });

[Piflo]

Tu as tester ton 2eme morceau de code?
je pense que sa devrai marcher.

Car j'aurais la même logique que toi. 2 $(#...).click(...); autant mettre les bout de code que d'en un. mais je peut pas t’affirmai que sa marchera.

Cependant dissocié en 2 fonction peut répondre a une 2eme logique: si tu doit retiré un champs et un check box correspondant, tu met en commentaire ou supprime la fonction qui fait référence a ces éléments de ta page.

[abrillant]

Le simplifier oui, suffirait pourquoi pas d'avoir une fonction avec les 2 champs en question

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function verifForm( pField1, pField2 ) {
 if ( !$(pField1).val() && $(pField2 ).attr("checked") ) {
       window.alert("Merci de remplir le champ");
       ( $pField1).focus();
       return false;
 }
 return true;
}
 
 $("#boutonOk").click(function() { 
    return verifForm( "#Q5R3", "#Q5R2" ) || verifForm( "#Q5R5", "#Q5R4" )
 }

Après c'est, est - ce que cela vaut le coup ? ?

[kOrt3x]

Comme je débute en jQuery, je trouvé très bizarre de ne pas pouvoir mettre plusieurs IF dans à la suite, alors qu'en PHP ça fonctionne.
Mais bon, après il y a peut être une explication, mais je ferai avec.

[Piflo]

Ce qui cloche peut être dans ta factorisation est que si le 1er test est concluant, avec le return false; le 2eme if n'est pas traiter.

[kOrt3x]

Ce qui cloche peut être dans ta factorisation est que si le 1er test est concluant, avec le return false; le 2eme if n'est pas traiter.

Effectivement, je viens de comprendre et de faire le test, mais comment on empêche de valider le formulaire dans ce cas là ?

Parce ce que là :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$("#boutonOk").click(function() { 
                        if ( !$("#Q1R1").attr('checked') && !$("#Q1R2").attr('checked') && !$("#Q1R3").attr('checked') ) { 
                            window.alert("Merci de répondre à la question 1");
                        }
 
                        if ( !$("#Q1R4").val() && $("#Q1R3").attr("checked") ) {
                            window.alert("Merci de remplir le champ");
                            $("#Q1R4").focus();
                        }
 
                        if ( !$("#Q4R1").attr('checked') && !$("#Q4R2").attr('checked') && !$("#Q4R3").attr('checked') && !$("#Q4R4").attr('checked') ) {
                            window.alert("Merci de répondre à la question 4");
                        }
                    });

[kOrt3x]

Bon, j'ai trouvé une solution qui me semble correcte :

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
$("#boutonOk").click(function() { 
 
                        var valid = false;
 
                        if ( !$("#Q1R1").attr('checked') && !$("#Q1R2").attr('checked') && !$("#Q1R3").attr('checked') ) { 
                            window.alert("Merci de répondre à la question 1");
                        } 
                        else if ( !$("#Q1R4").val() && $("#Q1R3").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 1");
                            $("#Q1R4").focus();
                        } 
                        else if ( !$("#Q4R1").attr('checked') && !$("#Q4R2").attr('checked') && !$("#Q4R3").attr('checked') && !$("#Q4R4").attr('checked') ) {
                            window.alert("Merci de répondre à la question 4");
                        } 
 
                        else if ( !$("#Q5R3").val() && $("#Q5R2").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 5");
                            $("#Q5R3").focus();
                        } 
                        else if ( !$("#Q5R5").val() && $("#Q5R4").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 5");
                            $("#Q5R5").focus();
                        } 
                        else if ( !$("#Q7R1").attr('checked') && !$("#Q7R2").attr('checked') && !$("#Q7R3").attr('checked') && !$("#Q7R4").attr('checked') && !$("#Q7R5").attr('checked') ) { 
                            window.alert("Merci de répondre à la question 7");
                        } 
                        else if ( !$("#Q7R6").val() && $("#Q7R5").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 7");
                            $("#Q7R6").focus();
                        } 
                        else if ( !$("#Q8R1").attr('checked') && !$("#Q8R2").attr('checked') && !$("#Q8R3").attr('checked') && !$("#Q8R4").attr('checked') && !$("#Q8R5").attr('checked') && !$("#Q8R6").attr('checked') && !$("#Q8R7").attr('checked') && !$("#Q8R8").attr('checked')  && !$("#Q8R9").attr('checked')) { 
                            window.alert("Merci de répondre à la question 8");
                        } 
                        else if ( !$("#Q8R10").val() && $("#Q8R9").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 8");
                            $("#Q8R10").focus();
                        }
                        else if ( !$("#Q9R1").attr('checked') && !$("#Q9R2").attr('checked') && !$("#Q9R3").attr('checked') && !$("#Q9R4").attr('checked') ) { 
                            window.alert("Merci de répondre à la question 9");
                        } 
                        else if ( !$("#Q9R5").val() && $("#Q9R4").attr("checked") ) {
                            window.alert("Merci de remplir le champ de la question 9");
                            $("#Q9R5").focus();
                        } 
                        else {
                            valid = true;
                        }
 
                        return valid;
                    });