Optimiser un programme

Dryusdan, lundi 26 janvier 2015 à 17h50

Ami de smoothie bien le bonjour Je suis en train de développer sous arduino un système de niveau de batterie. Jusqu'à la, aucun soucis, n'ayant pas de matériel nécessaire, j'ai utilisé un potentiomètre et un barregraphe fait maison (car le miens n'est pas encore arrivé). Niveau matériel c'est bon, niveau code aussi, mais… j'ai 200 lignes de if pour allumer et éteindre tel ou tel LED… Je ne sais pas si il y a un moyen d'optimiser ceci (je vous file le code mais ne me filez pas les réponses, juste des indices )

// niveau_batterie_v1.ino

//Led: composant
//R/J/V: Rouge/jaune/vert
//1,2,3... numéro de la led parant de la rouge tout en bas a la verte

int LedR1 = 2;
int LedR2 = 3;
int LedR3 = 4;
int LedJ4 = 5;
int LedJ5 = 6;
int LedJ6 = 7;
int LedV7 = 8;
int LedV8 = 9;
int LedV9 = 10;
int LedV10 = 11;
int LedV11 = 12;
int LedV12 = 13;
int battery = 0;
int tension;
int valBattery;

void setup() {
    /*---------------INITIALISATION---------------*/
    pinMode(LedR1, OUTPUT);
    pinMode(LedR2, OUTPUT);
    pinMode(LedR3, OUTPUT);
    pinMode(LedJ4, OUTPUT);
    pinMode(LedJ5, OUTPUT);
    pinMode(LedJ6, OUTPUT);
    pinMode(LedV7, OUTPUT);
    pinMode(LedV8, OUTPUT);
    pinMode(LedV9, OUTPUT);
    pinMode(LedV10, OUTPUT);
    pinMode(LedV11, OUTPUT);
    pinMode(LedV12, OUTPUT);

}

void loop() {
    valBattery = analogRead(battery);
    tension = map(valBattery, 0, 1023, 0, 5000); // reglage pour avoir une tension en milivolt
    if(tension == 0)
    {
        digitalWrite(LedR1, LOW);
        digitalWrite(LedR2, LOW);
        digitalWrite(LedR3, LOW);
        digitalWrite(LedJ4, LOW);
        digitalWrite(LedJ5, LOW);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
    else if(tension >= 0 && tension <= 416)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, LOW);
        digitalWrite(LedR3, LOW);
        digitalWrite(LedJ4, LOW);
        digitalWrite(LedJ5, LOW);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 416 && tension <= 833)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, LOW);
        digitalWrite(LedJ4, LOW);
        digitalWrite(LedJ5, LOW);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 833 && tension <= 1250)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, LOW);
        digitalWrite(LedJ5, LOW);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 1250 && tension <= 1667)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, LOW);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 1667 && tension <= 2084)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, LOW);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 2084 && tension <= 2501)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, LOW);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 2501 && tension <= 2918)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, LOW);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 2918 && tension <= 3335)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, HIGH);
        digitalWrite(LedV9, LOW);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 3335 && tension <= 3752)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, HIGH);
        digitalWrite(LedV9, HIGH);
        digitalWrite(LedV10, LOW);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension > 3752 && tension <= 4169)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, HIGH);
        digitalWrite(LedV9, HIGH);
        digitalWrite(LedV10, HIGH);
        digitalWrite(LedV11, LOW);
        digitalWrite(LedV12, LOW);
    }
        else if(tension >= 4169 && tension <= 4586)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, HIGH);
        digitalWrite(LedV9, HIGH);
        digitalWrite(LedV10, HIGH);
        digitalWrite(LedV11, HIGH);
        digitalWrite(LedV12, LOW);
    }
        else if(tension >= 4586 && tension <= 5000)
    {
        digitalWrite(LedR1, HIGH);
        digitalWrite(LedR2, HIGH);
        digitalWrite(LedR3, HIGH);
        digitalWrite(LedJ4, HIGH);
        digitalWrite(LedJ5, HIGH);
        digitalWrite(LedJ6, HIGH);
        digitalWrite(LedV7, HIGH);
        digitalWrite(LedV8, HIGH);
        digitalWrite(LedV9, HIGH);
        digitalWrite(LedV10, HIGH);
        digitalWrite(LedV11, HIGH);
        digitalWrite(LedV12, HIGH);
    }
}

Merci d'avance

