Utilisation de Switch a la place de if else if else (ArduinoRésolu/Fermé

Question

Bonjour, 

Les exemples que je trouve pour switch sont souvent comparés à des tests d'égalités (== 1, == 2 etc.)

 Comment utiliser des >= ou <=, pour définir des plages de valeurs, comme dans une gamme de notes de musique (par exemple, la note "Si" correspond à la plage allant de 495 à 527) ?
 Comment transformer le code ci-dessous avec un switch ?

if (sensorValue >= 528) {
    // note = Do4         Frequence = 528;
    note = "Do 4";
} else if (sensorValue >= 495) {
    // note = Si            Frequence = 495;
    note = "Si";
} // etc...

Merci de votre aide
a++

Whismeril · Answer

Bonjour

Les exemples c’est bien.
La doc aussi

https://docs.microsoft.com/fr-fr/cpp/c-language/switch-statement-c?view=msvc-170

En C, le case est une valeur unique, constante (ça ne peut pas être une variable) et de type integral (entier, caractère…)

Drum22 · Answer

Oui ! D'après cette documentation, j'ai bien l'impression qu'il s agisse toujours de tests de "pure" égalite !! (ou de progression de 1 en 1 ! on n'est pas rendu ...)
donc, l ordre switch ne répond sans doute pas à ma problématique ?
Je cherchais l'équivalent comme dans d autres langages, il y avait CASEQ, CASLT, CASGT par exemple...
Sinon, je ne suis pas abonné au switch, si une autre instruction fait le job, je suis preneur. ;-)
Merci de vos retours
a++

mamiemando · Answer

Bonjour, C'est simple, tu ne peux pas, à moins d'énumérer toutes les valeurs de ta plage :-) De plus un switch nécessite que la valeur testée soit un type entier (e.g. char, int, uint, ...), donc si ton code doit marcher avec un float ou un double, tu ne peux pas t'en sortir avec un switch. En fait c'est assez normal, c'est inhérent à la manière dont switch fonctionne : c'est relativement proche de la notion de labels et de goto. Note enfin que dans la version if ... else if ... il faut être vigilant à l'ordre dans lequel tu fais tes tests si tu ne contrôle que l'une des deux bornes de chaque plage de valeurs. Après, une manière plus élégante d'écrire ton code qui associe à chaque intervalle la donnée qui nous intéresse un arbre d'intervalles, mais comme ici les intervalles sont disjoints, on peut directement utiliser un arbre binaire de recherche. Comme aucune de ces structure n'est implémentée directement en C, et vu qu'il y a peu de notes, tu peux te contenter d'une structure moins efficace mais plus simple à coder, par exemple un bon vieux tableau. Ton code pourrait du coup ressembler à ceci : #include typedef struct _note_t { unsigned short min; unsigned short max; const char * name; } note_t; static const note_t NOTES[] = { {495, 528, "Do4"}, {470, 495, "Si3"}, // ... }; const char * find_note(double freq) { const note_t * note = &(NOTES[0]); unsigned num_notes = sizeof(notes) / sizeof(note_t); for (unsigned i = 0; i < num_notes; i++, note++) { if (note->min <= freq && freq < note->max) { return note->name; } } return NULL; } int main(){ double freqs[] = {400, 470, 475, 495, 500, 530}; unsigned num_freqs = sizeof(freqs) / sizeof(double); const double * pfreq = &freqs[0]; for (unsigned i = 0; i < num_freqs; i++, pfreq++) { double freq = *pfreq; printf("find_note(%lf) = %s ", freq, find_note(freq)); } return 0; } ... ce qui donne : (mando@silk) (~) $ gcc notes.c && ./a.out find_note(400.000000) = (null)find_note(470.000000) = Si3find_note(475.000000) = Si3find_note(495.000000) = Do4find_note(500.000000) = Do4find_note(530.000000) = (null) Bonne chance

georges97 · Answer

Bonjour,

Sauf erreur de ma part, l'exemple suivant permet de tester une condition correspondant à un intervalle et de la traiter dans un switch:

https://stackoverflow.com/questions/45645087/if-else-and-string-in-a-switch-case

voir également:

https://www.locoduino.org/spip.php?article23

Ce serait bien de revenir nous dire si cela convient ou si je me suis complètement fourvoyé concernant votre demande.

Drum22 · Answer

Merci à tous pour vos réponses. Je pensais "simplifier" la structure de mes tests (fort heureusement indentés !), mais le code propose reste assez  "complexe" pour quelqu'un comme moi qui aborde tout juste le C++ pour Arduino.  Je "lis" via une cellule photosensible pour "'retranscrire" sur un buzzer. Je vais donc conserver mon ancien code mais vous remercie encore pour vos idées de code.

Prenez soin de vous

Drum22 · Answer

Pardon en cliquant sur la rubrique, j ai vu "C" ... je me suis dit que ca allait pile poil ! Un grand merci à tous les intervenants, en tous cas, ça m'a bien aidé ! J ai "quand même" poussé, non pas la chansonnette, mais le code pour les tableaux.
Ça simplifie vachement le code (et le nombre de lignes !!) ;-)

Encore merci

    // Recherche dans le tableau [0,7] (frequence & nom de la note)
    for (int i = 0 ; i < Tab_Max; ++i)
     {
        // Si la recherche n a pas abouti, on compare
        if ((sensorValue >= Tab_Notes[i]) && (Trouve == 0))  
         {
            Frequence = Tab_Notes[i];
            Note      = Lib_Notes[i];
            Trouve    = 1;
         }
     }
    
    // Si Trouve = 0... (pas abouti)
    // la frequence est plus basse que Do3 ( <264 ), 
    // forcer a Do3 inferieur
    if (Trouve == 0)
    {
        Frequence = Tab_Notes[Tab_Max] - 64;    // frequence = 200
        Note      = "en dessous";
    }

Utilisation de Switch a la place de if else if else (Arduino

6 réponses

Newsletters