Discussion :

[legrandse]

Bonjour à tous,

Je n'ai pas fini dans mes déboires de code
il semble qu'après avoir encodé mes chiffres sur le keypad la chaine enteredcode ne se remplit pas.

D'ailleurs est-ce que la syntaxe est elle juste ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

enteredcode[10] += key;

Comment faire pour ajouter des suites de caractères à une chaine ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
int mastercode = 275012; 
 
  char enteredcode[10] = "";
  for (int i = 0; i < 6; i++){
    char key = kpd.waitForKey();
    if(key == '#'){     // # is used to cancel
    i = 6; //skip to the end
    lcd.print("Cancelled"); 
    }
    if(key != NO_KEY){  
      lcd.print(key);  
      enteredcode[10] += key;  
    }
  }
 char buffer[15];
  itoa(mastercode,buffer,10);
 
 int com = strcmp(enteredcode, buffer);
 if (com == 0)  
 
  {
    lcd.setCursor (0, 2);
    lcd.print("CODE CORRECT"); 
 
   }
else {
lcd.setCursor (0, 2);
    lcd.print("CODE INCORRECT"); 
}

Merci pour l'aide

[Matt_Houston]

Tu ne peux pas manipuler tes chaînes de cette manière, ce n'est pas du C++. En C on utilise string.h.

Revois immédiatement le chapitre sur la manipulation de chaînes en C, ou tu ne t'en sortiras pas.

[legrandse]

J'ai effectué qques modifications et la variable "enteredcode" se remplit correctement avec "strcat".

Cependant j'ai un "CODE INCORRECT" au résultat de comparaison et je ne vois pas pourquoi.

Pourtant je compare bien 2 variables de type char[].

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 int mastercode = 275012;
   char enteredcode[10] ="";
 
 
  for (int i = 0; i < 6; i++){
    char key = kpd.waitForKey();
    if(key == '#'){ 
    i = 6; 
    lcd.print("Cancelled"); 
    }
    if(key != NO_KEY){ 
      lcd.print(key); 
      char str[2];
      str[0]=key;
      str[1]='\0';
 
     strcat(enteredcode, str);
     }
  }//fin boucle for
 
 char buffer[15];
 itoa(mastercode,buffer,10);
 
 int com = strcmp(enteredcode, buffer);
 if (com == 0)  
 
  {
    lcd.setCursor (0, 2);
    lcd.print("CODE CORRECT"); 
 
   }
else {
lcd.setCursor (0, 2);
    lcd.print("CODE INCORRECT"); 
}

[legrandse]

J'essaye des pistes mais voici ce que je ne comprend pas.
Si j'ajoute un affichage sur la console série via

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Serial.print(......);

des variables buffer et enteredcode j'ai respectivement ces 2 valeurs qui s'affichent :

12222 pour buffer
275012 pour enteredcode

Pourquoi ai-je déjà une différence entre ces 2 variables ?

[Médinoc]

Franchement, d'ici je ne vois pas où est l'erreur, ton code m'a l'air correct.

[legrandse]

Je cherche toujours

Cette variante avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char mastercode[10]= "275012";

fonctionne correctement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
char mastercode[10] = "275012";
   char enteredcode[10] ="";
 
 
  for (int i = 0; i < 6; i++){
    char key = kpd.waitForKey();
    if(key == '#'){ 
    i = 6; 
    lcd.print("Cancelled"); 
    }
    if(key != NO_KEY){ 
      lcd.print(key); 
      char str[2];
      str[0]=key;
      str[1]='\0';
 
     strcat(enteredcode, str);
     }
  }//fin boucle for
 
 
 int com = strcmp(enteredcode, mastercode);
 if (com == 0)  
 
  {
    lcd.setCursor (0, 2);
    lcd.print("CODE CORRECT"); 
 
   }
else {
lcd.setCursor (0, 2);
    lcd.print("CODE INCORRECT"); 
}

L'erreur viendrait donc de la conversion de mastercode int vers char.
Sauf que je ne vois pas où.

[legrandse]

Voilà j'ai trouvé l'erreur mais je ne sais pas la solutionner.

avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

int mastercode = 27;

cela fonctionne correctement.

au delà de 2 chiffres il me dit que c'est incorrect.

Qu'en pensez vous ?

[legrandse]

j'ai encore trouvé un indice.
Je sais encoder sur 3 chiffres mais limité à la valeur de 255.
Donc c'est la limitation de itoa() à 256 qui cause le problème.

[legrandse]

J'ai testé itoa() à part et la limitation à 256 n'a rien à voir.

[Matt_Houston]

strtol est à préférer à itoa si disponible.

Combien vaut sizeof(int) sur ton système ? Quelle est l'architecture cible ? Comment compiles-tu ? Quel est la sortie complète de la compilation ?

Il te reste l'option de ne manipuler que des chaînes de caractères, sans passer par des entiers.

[legrandse]

Je compile sur un simulateur pour Arduino (Autodesk circuits.io)
Je n'ai donc pas accès au résultat complet de la compilation.

sizeof(int) me donne la valeur de 2 selon la machine hébergée.

Je ne sais pas tester dans un autre environnement car je n'ai ni le keypad, ni le LCD pour mon Arduino.

J'ai essayé un code "résumé" sur mon PC avec CodeBlocks et je n'ai pas d'erreur donc çà fonctionne.

Serait-ce un bug du simulateur, je n'en sais rien...
J'ai tout de même envoyé un mail aux développeurs Autodesk.

[Matt_Houston]

Si sizeof(int) == 2 et en supposant CHAR_BIT == 8, on ne peut pas y coder 275012 puisque les maxima sont (rappel) : 32767 en signé et 65536 en non signé. Utilise des chaînes de caractères pour stocker les codes.

[Médinoc]

Ah, je n'ai plus l'habitude des systèmes où les int ne font que 16 bits...

Mais dans ce cas, il est anormal qu'il n'y ait pas eu au moins un warning sur le int mastercode = 275012;! legrandse, tu devrais vérifier les options de compilation.

[legrandse]

Merci pour ces info.

J'ai testé directement dans mon Arduino à la place du simulateur et le problème est identique (et même pas de warning dans le compilateur).

Je suppose que c'est du à la limite de l'uC de l'Arduino.

Merci beaucoup pour votre aide sur ce problème.