Cordialement Un adorateur de zeste de savoir (Dryusdan)

26/01/15 à 17h50

+0 -0

anonyme, lundi 26 janvier 2015 à 18h30
Modifié

Tu peux déjà remarquer qu'il y a des redondances dans ta cascade de if. Le mieux est que tu réfléchisse non pas par intervalle de tension, mais variable par variable pour les LedR1, LedR2, etc : quand est-ce qu'une variable vaut HIGH, et quand vaut-elle LOW ? Une fois que tu as la réponse, tu pourra repenser ton code et virer ta cascade de if, en utilisant soit des ternaires, soit des variables temporaires pour chaque bit à envoyer sur une pin.

Je mets une proposition de code en balise secret qui devrait marcher. Si j'ai pas fait de conneries, tu peux descendre à 18 lignes de code pour la fonction loop.

void loop
{
    valBattery = analogRead(battery);
    tension = map(valBattery, 0, 1023, 0, 5000);

    digitalWrite(LedR1, (tension >= 417) ? HIGH : LOW) ;
    digitalWrite(LedR2, (tension >= 833) ? HIGH : LOW) ;
    digitalWrite(LedR3, (tension >= 1250) ? HIGH : LOW) ;
    digitalWrite(LedR4, (tension >= 1667) ? HIGH : LOW) ;
    digitalWrite(LedR5, (tension >= 2084) ? HIGH : LOW) ;
    digitalWrite(LedR6, (tension >= 2501) ? HIGH : LOW) ;
    digitalWrite(LedR7, (tension >= 2918) ? HIGH : LOW) ;
    digitalWrite(LedR8, (tension >= 3752) ? HIGH : LOW) ;
    digitalWrite(LedR9, (tension >= 4169) ? HIGH : LOW) ;
    digitalWrite(LedR10, (tension >= 4586) ? HIGH : LOW) ;
    digitalWrite(LedR11, (tension >= 4586) ? HIGH : LOW) ;
    digitalWrite(LedR12, (tension >= 5000) ? HIGH : LOW) ;
}

26/01/15 à 18h30
Modifié

+2 -0

Dryusdan, lundi 26 janvier 2015 à 18h38

J'ai pensé à un while et une variable temporaire mais je ne sis pas sur. (i'm on mobile donc difficile de coder x)) Heu… J'ai vu cette écriture de condition une fois en php mais j'ai un peu galéré à la comprendre, donc en arduino surtout que je pensais pas que cela étais possible et surtout d'un l'enceinte même d'une fonction Peux tu m'expliquer un peu ce type de condition s'il te plaît? Merci beaucoup

26/01/15 à 18h38

+0 -0

anonyme, lundi 26 janvier 2015 à 18h42
Modifié

Ces conditions sont ce qu'on appelle des ternaires.

Pour faire simple, l'expression int variable = (condition à tester) ? a : b ; est équivalent à :

int variable ;
if (condition a tester)
{
    variable = a ;
}
else
{
    variable  = b ;
}

De même, return (a>b) ? a : b est équivalent à :

if (a>b)
{
    return a ;
}
else
{
    return b ;
}

Quelques recherches sur le net pourront t'en dire plus si besoin, mais dans les faits, ces conditions sont pas franchement compliquées.

26/01/15 à 18h42
Modifié

+1 -0

Dryusdan, lundi 26 janvier 2015 à 18h44

Humm, merci

Et merci beaucoup pour cette aide

26/01/15 à 18h44

+0 -0

anonyme, lundi 26 janvier 2015 à 19h30

Je ne connais pas le langage utilisé par arduino, mais il doit aussi être possible d'utiliser un tableau pour les pins des leds et des boucles for pour accéder successivement à chaque led ?

Par exemple, le setup deviendrait :

int LEDS[12] = {2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13};
int BATT = 0;

void setup() {
    for(int index = 0; index < 12; index++)
        pinMode(LEDS[index], OUTPUT);
}

26/01/15 à 19h30

+0 -0

Dryusdan, lundi 26 janvier 2015 à 19h35

Aussi, le seul soucis c'est que pour une organisation j'avais mis LEDR, LEDJ (pour rouge et jaune) même si cela ce change;) Merci pour ton avis

26/01/15 à 19h35

+0 -0

Eskimon, mardi 27 janvier 2015 à 08h40
Modifié

une fonction "affichage" + une ternaire + un tableau de leds (comme propose mes VDD) ?

