Je ne comprends pas pourquoi mon code marche

BlackPirate · December 27, 2012, 9:00pm

Bonjour,

c'est une situation un peu bizarre, mais mon code marche sans que je ne comprenne pourquoi .
Le but de mon code est d'allumer certaines LEDs en fonction du caractère entré ( formant un caractère selon le code Braille avec la disposition des LEDs ), et tant que l'on a pas entré un nouveau caractère, les LEDs restent allumées.
En effet, je ne m'attendais pas à ce qu'il fonctionne tel que je le voulais et pourtant (j'avais changé certaines valeurs sans trop y croire ) si ! Je crois que c'est à cause des valeurs renvoyées par la librairie Sérial mais je ne comprends absolument pas pourquoi.
La valeur que j'ai changé et qui a rendu mon code fonctionnel est dans la fonction loop ( à la ligne "if(Serial.available()>1)" ).

Si vous pouviez m'éclairer, ce serait vraiment très sympa Voici le code :

Variables globales :

int ledHG=12; // led en haut à gauche
int ledHD=11; // led en haut à droite
int ledMG=10; // led au milieu à gauche
int ledMD=9;  // led au milieu à droite
int ledBG=8;  // led en bas à gauche
int ledBD=7;  // led en bas à droite
int boutonGauche=5; // bouton à gauche
int boutonDroit=4; // bouton à droite
boolean etat= true ;

Fonction pour éteindre les Leds :

void eteindreLed(){
   digitalWrite(ledHG,LOW); // éteindre les LEDs
   digitalWrite(ledHD,LOW); // éteindre les LEDs
   digitalWrite(ledMG,LOW); // éteindre les LEDs
   digitalWrite(ledMD,LOW); // éteindre les LEDs
   digitalWrite(ledBG,LOW); // éteindre les LEDs
   digitalWrite(ledBD,LOW); // éteindre les LEDs 
}

Fonction pour allumer la LED en fonction du caractère entré :

void allumerLed(char lettre) {
 
    if (lettre>=97 && lettre<=122) // upToLowerCase
    lettre=lettre-32;
    
    switch (lettre){        // Allumer les LEDs
      
    case 65: // A
    digitalWrite(ledHG,HIGH);
    break;
    
    case 66: // B
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMG,HIGH);
    break;
    
    case 67: // C
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    break;
    
    case 68: // D
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMD,HIGH);
    break;
    
    case 69: // E
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMD,HIGH);
    break;
    
    case 70: // F
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    break;
    
    case 71: // G
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    break;
    
    case 72: // H
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    break;
    
    case 73: // I
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    break;
    
    case 74: // J
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    break;
    
    case 75: // K
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 76: // L
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledBG,HIGH);
    break; 
    
    case 77: // M
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 78: // N
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 79: // O
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 80: // P
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 81: // Q
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 82: // R
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 83: // S
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 84: // T
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    break;
    
    case 85: // U
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledBG,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    
    case 86: // V
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledBG,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    
    case 87: // W
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    
    case 88: // X
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledBG,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    
    case 89: // Y
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledHD,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    
    case 90: // Z
    digitalWrite(ledHG,HIGH);
    digitalWrite(ledMD,HIGH);
    digitalWrite(ledBG,HIGH);
    digitalWrite(ledBD,HIGH);
    break;
    }
    
 }

Fonction setup :

void setup(){
  
Serial.begin(9600);
pinMode (ledHG,OUTPUT);
pinMode (ledHD,OUTPUT);
pinMode (ledMG,OUTPUT);
pinMode (ledMD,OUTPUT);
pinMode (ledBG,OUTPUT);
pinMode (ledBD,OUTPUT);
pinMode (boutonGauche,INPUT);
pinMode (boutonDroit,INPUT);
digitalWrite(boutonGauche,HIGH);
digitalWrite(boutonDroit,HIGH);
 
}

Fonction loop :

void loop (){

char lettre = Serial.read();
do {
  allumerLed(lettre);
  if(Serial.available()>1){
  eteindreLed();
  etat=false;
  }
} while ( etat==true);

etat=true;
}

B_tto · December 28, 2012, 8:46am

Je comprends pas trop le rôle de la boucle do. Perso je trouve plus simple et plus clair de faire :

void loop (){

  if(Serial.available()){
     eteindreLed();
     allumerLed(Serial.read());
    }

}

BlackPirate · December 28, 2012, 9:36am

