Problème LCD

Version imprimable

Problème LCD

Bonsoir,
je souhaite réaliser un coupe gâteau automatique. Pour cela, je sélectionne le nombre de parts souhaitées sur un écran LCD de référence Adafruit_RGBLCDShield. Cet écran possède des boutons poussoirs permettant d'augmenter le nombre de parts ainsi que de confirmer ce nombre. Or, lorsque je téléverse mon programme, l'écran s'allume, on voit "Nombre Parts : 2", mais impossible d'augmenter ce nombre, et je ne vois pas le problème. De plus, le programme contrôle également un moteur, mettant en rotation la plateau. Je vous laisse y jeter un œil afin de repérer une quelconque faute :

Code:

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
 
Adafruit_RGBLCDShield lcd = Adafruit_RGBLCDShield(); 
 
#define RED 0x1
#define YELLOW 0x3
#define GREEN 0x2
#define TEAL 0x6
#define BLUE 0x4
#define VIOLET 0x5
#define WHITE 0x7
#define maSortie       
 
int DirB = 8;
int BrakeB = 13;
int nbrPart = 1;
int I=0;
 
 
void setup() {
 
 
 
  lcd.begin(16, 2);         
  int time = millis();   
  lcd.print("Nbre parts : ");       
 
 
  pinMode(DirB,OUTPUT);
  pinMode(BrakeB,OUTPUT);
  }
 
void loop() {
    lcd.setCursor(0, 1);
 
 
  uint8_t buttons = lcd.readButtons();
 
  if (buttons) {
    lcd.clear();
    lcd.setCursor(0,0);
 
    if (buttons & BUTTON_LEFT) {
      nbrPart= nbrPart +1;
      if (nbrPart > 10) {
      nbrPart= 2; 
      }
 
      delay(200);
 
 
     lcd.print("Nbre parts = ");
     lcd.print(nbrPart);
 
 
    }
    if (buttons & BUTTON_SELECT) {
       lcd.print("Nbre parts validé");
    }
   }
  if (nbrPart==2) {
   while(I < nbrPart){ 
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(2431);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
  }
 if (nbrPart==3) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(2161);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
 }
 if (nbrPart==4) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(2026);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
 }
 if (nbrPart==5) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1945);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB LOW);
    I=I+1;
  }
 }
 if (nbrPart==6){
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1891);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
    }
 }
 if (nbrPart==7) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1853);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
 }
 if (nbrPart==8) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1823);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
 }
 if (nbrPart==9) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1801);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
 }
 if (nbrPart==10) {
    while(I < nbrPart){
    digitalWrite(DirB, HIGH);
    digitalWrite(BrakeB, LOW);
    delay(1783);
    digitalWrite(DirB, LOW);
    digitalWrite(BrakeB, LOW);
    I=I+1;
  }
}
 I=0;
 }

14/05/2017, 11h00
Vincent PETIT

Salut,
Dans tes conditions, celle là par exemple if (buttons & BUTTON_LEFT).

Tu es sur que tu ne voulais pas plutôt faire un ET logique (&&) au lieu d'un bit à bit (&) ?

N'est ce pas plutôt ceci qu'il faut écrire :
if (buttons && BUTTON_LEFT).

Il faut vérifier toutes tes conditions if pour voir si il n'y a pas la même bizarrerie.

A+
14/05/2017, 11h06
Sparrounet

Bonjour,
je vous remercie tout d'abord répondu. Effectivement, je souhaite un fonction logique ET, il manque donc un &.

Merci d'avance
Sébastien
14/05/2017, 12h43
Vincent PETIT

Et ça donne quoi comme résultat avec le ET logique ?

Tu rencontres toujours le même problème ?

Le problème, c'est que je n'ai pas le matériel à disposition en ce moment. Et puis j'ai regardé sur le site d'Adafruit, où l'on peut trouver le début des programmes permettant d'utiliser l'écran. Dans ces programmes, on voit la ligne

Code:

if (buttons & BUTTON_LEFT)

. Cette ligne ne possède qu'un seul &. De plus lorsque je fait fonctionner le programme commandant seulement l'écran LCD, ce dernier fonctionne parfaitement. Voila ce programme :

Code:

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
 
#include <Wire.h>
#include <Adafruit_RGBLCDShield.h>
#include <utility/Adafruit_MCP23017.h>
 
 
 
Adafruit_RGBLCDShield lcd = Adafruit_RGBLCDShield();
 
int nbrPart=1;
#define RED 0x1
#define YELLOW 0x3
#define GREEN 0x2
#define TEAL 0x6
#define BLUE 0x4
#define VIOLET 0x5
#define WHITE 0x7
#define maSortie
void setup() {
 
  Serial.begin(9600);
 
  lcd.begin(16, 2);
 
 
 
  int time = millis();
  lcd.print("Nbre parts : ");
 
 
  lcd.setBacklight(WHITE);
}
 