void affichage(char nb) {
   //on eteint/allume led eds
   for(char i=0; i<12; i++) {
      digitalWrite(led[i], (i<nb) ? HIGH : LOW);
   }
}

27/01/15 à 08h40
Modifié

ZdS, le best du Zeste ! | Tuto Arduino, blog, etc

+2 -0

Dryusdan, mardi 27 janvier 2015 à 10h14

Le seul soucis Eskimon, c'est que dans la void loop faut que je fasse un switch avec un nb? Cela va être plus long (et peut-être plus compliqué) mais ça ce laisse tenter Je verrais sur mon programme

27/01/15 à 10h14

+0 -0

Eskimon, mardi 27 janvier 2015 à 10h18

L'ideal serait d'avoir une fonction (mathematique) qui relie tes tensions et tes nb. Tu pourrais sortir un graphe liant le nombre de leds a allumer en fonction de la tension ?

27/01/15 à 10h18

ZdS, le best du Zeste ! | Tuto Arduino, blog, etc

+0 -0

Dryusdan, mardi 27 janvier 2015 à 10h27

Un graph? Comment ça? (/me est un superbe inculte x) )

27/01/15 à 10h27

+0 -0

Eskimon, mardi 27 janvier 2015 à 10h32

un graphique. Sur l'axe des absices tu mets tes tensions, et en ordonnées ton "nombre de leds a allumer" (ou le contraire, dans ce cas c'est pas bien grave)

Avec un peu de chance tu auras une relation sympa et mathématiquement applicable pour faire un nb = (int) x*tension+y par exemple

27/01/15 à 10h32

ZdS, le best du Zeste ! | Tuto Arduino, blog, etc

+0 -0

Dryusdan, mardi 27 janvier 2015 à 10h50

Exact, ça fais bien une courbe linéaire ax+b Bon, je vais devoir trouver "la fonction" pour passer des mathématiques à l'informatique (je peux pas poster d'image, srv-img est bloqué ><)

27/01/15 à 10h50

+0 -0

anonyme, mardi 27 janvier 2015 à 13h11

Si c'est linéaire une simple division entre la tension maximal et le nombre de leds donnera la différence de tension entre chaque led.

void loop() {
    int step = 5000 / 12 // tension maximal / nombre de leds
    int tension = map(analogRead(BATT), 0, 1023, 0, 5000);

    for(int index = 0; index < 12; index++)
        digitalWrite(LEDS[index], (tension > step * index) ? HIGH : LOW);
}

Je n'y connais pas grand chose en électricité, mais vous êtes sûr que l'on détermine la puissance restante d'une batterie avec un intervalle de tension régulier ?

27/01/15 à 13h11

+0 -0

Dryusdan, mardi 27 janvier 2015 à 13h48

J'ai à pris la semaine dernière que la courbe de décharge d'une batterie n'est pas linéaire. Et le gros délire c'est que je ne trouve pas cette courbe sur internet donc je vais devoir la faire jeudi ^^'

27/01/15 à 13h48

+0 -0

Eskimon, mardi 27 janvier 2015 à 13h51

C'est quoi comme technologie de batterie ?

27/01/15 à 13h51

ZdS, le best du Zeste ! | Tuto Arduino, blog, etc

+0 -0

Dryusdan, mardi 27 janvier 2015 à 18h53

Une ni-mh de 7,2v 1800mAh http://www.google.fr/search?q=7+2v+1800mah+ni-mh&prmd=ivns&source=lnms&tbm=isch&sa=X&ei=FdDHVKmMOMa5Ufe8grgG&ved=0CAYQ_AUoAQ#i=8

Il y a une différence entre les batteries? (de toute façon faut que je fasse des test jeudi :/)

27/01/15 à 18h53

+0 -0

Eskimon, mardi 27 janvier 2015 à 20h27

Ouip, entre de la batterie plomb et des li-po c'est quand meme pas la meme (et les ni-mh ou ni-cd doivent etre plus proche du comportement des li-po/li-ion je pense)

27/01/15 à 20h27

ZdS, le best du Zeste ! | Tuto Arduino, blog, etc

+0 -0

Dryusdan, mardi 27 janvier 2015 à 20h51

Oula heu…xD j'ai presque rien compris je penseaiso qu'il existai la batterie au plomb, au lithium et un autre type, mais pas autant ^_~ surtout, quel sont les différences (autant en apprendre beaucoup maintenant :P)