Problème exécution boucle if

Version imprimable

bonjour à vous,

j'utilise un arduino mega 2560 et un keypad.

le programme demande à l'utilisateur de rentrer une premiere fois le mot de passe (6 caractères) , puis demande de le resaisir.
Pour la première partie, pas de soucis.

En ce qui concerne la deuxième, ça bug.
ça ne prend pas en considération le !=NO_key.

Merci par avance pour votre aide.

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
 
//Arduino 
 
 
// PIN switch with 16-digit numeric keypad
// <a href="http://tronixstuff.com/tutorials" target="_blank">http://tronixstuff.com/tutorials</a> > chapter 42
 
#include "Keypad.h"
//#include <Wire.h>
//#include <TWILiquidCrystal.h>
//LiquidCrystal lcd(12, 11, 5, 4, 3, 2);
 
const byte ROWS = 4; //four rows
const byte COLS = 4; //four columns
char keys[ROWS][COLS] =
{
  {
    '1','2','3','A'  }
  ,
  {
    '4','5','6','B'  }
  ,
  {
    '7','8','9','C'  }
  ,
  {
    '*','0','#','D'  }
};
byte rowPins[ROWS] = {
  2, 3, 4, 5}; //connect to the row pinouts of the keypad
byte colPins[COLS] = {
  6, 7, 8, 9}; //connect to the column pinouts of the keypad
 
Keypad keypad = Keypad( makeKeymap(keys), rowPins, colPins, ROWS, COLS );
 
char PIN[6]={
  '0','0','0','0','0','0'}; //initialisation du code
char attempt[6]={ 
  '0','0','0','0','0','0'}; // used for comparison
int z=0;
int x=0;
 
void setup()
{
  pinMode(11,OUTPUT);
  Serial.begin(9600);
//  lcd.begin(16, 2);
 Serial.println("ENTRER CODE 1 ");
 
}
 
 
 void code()
 {
 
   int i=0;
for (int i=0; i<6; i++) 
  {
  char key = keypad.getKey();
  if (key != NO_KEY)
  {
  PIN[z]=key;
  z++;
 
     if (z==6)
     {
      Serial.println("code1 ok");
 
    code2();
     }
     }
 }
 
  }
 
void code2()
 {
Serial.println("code2 ");
 
 int j=0;
 int x=0;
 
 for (int j=0; j<6; j++) 
  {
 char key = keypad.getKey();
  if (key != NO_KEY)
  {
  attempt[x]=key;
  x++;
    Serial.println(x);
     if (x==6)
     {
      Serial.println("code2 ok");
 }
  }
 }
 }
 
  void loop()
{
  code();
           }

Salut,
Tu dis que dans la fonction Code2 la condition if (key != NO_KEY) n'est jamais satisfaite. Comment en es tu sur ? Est ce que le code n'aurait pas un problème plus loin ?

Je vois aussi que tu as une variable globale qui s'appelle "x" et tu déclares un variable locale du même nom "x" dans Code2 ! C'est normal ?

