Problem z kodem. Mikrokontroler, LCD i klawiatura.

Question

Witam.
Ten problem dotyczy programowania mikrokontrolera (Atmega8). Ale wydaje mi się że można to rozwiązać nawet bez wiedzy o mikrokontrolerach. A więc napisałem funkcję, która ma za zadanie pobranie od użytkownika dwie cyfry, które są zamieniane w czas. Niestety wyświetlacz ciągle się odświeża a cyfry pokazują się obok siebie tylko jak przytrzymam przycisk. Tu jest kod:

// Urywka z programu//

int inznak()
{
while(key = read_keypad()) 
znak = bin2bin(key);
return znak;
}

int inczas()
{
int t;
LCD_CLEAR;
LCD_LOCATE(1,0);
lcd_puts("Wprowadź czas:");
znakw = inznak();
t = 10*key;
LCD_LOCATE(6,1);
LCD_WRITE_DATA(znakw);
znakw = inznak();
t += key;
LCD_LOCATE(7,1);
LCD_WRITE_DATA(znakw);

return t;

znakw, znak, key to unsigned char, funkcje read_keypad i bin2bind działają poprawnie bo sprawdzały się w wielu poprzednich programach.

Wydaje mi się że problem, leży w funkcji inczas. Jeżeli ktoś zna rozwiązanie to prosiłbym o jakieś wskazówki.

Answer 1

read_keypad() co zwraca i w jakich sytuacjach? Czyżby 0 w przypadku gdy żaden klawisz nie jest wciśnięty? Jeżeli tak, to tutaj (w while) leży błąd błąd.

Comments

Tak wygląda ta funkcja :

unsigned char read_keypad(void)
{
   unsigned char row,col,kej;
   
   for(row=0x7e,kej=1; row>=0x77; row=(row<<1|0x1)&0x7f)
   {
        PORTD = row; 
      for(col=0x10; col< 0x80; col<<=1, kej++)
                   if(!(PIND & col)) return kej;
   }
   return 0;

Jeżeli naciśniemy np. 8 to zwraca 8, potem to w funkcji bin2bin zmienia na '8'. 

Jeżeli nic nie naciśniemy to zwraca 0. Próbowałem w innych formach zapisać ten kod, aby program poczekał aż coś wcisnę, ale wciąż się odświeża

---

bo u ciebie while się przerywa w momencie gdy zwrócisz 0, spróbuj while(!read_key()), ale nie gwarantuję że zadziała

---

Teraz już się nie odświeża, ale po wciśnięciu jakiego kolwiek przycisku zaczyna się odświeżać i nie jestem w stanie wpisać żadnej cyfry

---

hmm, jak myślałem. Nie wiem jak dokładnie masz przkeazywane wartości do metody :P W takim razie twój kod wymaga małego przebudowania