En fait, vu que ca ne marchait, j'ai essayé plein de méthodes différentes pour parvenir à faire ce que je souhaitais.
Et j'ai essayé votre fonction loop :
en entrant un caractère dans le moniteur série, les LEDs correspondant au caractère ( c'est du Braille, les LEDs diffèrent en fonction du caractère entré ) s'allument un dixième de seconde, puis s'éteignent directement.

En fait dans mon code précédent, je ne comprends pas pourquoi ca marche avec "if(Serial.available()>1)" dans la fonction loop et pas if(Serial.available()>0) car logiquement ça devrait être ça non ?

B_tto · December 28, 2012, 9:54am

Mais il ne marche pas du tout ou bien ça fait pas ce que tu demandes ?

EDIT : il semble y avoir un soucis puisque mon code ne marche pas, et ce dysfontionnement laisse entendre que 2 caractères arrivent. Ce qui expliquerait pourquoi ton code fonctionne avec >1 et pas >0. Par quoi tu passes pour envoyer ton caractère à l'arduino ?

jfs · December 28, 2012, 10:03am

Ton 1 et ton 0 sont un booléen, ce qui veut dire que 0 à la valeur 0 et tout ce qui est au dessus à la valeur de 1 (2=1, 3447=1... etc)

Donc tu testes ton entrée sérial et si ta valeur est supérieure à 1 ça fonctionne..... car elle est toujours égale à 1

et si tu testes avec 0, ça revient au même car en fait tu testes si quelque chose entre sur le port série.... donc ça teste "vide" ou "plein"

Et toi tu teste plus grand que "vide" ou plus grand que "plein".... ce qui en terme de valeur revient au même

B_tto · December 28, 2012, 10:34am

Serial.available() ne renvoi pas un booleen, mais le nombre byte dispo dans le buffer

jfs · December 28, 2012, 10:41am

Ah ouais, c'est juste ...... joyeux Noël.... comment ça c'est déjà passé XD

Artouste · December 28, 2012, 11:28am

Jean-François:
Ah ouais, c'est juste ...... joyeux Noël.... comment ça c'est déjà passé XD

bonjour JF
Alors ? "on" a abusé d'humagne ?

BlackPirate · December 28, 2012, 11:36am

En fait le but de mon programme est le suivant :

lorsqu'on entre une lettre dans le moniteur série ( qui est intégré au compilateur ), certaines LEDs s'allument ( en fonction du code Braille). Et puis elles restent allumées tant que l'on n'entre pas un nouveau caractère. Si on entre un nouveau caractère, les LEDs s'allument en fonction de ce nouveau caractère et restent allumées. Et ainsi de suite ^^

La ce qui me perturbe c'est que lorsqu'on utilise la fonction Serial.read(), la fonction Serial.available() devrait renvoyer 0 ( du moins, si l'on a rentré qu'un seul caractère. Et puis le programme les LEDs et vérifient si il y a d'autres caractères dans le buffer. Logiquement on devrait utiliser if ( Serial.available()>0) mais ca ne marche pas ! Les LEDs s'allument un dixième de seconde puis s'éteignent. Je me casse la tête pour comprendre pourquoi, en vain !

jfs · December 28, 2012, 11:37am

Artouste:
bonjour JF
Alors ? "on" a abusé d'humagne ?

Salut Artouste,

Non, juste pas beaucoup dormi cette nuit

B_tto · December 28, 2012, 11:48am

@BlackPirate : essaye en rajoutant un delay :

void loop (){

  if(Serial.available()){
Serial.println(Serial.available()); //histoire de voir si c'est bien une question de nombre de caractère
delay(20);
     eteindreLed();
     allumerLed(Serial.read());
    }

}

BlackPirate · December 28, 2012, 11:50am

Alors j'ai essayé ton code :

quand je rentre 1 caractère dans le moniteur série : il affiche 1, saute une ligne, réaffiche 1. Pourtant il ne devrait afficher qu'un seul 1, non ?

B_tto · December 28, 2012, 2:25pm

Essaye en mettant le delay avant le println

BlackPirate · December 28, 2012, 2:45pm

Ca m'affiche :

2
1

Et les LEDs s'allument un dixième de seconde puis s'éteignent. Mais mon code d'avant marchait, le problème n'est pas de faire un code qui marche. Je veux juste savoir pourquoi mettre 1 et pas 0 à la ligne if ( Serial.available()>0). Ca m'intrigue cette histoire !

system · December 28, 2012, 4:12pm

Si tu utilises le moniteur série de l'Arduino, et que tu rentre "1" suivit de "Entrée" alors par défaut le caractère "1" est suivit du caractère de fin de ligne \x0D (appellé Carriage Return)

Tu peux configurer le moniteur série pour envoyer :

rien (juste tes caractères)
tes caractères suivit de CR (\x0D)
tes caractères suivit de CR (\x0D) puis LF (\x0A)

Ceci explique cela

BONNES FETES DE FIN D'ANNEE !!!!!!!

BlackPirate · December 28, 2012, 5:18pm

Mystère résolu ! Merci beaucoup à vous tous pour votre aide

Et bonnes fêtes de fin d'année à vous aussi

Topic		Replies	Views
[RESOLU] Led qui ne veulent pas s'éteindre Français	5	969	May 6, 2021
Allumer une Led en fonction du contenu lu dans le moniteur série. Français	10	1660	May 6, 2021
Soucis de débutant avec une led!! Français	3	624	May 6, 2021
(RESOLU)Petits problèmes avec mes leds Français	15	1353	May 19, 2021
Pb rendre à jour led Français	6	196	June 16, 2024

Je ne comprends pas pourquoi mon code marche

Related topics