Optimiser un programme

L'auteur de ce sujet a trouvé une solution à son problème.
Auteur du sujet

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 ;) )

  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
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
// 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)

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0

Cette réponse a aidé l'auteur du sujet

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.

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
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) ;
}

Édité par anonyme

+2 -0
Auteur du sujet

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

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0

Ces conditions sont ce qu'on appelle des ternaires.

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

1
2
3
4
5
6
7
8
9
int variable ;
if (condition a tester)
{
    variable = a ;
}
else
{
    variable  = b ;
}

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

1
2
3
4
5
6
7
8
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.

Édité par anonyme

+1 -0

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 :

1
2
3
4
5
6
7
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);
}
+0 -0

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

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

Édité par Eskimon

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

+2 -0
Auteur du sujet

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 :p

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0

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

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

+0 -0
Auteur du sujet

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é ><)

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0

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.

1
2
3
4
5
6
7
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 ?

+0 -0
Auteur du sujet

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 ^^'

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0
Auteur du sujet

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 :/)

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0
Auteur du sujet

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)

Un projet de gestion de container sur Zds | dryWamp une page d'accueil pour Wamp

+0 -0
Vous devez être connecté pour pouvoir poster un message.
Connexion

Pas encore inscrit ?

Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte