Incompréhension analyse des bits d'un byte

Bonjour à tous
Je tente en vain de stocker l'état des bits d'un octet dans un tableau de booleen:

Code:

---

1 2	byte memRel = 0b11011011; //mémoire d'état des relais bool etatRel[8]; //etat des relais actifs

---

Puis je tente de stocker l'état des mes bits dans le tableau de booleen:

Code:

---

1 2 3 4 5 6 7 8

//*****************lecture etat************************** void lectureEtat(){ if(memRel != 0b11111111){ //0b11111111 tous les relais carte seraient OFF for(int i=0;i<8;i++){ etatRel[i]= (memRel & (1<<i) == 0)? true: false ; } } }

---

mais lorsque je lis mon tableau pour y affecter 'F' ou 'O':

Code:

---

1 2 3 4 5 6 7 8

if (PCF8574Ok) { char tempo [10]= {""}; for (int j = 0; j < 8; j++) { tempo[j] = (etatRel[j]) ? 'O' : 'F'; } tempo[8]= 'Z'; tempo[9]='\0'; Serial1.print(tempo); }

---

j'obtiens FFFFFFFF au lieu de FFOFFOFF

Où est la bétise?

sans doute dans la priorité des opérateurs

essayez de modifier dans lectureEtat() la boucle avec:

Code:

---

1 2 3

for (int i = 0; i < 8; i++) { etatRel[i] = (memRel & (1 << i)) != 0 ; // notez l'ajout de parenthèses }

---

mais le plus simple plutôt que de créer un tableau qui va manger 8 octets et d'utiliser bitRead() quand vous voulez lire l'état ou bitWrite() quand vous voulez forcer un état sur un bit particulier. Comme cela tout tient sur un seul octet

https://reference.arduino.cc/referen...bytes/bitread/
https://reference.arduino.cc/referen...ytes/bitwrite/

Merci pour cette réponse qui a solutionné mon problème. J'ai écrit:

Code:

---

1 2 3 4 5

if(memRel != 0b11111111){ //0b11111111 tous les relais carte sont OFF for(int i=0;i<8;i++){ etatRel[i]= ((memRel & (1<<i)) == 0)? true: false ; } }

---

J'avais bien pensé utiliser bitRead() et bitWrite() mais je n'ai pas trouvé la syntaxe. Dans l'aide Arduino, il n'y a pas d'exemple d'utilisation. Ou bien j'ai mal cherché.
Cordialement
Michel

oui votre écriture fonctionne aussi

comme

Code:

---

((memRel & (1<<i)) == 0)

---

est déjà une valeur de vérité (le résultat de la comparaison est true ou false) vous pourriez utiliser directement cette valeur au lieu de prendre l'opérateur ternaire

Code:

---

1 2 3 4 5

if(memRel != 0b11111111){ //0b11111111 tous les relais carte sont OFF for(int i=0;i<8;i++){ etatRel[i]= (memRel & (1<<i)) == 0 ; } }

---

avec bitRead vous auriez écrit

Code:

---

1 2 3 4

for(int i=0;i<8;i++){ etatRel[i]= bitRead(memRel, i) == 0; }

---

==> on dit dans quelle variable on veut lire puis quel N° de bit

Parfait, j'ai tout compris.
Si j'ose abuser de vos compétences, j'ai encore un petit problème de compréhension.
J'ai:

Code:

---

1 2 3 4

const char *rls= "Relais"; char buf1[16]; sprintf(buf1,"%s N %c ONZ%c", rls,car[3],'\0'); Serial1.print(buf1);

---

Avec car[3]=1 par exemple. Et 'Z' le caractère de fin de message.
Mon problème: en réception je vois que

Citation:

---

"R N 1 ON"

---

.
"elais" n'apparaît pas.
Pourquoi le %s ne prend que le premier caractère.
Ce n'est pas vraiment facile le C pour ne pas utiliuser l'objet String.

ça ne devrait pas être le cas sauf si vous avez un dépassement mémoire quelque part.

Il n'y a pas besoin de mettre le caractère nul à la fin, sprintf() le fait pour vous.
il vaut mieux aussi utiliser la version snprintf() qui prend en paramètre la taille du buffer pour éviter tout dépassement mémoire

essayez ce bout de code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

const char *rls = "Relais"; char buf1[16];   void setup() { char car[] = "00010000"; // un '1' à l'index 3   Serial.begin(115200); snprintf(buf1, sizeof buf1, "%s N %c ONZ", rls, car[3]); Serial.print(buf1); }   void loop() {}

---

vous devriez bien voir Relais N 1 ONZ

Merci bien. C'est résolu. Le problème venait du côté de la réception! Toujours avec ces fameux HC12.