Est ce que tu peux ajouter les 2 println("debug") que j'ai commenté afin de vérifier là où ça ne fonctionne pas.
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 void code2() { Serial.println("code2 "); int j=0; int x=0; for (int j=0; j<6; j++) { char key = keypad.getKey(); Serial.println("debug1"); // debug trace 1 if (key != NO_KEY) { Serial.println("debug2"); // debug trace 2 attempt[x]=key; x++; Serial.println(x); if (x==6) { Serial.println("code2 ok"); } } } }

13/10/2015, 17h34
hugodu28

Bonjour,
Pour ce qui est de la variable déclarée un peu partout c'est normal pas normal car j'ai fait plein de test.

Pour ce qui est du debug je te donne déjà la réponse .
Le debug1 sera écrit 6 fois. Si je demande la valeur de x ça s'incremente.

Debug2, n'est jamais écrit .

Franchement j'ai du mal à comprendre.

Merci
13/10/2015, 18h43
Vincent PETIT

Donc tout fonctionne bien mise à part le println ?
13/10/2015, 20h43
hugodu28

je me demande si

Code:

keypad.getKey()

est toujours pris en compte

Ca me semble bizarre ce problème.
Tu as essayer de ne pas appeler code2() depuis code() pour rendre plus fluide la lecture et la compréhension du programme ?

Un peu dans ce genre là :

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
#include "Keypad.h"
 
const byte ROWS = 4; //four rows
const byte COLS = 4; //four columns
char keys[ROWS][COLS] = {
  {'1','2','3','A'},
  {'4','5','6','B'},
  {'7','8','9','C'},
  {'*','0','#','D'}
};
 
byte rowPins[ROWS] = {2, 3, 4, 5}; //connect to the row pinouts of the keypad
 
byte colPins[COLS] = {6, 7, 8, 9}; //connect to the column pinouts of the keypad
 
char PIN1[6]={'0','0','0','0','0','0'}; //initialisation du code
char PIN2[6]={'0','0','0','0','0','0'}; // used for comparison
 
 
void setup()
{
   Keypad keypad = Keypad( makeKeymap(keys), rowPins, colPins, ROWS, COLS );
   pinMode(11,OUTPUT);
   Serial.begin(9600);
   Serial.println("ENTRER CODE 1 ");
}
 
 
void code()
{
   char key = '0';
   int x = 0;
   int i = 0;
 
   for (i = 0; i < 6; i++) 
   {
      key = keypad.getKey();
 
      if (key != NO_KEY)
      {
  	  PIN1[x] = key;
  	  x++;
 
     	  if (x >= 6)
      		Serial.println("PIN1 ok");
      }
   }
 
   x = 0;
 
   for (i = 0; i < 6; i++) 
   {
      key = keypad.getKey();
 
      if (key != NO_KEY)
      {
  	  PIN2[x] = key;
  	  x++;
 
     	  if (x >= 6)
      		Serial.println("PIN2 ok");
      }
   }
 
   x = 0;
 
 
// verification
   for (i = 0; i < 6; i++)
   {
      if(PIN1[i] != PIN2[i])
	 x++;
   }
 
   if(x != 0)
	Serial.println("PIN 1 different de PIN2");
   else
	Serial.println("PIN 1 egale a PIN2");
 
}
 
 
void loop()
{
  code();
}

A vérifier si ça compile car je n'utilise pas l'IDE Arduino et je n'ai donc pas pu essayer.

Salut,
je viens de découvrir quelque chose d'étrange.

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
 
  char key = '0';
   int x = 0;
   int i = 0;   //on ne peut pas mettre dans la fonction code() sinon la valeur reste à 0.
 
void code()
{
 
 
   for (i = 0; i < 6; i++) 
   {
      key = keypad.getKey();
 
      if (key != NO_KEY)
      {
          Serial.println("debug1")  // quelque soit le code même ceux de début de discution, 
                                              // pour x==6, on a bien ("PIN1 ok")
                                             // mais puor x>6, ça renvoi toujours "debug1"; dans le code précédent, ça ne renvoie pas à code2()
  	  PIN1[x] = key;
  	  x++;
 
     	  if (x == 6)
      		Serial.println("PIN1 ok");
                code2();              // pas de renvoie si code 2 existe, reprise de la   boucle code() et toutes les touches du clavier ne fonctionnent pas.
      }
   }
 
 ...
// verification
   for (i = 0; i < 6; i++)
   {
      if(PIN1[i] != PIN2[i])
	 x++;
   }
 
   if(x != 0)
	Serial.println("PIN 1 different de PIN2");
   else
	Serial.println("PIN 1 egale a PIN2");
 
}
 
 
void loop()
{
  code();
}

autre chose, la variable i varie de façon très folklorique 0,5,4,3,3,0,1...
Merci à toi, pour ton aide

17/10/2015, 15h45
Vincent PETIT

Donc ça fonctionne ?
Par contre je ne comprends pas non plus pourquoi une variable locale ne fonctionne pas !?

Et le code que j'ai posté, il marche ou pas ?

A+

Pour le int i = 0; je pense que tu as un problème de portée de varibale.
Tu as du la déclarer en global car tu t'en sers aussi dans code2(), n'est ce pas ?

Si tu déclares "i" en local dans code() alors code2() ne peut pas la voir ou bien il faut aussi déclarer "i" dans code2(). Mais attention ce ne sera pas les mêmes variable "i".

Pour ta deuxième erreur, il faut mettre les accolades au niveau du if
Code:

1 2 3 4 5 if (x == 6) { Serial.println("PIN1 ok"); code2(); // pas de renvoie si code 2 existe, reprise de la boucle code() et toutes les touches du clavier ne fonctionnent pas. }
Sans accolade code2() se trouve dans la boucle for et va donc s'exécuter à chaque tour de boucle.