password

[michmatar]

Hello everyone, i just want to ask you if you could help me with my code, i want to write a code that asks for a password and when i write the code it gives me correct pass if its correct or incorrect pass if its wrong
my code :
int first_key;
int second_key;
int third_key;
int fourth_key;
lcd_putc('\f');
lcd_gotoxy(1,1);
printf(lcd_putc,"Enter pin:");
delay_ms(10);
lcd_gotoxy(1,2);
while(!enter){
ad = kbd_getc();
if(ad!=' ') {
if(ad == '#') enter = true;
else { printf ( lcd_putc, "%c", ad);
posX++;
}
}
}

first_key=kbd_getc();
second_key=kbd_getc();
third_key=kbd_getc();
fourth_key=kbd_getc();
if((first_key =='1')&&(second_key =='2')&&(third_key =='3') &&(fourth_key =='4'))
{
lcd_putc('\f');
printf(lcd_putc,"incorrect pass");
delay_ms(10);
}
else
{
lcd_putc('\f');
printf(lcd_putc,"correct pass");
delay_ms(10);
}
}
the problem is when i type any number it gives me correct pass
any help?

[Mike Walne]

Several points:-

!) Use the General Discussion section for this type of query.
2) Provide complete compilable code we can copy and paste to test.
3) Learn to use the code button.
4) Learn to indent code.
5) Read the forum guidelines.

Mike

[PCM programmer]

Your basic assumption is that kbd_getc() is similar to getc(), such that
it waits for a key to be pressed. Unfortunately, kbd_getc() doesn't work
that way. It needs to be polled continuously at a suitable interval.
Your code doesn't do that.

In this post below, I've written a little routine called keypad_getc().
It waits in a loop until a key is pressed, and then it returns the key:
http://www.ccsinfo.com/forum/viewtopic.php?t=41848&start=4
If you put that routine above main() and substitute keypad_getc()
for every place in your code where you have kbd_getc(), your program
will work better. To make this work, you have to edit kbd.c as follows: