Discussion :

[mormic]

Bonjour à tous

Mon JS reçoit en retour d'une requète un message "0" ou "1".
Pour appeler une fonction une variable doit être mise en forme:

Code :

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

1
2
3
4
5
            case 15:
                libel="xxx" + message;
                //alert(libel);
                chargeZonesdeControle(libel);
            break;

Si alert est en fonction j'obtiens bien : xxx1
La fonction:

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
function chargeZonesDeControle(zone){
    //alert(zone);
    const ch="Controle horaire ";
    const cc="Controle capteur ";
    const ci="Controle inters base ";
    const trp="transparent";
    const pg="palegoldenrod";
    if(zone.substring(0,1)=="1"){
        alConnex.textContent="TimeOut dans 5 mn";
        alConnex.style.backgroundColor=trp;
    }
    else if(zone.substring(0,1)=="0"){
        alConnex.textContent = "Connexion permanente";
        alConnex.style.backgroundColor=pg;
    }
    if(zone.substring(1,2)=="1"){
        alControlH.textContent= ch + "On";
        alControlH.style.backgroundColor="palegreen";
    }
    else if(zone.substring(1,2)=='0'){
        alControlH.textContent= ch + "Off";
        alControlH.style.backgroundColor=trp;
    }
    if(zone.substring(2,3)=='1'){
        alControlC.textContent = cc + "On";
        alControlC.style.backgroundColor=pg;
    }
    else if(zone.substring(2,3)=='0'){
        alControlC.textContent = cc + "Off";
        alControlC.style.backgroundColor=trp;
    }
    if(zone.substring(3,4)=='1'){
        alControlI.textContent= ci + "On";
        alControlI.style.backgroundColor="palegreen";
    }
    else if(zone.substring(3,4)=='0'){
        alControlI.textContent= ci + "Off";
        alControlI.style.backgroundColor=trp;
    }
}

La variable 'zone' ne reçoit jamais xxx1

Par contre si je remplace "xxx1" par "0001", cela fonctionne.

Y a-t'il une explication?

[mormic]

Désolé de vous avoir dérangé mais c'est tout simplement une erreur de frappe!

ma fonction s'appelle: chargeZonesDeControle

et je l'appelait par: chargeZonesdeControle avec un d minuscule

Un vrai débutant !

[gabi7756]

Bonjour,
Tout le monde est débutant par moment.
Je viens d'apprendre que faire des substring dans un if c'est plus rapide que d'initialiser une variable au préalable :

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
 
function measureTime() {
    const js = "thfshgispgh";  // La chaîne de caractères à tester
    const sub_js = js.substring(0, 5); // Pré-déclare une sous-chaîne
 
    // Test 1 : Déclarer la sous-chaîne dans une variable et utiliser dans le if
    console.time("Declared substring then if");
    for (let i = 0; i < 100000; i++) {
        if (sub_js  === "a") {
            // Ne rien faire, juste comparer
        }
        if (sub_js  === "b") {
            // Ne rien faire, juste comparer
        }
        if (sub_js === "c") {
            // Ne rien faire, juste comparer
        }
        if (sub_js === "thfsh") {
            // Ne rien faire, juste comparer
        }
    }
    console.timeEnd("Declared substring then if");
 
    // Test 2 : Faire directement le substring dans le if
    console.time("Substring in if");
    for (let i = 0; i < 100000; i++) {
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
    }
    console.timeEnd("Substring in if");
}
 
measureTime(); // Exécution de la fonction

Genre deux fois plus rapide
Comme quoi ...

[mathieu]

Je viens d'apprendre que faire des substring dans un if c'est plus rapide que d'initialiser une variable au préalable :

j'utilise la dernière version de firefox et j'ai le résultat inverse, le 1er test affiche "0 ms" et le 2e test affiche "6 ms".

en plus je pense que la clarté du code devrait être prioritaire sur l'optimisation donc je préfère le code du 1er test quelque soit le délai d'exécution.

[NoSmoking]

Bonjour,
@gabi7756 :
• tes tests ne sont pas équitables, il conviendrait d'avoir le même nombre de clause if, ton test #2 devrait donc plutôt être :

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
    // Test 2 : Faire directement le substring dans le if
    console.time("Substring in if");
    for (let i = 0; i < 100000; i++) {
/*    
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5) === "thfsh") {
            // Ne rien faire, juste comparer
        }
/*--*/
        if (js.substring(0, 5)   === "a") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5)   === "b") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5)  === "c") {
            // Ne rien faire, juste comparer
        }
        if (js.substring(0, 5)  === "thfsh") {
            // Ne rien faire, juste comparer
        }        
    }
    console.timeEnd("Substring in if");
}

• les résultats sont dépendant du navigateur, un coup je gagne un coup je perd !
• comme dit par mathieu je pense également que la clarté et la maintenance sont à privilégier,

Code :

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

const sub_js = js.substring(0, 5); // Pré-déclare une sous-chaîne

et dans ce sens plus « efficace » !

Ceci dit, c'est également sans compter des éventuelles optimisations des moteurs JavScript qui font de plus en plus de merveilles !