Problème condition if

Version imprimable

Bonjour
Je suis débutante en ARDUINO, je voudrais commander un LED lié à ARDUINO, par interface python.
-si on clique sur commande automatique: le led se clignote
-si on clique sur commande réel:
-si on clique sur led_on:le led s'allume.
-si on clique sur led_off:le led s'éteint.
vous trouvez ci-dessous mon code:
en le testant sur proteus:
la condition de clignotant fonctionne parfaitement, parmi que celle de led_on led_off ne fonctionne jamais(malgré que l'ARDUINO a reçu la donnée correspondante.
Merci d'avance.
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 int led = 12; int mode= 4; int readByte; int readByte1; void led_On_Off(byte ledStatus) { digitalWrite(led, ledStatus); } void led_Commande(byte ledPin) { led_On_Off(1); delay(500); led_On_Off(0);delay(500); } void led_reel() { if (readByte=='c'){digitalWrite(led, 1);} else if (readByte=='d'){digitalWrite(led, 0);} } void setup() { // initialize the digital pin as an output. pinMode(led, OUTPUT); pinMode(mode,INPUT); Serial.begin(9600); } // the loop routine runs over and over again forever: void loop() { while (Serial.available() > 0) { readByte=Serial.read(); if (readByte=='a') {led_reel();} else if (readByte=='b') {led_Commande(12);} } }

16/07/2020, 21h52
Jay M

qu'est-ce que le programme python envoie ? n'oubliez pas qu'à chaque tour de boucle s'il n'y a rien à lire, rien ne se passe

sinon vous appelez led_reel() dans la loop quand vous avez reçu 'a' et dans cette fonction vous comparez avec 'c' et 'd'... il n'y a AUCUNE chance pour que readByte vaille 'c' ou 'd' puisque c'est 'a'...
17/07/2020, 09h29
MARIE MARY

l'ARDINO avait bien reçu les caractères a,b,c ou d, à chaque clique sur le label python correspondant.
J’ai ajouté un loop(),et rien ne s’est passé.
17/07/2020, 09h55
Jay M

Je veux bien croire qu’il reçoit chaque caractère puisque vous les lisez mais regardez vos fonctions. Quelle est la valeur reçue et comme je le dis ci dessous avec quoi comparez vous ?

Si vous n’êtes pas sure de vous ou convaincue imprimez la valeur dans la fonction avant de faire les if

Bonjour MARIE

Citation:

Envoyé par MARIE MARY

-si on clique sur commande automatique: le led se clignote
-si on clique sur commande réel:
-si on clique sur led_on:le led s'allume.
-si on clique sur led_off:le led s'éteint.

Pour le mode automatique (b), pas de problème.
Par contre, si j'ai bien compris, pour les 2 fonctions du mode commande réel,

il faut cliquer commande réel (a) et led_on (c) pour allumer la LED.
l faut cliquer commande réel (a) et led_off (d) pour éteindre la LED.

Si c'est ainsi, il faut mémoriser le fait du clic commande réel (variable boolean commandeReelOn) pour, au caractère suivant (c ou d), aiguiller le programme sur led_reel().
Et ça donne ceci:
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 int led = 12; int mode= 4; int readByte; int readByte1; boolean commandeReelOn = false; void led_On_Off(byte ledStatus) { digitalWrite(led, ledStatus); } void led_Commande(byte ledPin) { led_On_Off(1); delay(500); led_On_Off(0);delay(500); } void led_reel() { if (readByte=='c'){digitalWrite(led, 1);} else if (readByte=='d'){digitalWrite(led, 0);} commandeReelOn = false; // Sortir du mode commande reel } void setup() { // initialize the digital pin as an output. pinMode(led, OUTPUT); pinMode(mode,INPUT); Serial.begin(9600); } // the loop routine runs over and over again forever: void loop() { while (Serial.available() > 0) { readByte=Serial.read(); if (readByte=='a') {commandeReelOn = true;} // Memorise la mise en mode commande reel else if (commandeReelOn == true) {led_reel();} else if (readByte=='b') {led_Commande(12);} } }
Cordialement
jpbbricole

17/07/2020, 12h49
MARIE MARY

Bonjour Monsieur
C’est gentil de votre part de m’aider. J’ai essayé ce programme , il répond justement comme le fonctionnement que vous avez décrivez.
Après que l’ARDUINO reçu (a) :
Ie led s’allume lorsqu’on clique sur led_on.
Alors je voudrais qu’il s’éteigne lorsqu’on clique sur led_off (sans cliquer de nouveau sur (a)).
Cordialement.

Bonjour Marie

J'avais pas tout compris :?
Voilà c'est corrigé:

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
 
 int led = 12;
 int mode= 4;
 int readByte;
 int readByte1;
 boolean commandeReelOn = false;
 
 
 void led_On_Off(byte ledStatus)
 {
	 digitalWrite(led, ledStatus);
 }
 void led_Commande(byte ledPin)
 {
	 led_On_Off(1);
	 delay(500);
	 led_On_Off(0);delay(500);
 }
 void led_reel()
 {
	 if (readByte =='c' && commandeReelOn == true){digitalWrite(led, 1);}
	 else if (readByte=='d'){digitalWrite(led, 0);}
	commandeReelOn = false;                           // Sortir du mode commande reel
 }
 
 void setup() {
	 // initialize the digital pin as an output.
	 pinMode(led, OUTPUT);
	 pinMode(mode,INPUT);
	 Serial.begin(9600);
 }
 
 // the loop routine runs over and over again forever:
 void loop() {
 
	 while (Serial.available() > 0) {
		 readByte=Serial.read();
		 if (readByte=='a') {commandeReelOn = true;}       // Memorise la mise en mode commande reel
		 else if (readByte=='c' || readByte=='d') {led_reel();}
		 else if (readByte=='b') {led_Commande(12);}		 
	 }
 }

Voilà une autre façon de prendre les décisions, avec Switch/case:

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
 
 int led = 12;
 int mode= 4;
 int readByte;
 int readByte1;
 boolean commandeReelOn = false;
 
 
 void led_On_Off(byte ledStatus)
 {
	 digitalWrite(led, ledStatus);
 }
 void led_Commande(byte ledPin)
 {
	 led_On_Off(1);
	 delay(500);
	 led_On_Off(0);delay(500);
 }
 void led_reel()
 {
	if (readByte =='c' && commandeReelOn == true){
		digitalWrite(led, 1);
	}
	else if (readByte=='d'){
		digitalWrite(led, 0);
	}
	commandeReelOn = false;                           // Sortir du mode commande reel
 }
 
 void setup() {
	 // initialize the digital pin as an output.
	 pinMode(led, OUTPUT);
	 pinMode(mode,INPUT);
	 Serial.begin(9600);
 }
 
 // the loop routine runs over and over again forever:
 void loop() {
 
	while (Serial.available() > 0) {
		readByte=Serial.read();
 
		switch(readByte)                                   // Choix en fonction du caractere recu
		{
			case 'a':                                      // Si passe en mode reelOn
				commandeReelOn = true;
				break;
			case 'b':                                     
				led_Commande(12);
				break;
			case 'c':
				led_reel();
				break;
			case 'd':
				led_reel();
				break;
		}
	}
}

Cordialement
jpbbricole

17/07/2020, 15h28
MARIE MARY

Bonjour jpbbricole
Merci bien Monsieur.
Cordialement.:D

J’aime bien la version avec switch case on se rapproche de la machine à états :)

On peut simplifier alors un peu puisque lorsqu’on appelle led_reel() on sait ce que l’on a reçu donc on peut en tenir compte et avoir une fonction pour chaque cas et on n’a plus à mémoriser le caractère reçu.
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 const byte ledPin = 12; bool commandeReelOn = false; void led_On_Off(byte ledStatus) { digitalWrite(ledPin, ledStatus); } void led_Commande() { led_On_Off(HIGH); delay(500); led_On_Off(LOW); delay(500); } void led_reel_on() { if (commandeReelOn){ led_On_Off(HIGH); commandeReelOn = false; // Sortir du mode commande reel } } void led_reel_off() { if (commandeReelOn){ led_On_Off(LOW); commandeReelOn = false; // Sortir du mode commande reel } } void setup() { pinMode(ledPin, OUTPUT); Serial.begin(9600); } void loop() { switch(Serial.read()) { // Choix en fonction du caractere recu. Read retourne -1 si rien à lire case 'a': // Si passe en mode reelOn commandeReelOn = true; break; case 'b': // on clignote led_Commande(); break; case 'c': // allume si mode commande et sort du mode commande led_reel_on(); break; case 'd': // éteint si mode commande et sort du mode commande led_reel_off(); break; default: break; // on ignore toute autre commande sur le port série } }
Comme cela quand on lit les options possibles dans le switch on voit tout de suite ce que ça fait.

Notez que lorsqu’on travaille en mode commande sur 1 caractère ce nest pas la peine de tester available() dans la loop(), il suffit d’essayer de lire. Comme read() retourne -1 si on a rien à lire alors ça ne correspond à aucun des case et donc rien ne se passe. (La fonction available est lente car elle fait un calcul avec un modulo et vous n’avez pas vraiment besoin de savoir combien de caractères sont dispos, juste traiter celui qui arrive quand il arrive).

22/07/2020, 14h51
MARIE MARY

Bonjour
Merci bien Jay M.Surtout poue cet astuce d'éliminer le available.
Cordialement.