ATNEL tech-forum

Re: Obsługa kilku przycisków

2013-04-25T23:43:36+01:00

W końcu znalazłem chwilę żeby się temu przyjrzeć i okazało się że podwójny przełącznik którego używam po wciśnięciu pierwszego rozwierał masę od drugiego! Rozebrałem, połączyłem i jest ok. Ehh ale namieszał .
Dzięki za pomoc.

PZDR.

Statystyki: Napisane przez żbik — 25 kwi 2013, o 23:43

Re: Obsługa kilku przycisków

2013-04-24T17:05:35+01:00

Podejrzewam, że jest tu błąd, bo poniższe linie są mało logiczne.
[syntax=c]if(pwm[diode_nr][0]!=bright)
{
pwm[diode_nr][1]=bright;
pwm[diode_nr][2]=1;
}[/syntax]
Jak sprawdzasz pwm[diode_nr][0] to powinieneś wpisywać do pwm[diode_nr][0] a nie do pwm[diode_nr][1]. Jaki rozmiar ma tablica pwm? Możliwe, że wpisując 1 do pwm[diode_nr][2] zmieniasz sobie jasność następnej diody.
Polecam strukturę, bo wtedy nazwa pola informuje o tym co zmieniasz lub chociaż zastąpienie 0 i 1 odpowiednimi definicjami np.
[syntax=c]//1
typedef struct {
uint8_t bright;
uint8_t lock;
} PWM; //nowy typ zmiennej

PWM pwm[4]; //utworzenie tablicy struktur dla 4 diod

//...
brightChanger(&pwm[i],255);//przekazanie adresu struktury do funkcji
//...
void brightChanger(PWM * ptr, uint8_t bright) //funkcja pobierająca wskaźnik do struktury
{
if(ptr->bright != bright)
{
ptr->bright = bright;
ptr->lock = 1;
}
}
//2
#define BRIGHT 0
#define LOCK 1
//
void brightChanger(uint8_t diode_nr, uint8_t bright)
{
if(pwm[diode_nr][BRIGHT]!=bright)
{
pwm[diode_nr][BRIGHT]=bright;
pwm[diode_nr][LOCK]=1;
}
}[/syntax]

Statystyki: Napisane przez Krauser — 24 kwi 2013, o 17:05

Re: Obsługa kilku przycisków

2013-04-23T21:34:12+01:00

Dzięki za odp.
Ale brightChanger jest raczej ok, bo pilotem działa zmiana kilku kanałów na raz, a zresztą nie ma co tu być "skopane":[syntax=c]void brightChanger(uint8_t diode_nr, uint8_t bright)
{
if(pwm[diode_nr][0]!=bright)//jeśli jasność aktualna inna od rządanej
{
pwm[diode_nr][1]=bright;
pwm[diode_nr][2]=1;
}
}[/syntax]W pętli głównej jest sprawdzane czy trzeba zmienić wypełnienie i jeśli tak, to zmiana z odpowiednim opóźnieniem do czułości oka.

Statystyki: Napisane przez żbik — 23 kwi 2013, o 21:34

Re: Obsługa kilku przycisków

2013-04-23T18:23:08+01:00

Kod jest w porządku. Może błąd jest w funkcji brightChanger.

PS.
Wygodniej zamiast tablicy 2 wymiarowej byłoby użyć tablicy struktur.

Statystyki: Napisane przez Krauser — 23 kwi 2013, o 18:23

Obsługa kilku przycisków

2013-04-22T22:49:24+01:00

Chcę zrobić nieblokującą obsługę kilku przycisków na raz i prawie działają moje wypociny, ale nie ogarniam, dlaczego nie chcą działać na raz. Tzn Jeśli wcisnę przycisk 2 i przytrzymam, to wciskając 1 wywołuje się zmiana ( w tym przypadku wypełnienia PWM ) dla obu kanałów, ale trzymając 1, wciskanie drugiego nie działa i wciskając oba na raz działa tylko pierwszy.
Utworzyłem tablicę "ButtonTable[5][4]" gdzie:
[x][0] = pin przycisku np. (1<[x][1] = flaga kierunku - jaśniej / ciemniej
[x][2] = debounce
[x][3] = flaga czy wciśnięty

[syntax=c]volatile uint8_t buttonTable[5][4];
.
.
.
//w pętli głównej sprawdzanie przycisków:
.
.
ButtonChecker();
.
.
.

//sama funkcja:
void ButtonChecker()
{
for (uint8_t i = 0; i < 5; i++)//przelatujemy po wszystkich przyciskach
{
if (!(PIND & (buttonTable[i][0])))// jeśli wciśnięty
{
if ((!buttonTable[i][2]) && (!buttonTable[i][3]))//jeśli debounce=0 i flaga nie wciśnięty
{
buttonTable[i][3]=1;//wciśnięty
if (buttonTable[i][1]==0)//kierunek
{
brightChanger(i,255);//jaśniej
buttonTable[i][1]=1;//zmiana kierunku
}
else
{
brightChanger(i,0);//ciemniej
buttonTable[i][1]=0;//zmiana kierunku
}
}
}
else//jeśli przycisk nie wciśnięty
{
buttonTable[i][3]=0;//zerujemy
buttonTable[i][2]=5;//debounce 50ms, wojna z timerem jak nie wciśnięty;)
}
}
}

//dalej timer zmniejszający debounce

ISR(TIMER2_COMP_vect)
{
uint16_t n;

for ( uint8_t i = 0; i < 5; i++)
{
n=buttonTable[i][2];
if (n) buttonTable[i][2] = --n;
}
}[/syntax]

Gdzie zbłądziłem?
A może w ogóle nie tędy droga?

pzdr

Statystyki: Napisane przez żbik — 22 kwi 2013, o 22:49