problème d'ajout de caractère à une chaine

Version imprimable

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:

enteredcode[10] += key;

Comment faire pour ajouter des suites de caractères à une chaine ?
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 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 ;)

28/09/2016, 16h31
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.

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:

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"); 
}

29/09/2016, 12h00
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:

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 ?
29/09/2016, 13h20
Médinoc

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

Je cherche toujours ;)

Cette variante avec

Code:

char mastercode[10]= "275012";

fonctionne correctement.

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
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ù.

29/09/2016, 14h36
legrandse

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

avec

Code:

int mastercode = 27;

cela fonctionne correctement.

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

Qu'en pensez vous ?
29/09/2016, 14h47
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.
29/09/2016, 15h21
legrandse

J'ai testé itoa() à part et la limitation à 256 n'a rien à voir. :(
29/09/2016, 16h10
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.
29/09/2016, 16h33
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.
29/09/2016, 17h12
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.
30/09/2016, 09h33
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.
30/09/2016, 09h55
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.