ATNEL tech-forum

Re: makro do operowania diodami LED

2016-01-06T22:10:39+01:00

Czasem proste rozwiązania są na wyciągnięcie ręki, wystarczy tylko spojrzeć z ciut innej strony dziękuję za pomoc

Statystyki: Napisane przez Dariu5z — 6 sty 2016, o 22:10

Re: makro do operowania diodami LED

2016-01-06T20:22:04+01:00

Napisałem:

Quote:

Jak potrzebujesz koniecznie mieć gdzieś indziej zdefiniowane LEDy, to możesz zrobić tablicę LED[] i w niej przypisywać wartości.

więc nie zgodzę się z:

Quote:

Nie wiem dlaczego tak w stronę makr ciągniecie.

Zamiast funkcji robisz tablicę:
[syntax=c]uint8_t LED[] = { LED_1, LED_1, LED_1, LED_2, LED_2, LED_2, LED_3, LED_3 };[/syntax]
i możesz zamiast funkcji ją użyć:
[syntax=c]if (pomiar_temp[i][4] == -1) PORTD &= ~LED[i];[/syntax]

--
Pozdrawiam,
Robert

Statystyki: Napisane przez rskup — 6 sty 2016, o 20:22

Re: makro do operowania diodami LED

2016-01-06T20:08:42+01:00

Nie wiem dlaczego tak w stronę makr ciągniecie. Od makr należy uciekać jak najdalej. Wiadomo, że nieraz ułatwią wiele, ale mają jedną podstawową wadę - nie są sprawdzane przez kompilator (np. zgodność typu). Zamiast makr należy stosować funkcje - po to one są.
Fragment:
[syntax=c]for (i = 0; i < 8; i++) {
if (pomiar_temp[i][4] == -1) PORTD &= ~(LED_(i));
if (pomiar_temp[i][4] == 0) PORTD |= LED_(i);
if (pomiar_temp[i][4] == 1) PORTD ^= LED_(i);
}[/syntax]
można zastąpić:
[syntax=c]for (i = 0; i < 8; i++) {
if (pomiar_temp[i][4] == -1) zapal_led(i);
if (pomiar_temp[i][4] == 0) zgas_led(i);
if (pomiar_temp[i][4] == 1) migaj_led(i);
}[/syntax]
a funkcje obsługi LEDów robisz osobno, np.:
[syntax=c]void zapal_led(char led)
{
if (led == 1) PORTD &= ~(LED_1);
if (led == 2) PORTD &= ~(LED_2);
//itd. dla liczby LEDów - oznaczmy jako max_led
if ((led <1) || (led >max_led)) wywal_blad();
}[/syntax]
Zaletą jest, że można dowolnie podłączyć LED (port, pin, sposób sterowania itp.), kompilator sprawdza poprawność, w funkcji wywal_blad() możemy zdefiniować co ma się dziać w programie, gdy w jakimś nieprzewidzianym przypadku zmienna i wyjdzie poza dopuszczalny zakres. Łatwo też będzie zmienić, gdy zamiast zapalania diody będziemy chcieli jeszcze coś obsłużyć (np. dodać efekt dżwiękowy) - wystarczy zmodyfikować zapal_led(). Oczywiście lepiej nazwę wtedy zmienić, np. na: reakcja_temp_za_wysoka() - taka nazwa pozwala też łatwo rozczytać się w kodzie i znaleźć błąd w złym przypisaniu funkcji w danej sytuacji.
Aha, być może ciało funkcji zapal_led gasi, a nie zapala, ale to już sam wiesz. Wiem, że użyłem makr, ale tylko tych, które definiują porty, piny itp. Tego się raczej nie przeskoczy, jeśli chce się zachować czytelność kodu. Zresztą zaleca się używanie definicji zastępujących wartości liczbowe, np. zamiast 3.14 lepiej użyć
[syntax=c]#define liczba_pi 3.14[/syntax]
gdyż, jeśli będzie potrzeba zwiększyć jej dokładność to wystarczy zrobić to w definicji a nie w wielu miejscach programu, gdzie jej użyliśmy. Podobnie jeśli przyjdzie zmienić typ można zapisać:
[syntax=c]#define liczba_pi 3.14f[/syntax]

Statystyki: Napisane przez PJS — 6 sty 2016, o 20:08

Re: makro do operowania diodami LED

2016-01-06T19:45:07+01:00

Mieszasz kod dla preprocesora (czyli wykonywany przed kompilacją) z kodem dla kompilatora.
i jest zmienną kodu w C. Więc preprocesor robi z tego LED_i.

--
Pozdrawiam,
Robert

------------------------ [ Dodano po: 7 minutach ]

Jak potrzebujesz koniecznie mieć gdzieś indziej zdefiniowane LEDy, to możesz zrobić tablicę LED[] i w niej przypisywać wartości.

Statystyki: Napisane przez rskup — 6 sty 2016, o 19:45

makro do operowania diodami LED

2016-01-06T19:08:24+01:00

Cześć, mampewien problem z makrami, potrzebuję pomocy bardziej doświadczonej osoby.
mam 8 czujników DS18B20 i 8 diod
jeśli temperatura jest za wysoka to dioda mruga co 0.5s
jeśli za niska - świeci światłem ciągłym
jeśli ok - nic się nie dzieje
sprawdzanie i mruganie mam załatwione wewnątrz takiej konstrukcji
[syntax=c]if (flaga_05sek) {
if (pomiar_temp[0][4] == -1) {PORTD &= ~(LED_1);}
if (pomiar_temp[0][4] == 0) {PORTD |= LED_1;}
if (pomiar_temp[0][4] == 1) {PORTD ^= LED_1;}
if (pomiar_temp[1][4] == -1) PORTD &= ~(LED_2);
if (pomiar_temp[1][4] == 0) PORTD |= LED_2;
if (pomiar_temp[1][4] == 1) PORTD ^= LED_2;

...

flaga_05sek = 0;
}[/syntax]
pomiar_temp[][] to bufor do odczytu temperatury
wcześniej instrukcja sprawdza, czy temp jest za niska (-1), ok (0) czy za wysoka (1) i daje wartość w 4 kolumnie
diody mam na LED_1, LED_2, ..., LED_8
pytanie brzmi, jak te if'y zastąpić makrem LED_(x)?

Zrobiłem takie cudo, ale kompilator krzyczy mi, że LED_(i) to niezdefiniowana funkcja...
[syntax=c]#define LED_(x) XLED_(x)
#define XLED_(x) (LED_##x)

...

for (i = 0; i < 8; i++) {
if (pomiar_temp[i][4] == -1) PORTD &= ~(LED_(i));
if (pomiar_temp[i][4] == 0) PORTD |= LED_(i);
if (pomiar_temp[i][4] == 1) PORTD ^= LED_(i);
}[/syntax]

Statystyki: Napisane przez Dariu5z — 6 sty 2016, o 19:08