ATNEL tech-forum

Re: wyświetlanie menu +pilot RC5

2016-10-01T18:38:22+01:00

ciężko jest mi coś więcej doradzić ... kod który tu przedstawiłeś jest niekompletny, w sumie do tematu menu należało by podejść inaczej.

spróbuj przebudować swój kod w taki sposób:

[syntax=c]typedef enum {
WLACZ_POMPKE,
WYLACZ_POMPKE
}POMPKA;
...

void wybor_opcji_menu(POMPKA zmienna){
switch(zmienna)
{
case WLACZ_POMPKE:
//jakiś kod do obsługi gdy pompka ma być włączona
break;
case WYLACZ_POMPKE:
//jakiś kod do obsługi gdy pompka ma być wyłączona
break;
default:
//sekcja dyfault może być zaimplementowana w przypadku wystąpienia jakiejś innej sytuacji, a chcemy powiadomić o tym urzytkownika.
break;
}
}
...
void rysuj_menu( void ){
//kod na potrzebę wyrysowania menu
}[/syntax]

Po wyświetleniu menu oczekuje na wykonanie się przerwania do obsługi pilota podczerwieni (tu tylko domyślam się, że poruszasz się po menu przy pomocy pilota)
I po odebraniu odpowiedniego "kodu guzika" przekazujesz odpowiedni typ enum POMPKA do funkcji void wybor_opcji_menu(POMPKA zmienna).
Powinno to być znacznie czytelniejsze

pozdrawiam,
Tomek.

Statystyki: Napisane przez Tomek86 — 1 paź 2016, o 18:38

Re: wyświetlanie menu +pilot RC5

2016-10-01T16:33:49+01:00

Wyzerowanie zmiennej command spowodowało że program sie zawiesza lub zapętla na drugim do while:

[syntax=c]void wyswietlanie_menu(void) {
uint8_t flaga = 0;
do {
lcd_locate(0, 0);
lcd_str("zalaczania pompki");
wyswietlanie_czasu_na_lcd(zalaczania_pompki.godzina,
zalaczania_pompki.minuty, zalaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1)
break;

} while (1);

flaga = 0;
command=-1;
do {
lcd_locate(0, 0);
lcd_str("wylaczania pompki");
wyswietlanie_czasu_na_lcd(wylaczania_pompki.godzina,
wylaczania_pompki.minuty, wylaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;
} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1) {
break;
}

} while (1);

flaga = 0;
command=-1;

do {

if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1) {
break;
}
} while (1);
}[/syntax]

Statystyki: Napisane przez mariuszA1985 — 1 paź 2016, o 16:33

Re: wyświetlanie menu +pilot RC5

2016-10-01T16:03:54+01:00

Zapomniałem dopisać, że tak jak kolega @Rafał555 napisał, że musisz wyzerować zmienną command, np command = -1.

Bo zwróć uwagę, kiedy na pilocie wciśniesz guzik z kodem 40, to przy kolejnej pętli do..while, ta zmienna ma dalej 'command=40'.

Więc dodatkowo musisz "zerować" (bądź ustawić inną wartość) zmienną command.

Bo jak tego nie zrobisz, to druga pętla w lini 32.[syntax=c]if (command == 40[/syntax] wykona się i zostanie ustawiona flaga = 1;

pozdrawiam,
Tomek.

Statystyki: Napisane przez Tomek86 — 1 paź 2016, o 16:03

Re: wyświetlanie menu +pilot RC5

2016-10-01T15:39:03+01:00

Zrobiłem tak jak poradził mi kolega Tomek86. Teraz kod wygląda tak:

[syntax=c]void wyswietlanie_menu(void) {
uint8_t flaga = 0;
do {
lcd_locate(0, 0);
lcd_str("zalaczania pompki");
wyswietlanie_czasu_na_lcd(zalaczania_pompki.godzina,
zalaczania_pompki.minuty, zalaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1)
break;

} while (1);

flaga = 0;

do {
lcd_locate(0, 0);
lcd_str("wylaczania pompki");
wyswietlanie_czasu_na_lcd(wylaczania_pompki.godzina,
wylaczania_pompki.minuty, wylaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;
} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1) {
break;
}

} while (1);

flaga = 0;

do {

if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1) {
break;
}
} while (1);
}[/syntax]

i co dziwne efekt jest ten sam.

Statystyki: Napisane przez mariuszA1985 — 1 paź 2016, o 15:39

Re: wyświetlanie menu +pilot RC5

2016-10-01T15:05:55+01:00

A po co Ci tyle pętli while ?

Wysłane z mojego LIVE przy użyciu Tapatalka

Statystyki: Napisane przez majster — 1 paź 2016, o 15:05

Re: wyświetlanie menu +pilot RC5

2016-10-01T15:02:10+01:00

Po spełnieniu warunku w lini 18. "flaga == 1", następuje wyjście z pętli i przejście do drugiej pętli do..while, niestety zmienna flaga=1, (bo nie została ustawiona na 0), dlatego pętla 2 do..while "samoczynnie", kończy swój cykl przed wcześnie.

Zauważ, że przy 3 pętlą do..while masz już ustawione flaga=0

wystarczy zmienną "flaga" wyzerować przed 2 pętlą do..while.

pozdrawiam,
Tomek.

Statystyki: Napisane przez Tomek86 — 1 paź 2016, o 15:02

Re: wyświetlanie menu +pilot RC5

2016-10-01T14:22:30+01:00

Niestety to rozwiązanie zapętla program na drugiej pętli do while.

Statystyki: Napisane przez mariuszA1985 — 1 paź 2016, o 14:22

Re: wyświetlanie menu +pilot RC5

2016-10-01T12:16:54+01:00

W tym warunku gdzie ustawiasz flagę wyzeruj zmienną command - tak na szybko co rzuca mi się w oczy.

Statystyki: Napisane przez Rafał555 — 1 paź 2016, o 12:16

wyświetlanie menu +pilot RC5

2016-10-01T10:48:34+01:00

Witam,
programowaniem mikroprocesorów zajmuje się od niedawna wiec proszę o wyrozumiałość.
Napisałem krótką funkcję do wyświetlania na lcd menu (void wyswietlanie_menu(void)), która ma prezentować wprowadzony wcześniej czas załączania i wyłączania pompki do akwarium. Po naciśnięciu klawisza pilota (command =40) program powinien przeskakiwać do następnego wyświetlanego parametru. Problem polega na tym, że samoczynnie bez oczekiwania na klawisz pilota, program wychodzi z pętli do while. Dzieje się tak w przed ostatniej pętli do while która wyświetla czas wylaczania pompki.
Proszę o pomoc w rozwiązaniu tego problemu.

[syntax=c]void wyswietlanie_menu(void) {
uint8_t flaga = 0;
do {
lcd_locate(0, 0);
lcd_str("zalaczania pompki");
wyswietlanie_czasu_na_lcd(zalaczania_pompki.godzina,
zalaczania_pompki.minuty, zalaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1)
break;

} while (1);

do {
lcd_locate(0, 0);
lcd_str("wylaczania pompki");
wyswietlanie_czasu_na_lcd(wylaczania_pompki.godzina,
wylaczania_pompki.minuty, wylaczania_pompki.sekundy, 1);
if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;
} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1)
break;

} while (1);
flaga=0;
do {

if (Ir_key_press_flag) {

if (command == 40)
flaga = 1;

} else {
Ir_key_press_flag = 0;
command = -1;
flaga = 0;
}
if (flaga == 1)
break;

} while (1);
}[/syntax]

Statystyki: Napisane przez mariuszA1985 — 1 paź 2016, o 10:48