piszę mały programik i prosiłbym o weryfikację pomysłu obsługi buzzera,
założenia:
-niezależne działanie buzzera od programu (nieblokowanie)
-ustawienie czasu "time" (na który załączany jest buzzer)
-ustawienie czasu "break_time" (na który jest wyłączany buzzer)
-ustawienie ilości "count" (ilość powtórzeń)
-Timer2 klika co 10ms
Po chwili powstał następujący kod:
[syntax=c]#define BUZZER (1<<PA1)
#define BUZZER_ON PORTA |= BUZZER // STAN WYSOKI NA PA1
#define BUZZER_OFF PORTA &= ~BUZZER // STAN NISKI NA PA1
#define BUZZER_STATUS (PINA & BUZZER)
uint8_t time;
uint8_t break_time;
uint8_t count;
void buzzer(void)
{
if(!time)
time = 20;
if(!break_time)
break_time = 20;
if(count>0)
{
if(!BUZZER_STATUS && !Timer2) // jeśli buzzer wyłączony
{
if(!Timer2)
{
Timer2 = time; //włączamy buzzer na czas time
BUZZER_ON;
}
}
else if(!Timer2)//jeśli buzzer włączony oraz minał czas time
{
if(!Timer2)
{
Timer2 = break_time; //wyłączamy buzzer na czas break_time
BUZZER_OFF;
count-=1;
}
}
}
}[/syntax]
Moja funkcja wywoływana jest co "obrót" pętli głównej, wszystko działa zgodnie z założeniami, jest jednak jedna rzecz która nie do końca się mi podoba - aby "ustawić" buzzer mam drugą funkcję która zmienia parametry (time, break_time, count)
i wywoływana jest w miejscu w którym chce włączyć buzzer,
Proszę o konstruktywną krytykę do funkcji, co można uprościć, ulepszyć?
Pozdrawiam
KoreXStatystyki: Napisane przez KoreX — 5 gru 2015, o 20:22
]]>