uint8_t i=0;
void loop() {
 
 
  lcd.setCursor(0, 1);
 
 
  uint8_t buttons = lcd.readButtons();
 
  if (buttons) {
    lcd.clear();
    lcd.setCursor(0,0);
 
    if (buttons & BUTTON_LEFT) {
      nbrPart= nbrPart +1;
      if (nbrPart > 10) {
      nbrPart= 2;  
      }
 
      delay(200);
 
     Serial.print("nbrPart = ");
     Serial.println(nbrPart);
     lcd.print("Nbre parts = ");
     lcd.print(nbrPart);
 
 
    }
    if (buttons & BUTTON_SELECT) {
       lcd.print("Nbre parts validé");
    }
   }

14/05/2017, 12h59
Delias

Bonjour

Au sujet de la réponse à Vincent, c'est bien un ET bit à bit & qu'il faut, car c'est un masquage de la variable sur 8bits buttons qui contient l'état des 5 boutons après la ligne 36 uint8_t buttons = lcd.readButtons();.

Le problème est que le code de gestion des moteurs n'attend pas la validation pour s'exécuter, occupant la plus grande partie du temps d'exécution du micro en réduisant la durée de captation des boutons.

Delias
14/05/2017, 13h06
Sparrounet

Merci de vos réponses

comment solutionner ce problème dans ce cas?
14/05/2017, 13h40
Delias

J'ai déjà dit que le problème c'est que le code de gestion des moteurs (à partir de la ligne 60) n'attend pas la validation pour s'exécuter. Il y a plusieurs méthodes pour corriger cela.
Il faut corriger cela (donc faire l'inverse de ce que je dis comme étant le problème), si on te donne directement la réponse tu n'apprendras rien!

N'as-tu pas le moyen de calculer le délais en fonction du nombre de part, histoire à ne pas devoir copier 9x le même code?
Et pour l'instant, tu dois être Superman pour couper les parts. Car tes boucles, mettent le moteur en marche pour la durée du délais, l'arrêtent, puis le redémarrent pour l'itération de la boucle suivante. La durée d'arrêt devant être de l'ordre de la nanoseconde (au pif c'est 10 à 20 coups d'horloge)

Delias
14/05/2017, 15h16
Vincent PETIT

Citation:

Envoyé par Delias

Au sujet de la réponse à Vincent, c'est bien un ET bit à bit & qu'il faut, car c'est un masquage de la variable sur 8bits buttons qui contient l'état des 5 boutons après la ligne 36 uint8_t buttons = lcd.readButtons();

Ok, il n'y avait pas de lien vers la bibliothèque qui gère le LCD donc ma suggestion était fausse.

A partir de maintenant, je ne répondrai que lorsque j'aurai toutes les infos (plus de pile dans la boule de cristal)
14/05/2017, 15h24
Sparrounet

J'ai modifié mon code en rajoutant un délai lorsque j'arrête la mise sous tension du moteur. De plus, j'ai mis la partie commandant le moteur dans l'instruction

Code:

if (buttons && BUTTON_SELECT) {

. Je ne sais pas si cela change quelque chose...

Merci de vos réponses
14/05/2017, 15h41
Delias
C'est malin j'ai fâché Vincent.:mouarf:
Ce fut d’abord une simple intuition (en relation avec les noms de la fonction et des constantes), puis un petit :google: sur "Adafruit_RGBLCDShield" me donne en 1er lien le GitHub dans le quel j'ai trouvé le fichier .h et ces quelques lignes:
Code:

1 2 3 4 5 #define BUTTON_UP 0x08 #define BUTTON_DOWN 0x04 #define BUTTON_LEFT 0x10 #define BUTTON_RIGHT 0x02 #define BUTTON_SELECT 0x01
C'est très classique d'un masque. Pour le coup ma boule de cristal était en meilleur forme :question:

Citation:

Envoyé par Sparrounet

J'ai modifié mon code en rajoutant un délai lorsque j'arrête la mise sous tension du moteur. De plus, j'ai mis la partie commandant le moteur dans l'instruction

Code:

if (buttons && BUTTON_SELECT) {

. Je ne sais pas si cela change quelque chose...

Tu en penses quoi? D'après toi, quand ou à quelle condition le code de gestion du moteur doit être exécuté?

Delias
14/05/2017, 15h48
Sparrounet

Le moteur doit se lancer lorsque l'on a validé le nombre de parts souhaitées.
14/05/2017, 16h05
Delias

Citation:

Envoyé par Sparrounet

Le moteur doit se lancer lorsque l'on a validé le nombre de parts souhaitées.

Et cela correspond à quelle partie du code?

Je crois que l'on a fini ici...

Delias
14/05/2017, 16h10
Sparrounet

Eh bien cela correspond au moment où l'on confirme ce nombre soit lorsque l'on appuie sur le bouton Select.
14/05/2017, 16h11
Sparrounet

C'est donc pour ça que j'ai intégré la partie concernant le moteur dans l'instruction

Code:

if (buttons && BUTTON_SELECT) {
14/05/2017, 16h29
Delias

Citation:

Envoyé par Sparrounet

C'est donc pour ça que j'ai intégré la partie concernant le moteur dans l'instruction

Code:

if (buttons && BUTTON_SELECT) {

Oups j'avais lu trop vite :oops:

Et alors cela marche ou pas?

Delias
14/05/2017, 16h32
Sparrounet

Eh bien je ne peux pas le tester aujourd'hui, mais j'essaierai de le tester demain. Je te tiens au courant.

Merci de ton aide précieuse

Sébastien
14/05/2017, 21h55
Vincent PETIT

Citation:

Envoyé par Delias

C'est malin j'ai fâché Vincent.:mouarf:

Non non pas du tout ;)
Mais si j'ai dit ça, c'est parce que j'ai compris que ça allait amener à une nouvelle erreur

A+