ATNEL tech-forum

Re: Obsługa klawisza

2015-05-15T17:06:49+01:00

jerrylu napisał(a):

Pozostaje obsługa przytrzymania przycisku jak napisałem.
Ja w tym wątku nie mam już nic do dodania.

Oj tam - jesteś świetnym obserwatorem i dokumentalistą.
Kol. kamer uzyskał świetny pseudo random generator, a nie było to Jego zamiarem. Ty podpowiadasz, że ma wyjść z wartością 0 gdy przycisk ciągle wduszony, wtedy nie będzie "liczba" się zmieniać - mówię i ja.

Statystyki: Napisane przez Wirnick — 15 maja 2015, o 17:06

Re: Obsługa klawisza

2015-05-15T15:46:08+01:00

No właśnie chyba się nie zrozumieliśmy. Proszę o wybaczenie.
A błąd Koledze kamer wskazałem 2 razy - wczoraj bez rozwinięcia, dzisiaj z rozwinięciem - godz. 08:57
Co więcej sprawdziłem, że na 16MHz pętla z inkrementacją przepełniającą key_lock trwa ok. 22ms.
Czyli bez dodatkowych zmiennych i sprawdzania warunków w jej obiegu (jak wspomniałem powyżej) nie ma siły,
żeby nie poleciał ciąg kolejnych cyfr na plus, bądź na minus w zależności od klawisza.
W końcu to tylko 1/50 sekundy i warunek ponownie jest spełniony do inkrementacji/dekrementacji licznika.
Delay aż 200ms w pierwszym warunku funkcji pozwala spokojnie zwiększać i zmniejszać licznik ale tylko przy krótkich naciśnięciach.
Ale to tylko rozwiązanie dla pokazania, że podstawowa funkcjonalność programu jest zachowana.
Pozostaje obsługa przytrzymania przycisku jak napisałem.
Ja w tym wątku nie mam już nic do dodania.

Statystyki: Napisane przez jerrylu — 15 maja 2015, o 15:46

Re: Obsługa klawisza

2015-05-15T10:43:10+01:00

Chyba się nie zrozumieliśmy. Zainteresowany nie chce Twojego gotowca. Chce dojść do błędu - którego nie wskazałeś.

Statystyki: Napisane przez Wirnick — 15 maja 2015, o 10:43

Re: Obsługa klawisza

2015-05-15T10:25:31+01:00

Przecież zwracało prawdę i była inkrementacja/dekrementacja przy wciśnieciu klawisza - poprawne działanie.
Problemem było to, że przy dłuższym przytrzymaniu przycisku szła seria tych inkrementacji/dekrementacji.

Statystyki: Napisane przez jerrylu — 15 maja 2015, o 10:25

Re: Obsługa klawisza

2015-05-15T10:05:35+01:00

jerrylu napisał(a):

Z uporem szukasz dziury w działających poprawnie fragmentach kodu kolegi

Na pewno to widzisz? - Ta funkcja ( klawisz_wcisniety( KEY1_PIN) ) ma zwrócić prawdę, by wykonało się "liczba+=1"
A tak nie było

Statystyki: Napisane przez Wirnick — 15 maja 2015, o 10:05

Re: Obsługa klawisza

2015-05-15T09:42:01+01:00

"klawisz_wcisniety" to funkcja, do funkcji jest przekazywany klawisz jako parametr. Dopiero w funkcji sprawdzany jest jego stan.
Z uporem szukasz dziury w działających poprawnie fragmentach kodu kolegi

Statystyki: Napisane przez jerrylu — 15 maja 2015, o 09:42

Re: Obsługa klawisza

2015-05-15T09:30:31+01:00

kamer napisał(a):

Ale nadal nie rozumiem dlaczego mój prosty kod nie działał. Coś musiało być nie tak skoro teraz działa.
pozdrawiam
Łukasz

Popatrzmy na funkcje:
[syntax=c]if( klawisz_wcisniety( KEY1_PIN) ) liczba+=1; //sprawdzenie czy klawisz 1 wciśnięty[/syntax]
Funkcja if wykona się, gdy argument jest prawdą - i tak jest - PRAWDA = ( klawisz_wcisniety( KEY1_PIN) ), a powinna być FALSE (klawisz puszczony).
Sprawdź taką drobnostkę - ( klawisz_wcisniety(~ KEY1_PIN) ) lub (~ klawisz_wcisniety( KEY1_PIN) ).

Statystyki: Napisane przez Wirnick — 15 maja 2015, o 09:30

Re: Obsługa klawisza

2015-05-15T08:57:17+01:00

Tak jak pisałem, nie zdajesz sobie sprawy z szybkości pętli.
Pętla while + inkrementacja key_lock do 255 i zmiana na zero wykręca się b. szybko - 20ms?
I masz znowu spełniony pierwszy warunek w funkcji, która zwraca Ci 1 i zwieksza/zmniejsza licznik wyświetlacza.
20ms to 1/50 sekundy, czyli już b. krótkie nacisnieęie zmieni licznik o 1. Dłuższe o serię następnych.
Przytrzymaj przycisk przez 10 sekund. Zobacz ile Ci nabije licznik.
Podziel 10sek przez to co nastukało ci na liczniku i masz czas obiegu pętli z przepełnieniem key_lock.

Pozdrówka!

------------------------ [ Dodano po: 5 minutach ]

Dałoby się obsłużyć tą sytuację w Twojej prostej funkcji, ale wprowadzając kolejne zmienne.
Zmienna na przepełnienie licznika + zmienna zmiany stanu klawisza.
Jeśli nastąpiło przepełnienie key_lock bez zmiany stanu klawisza (klawisz cięgle wciśnięty) - zwróć zero.

Statystyki: Napisane przez jerrylu — 15 maja 2015, o 08:57

Re: Obsługa klawisza

2015-05-15T06:19:13+01:00

Podciągnięcie do 5V:

[syntax=c]KEY_DIR &= ~(KEY1_PIN | KEY2_PIN);//piny są wyjściowe
KEY_PORT |= KEY1_PIN | KEY2_PIN;//podciągnięte do 5V[/syntax]

------------------------ [ Dodano po: 12 minutach ]

jerrylu:
Sprawdziłem kod od Ciebie - działa jak należy Zmieniłem jedynie funkcję obsługi drugiego przycisku tak by też działał przy wciśnięciu a nie zwolnieniu. Dzięki wielkie za pomoc i poświęcony czas.

Ale nadal nie rozumiem dlaczego mój prosty kod nie działał. Coś musiało być nie tak skoro teraz działa.

pozdrawiam
Łukasz

Statystyki: Napisane przez kamer — 15 maja 2015, o 06:19

Re: Obsługa klawisza

2015-05-15T06:14:28+01:00

kamer napisał(a):

Przyciski, tak jak kolega wspomniał, są zadeklarowane w pliku d_led.h i działają poprawnie.
Wydaje mi się że wszystko rozumiem z tego kodu i że powinno działać ok, a tak nie jest.

Hmm. Deklaracje są, reset ustawia wejścia, a podciąganie wejść jak rozwiązałeś?

Statystyki: Napisane przez Wirnick — 15 maja 2015, o 06:14

Re: Obsługa klawisza

2015-05-14T23:43:48+01:00

Zapodaję Ci przetestowany kod nieblokującej obsługi klawiszy pod Twój przykład na bazie kodu MK:

[syntax=c]/*
* main.c
*
* Created on: 2010-03-30
* Autor: Mirosław Kardaś
* modified by jerrylu for kamer
*/
#include // dołączenie głównego systemowego pliku nagłówkowego
#include
#include

#include "d_led.h"// dołączenie naszego pliku nagłówkowego (obsługa LED)

#define KEY1 (1<#define KEY2 (1<
void licz_inc(void);
void licz_dec(void);
uint8_t key1_lock, key2_lock;

uint16_t liczba=2015; //deklaracja zmiennej liczba

void licz_inc(void) {
liczba++;
}

void licz_dec(void) {
liczba--;
}

void key_press( uint8_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) ) ;
void key_push_up( uint8_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) ) ;

//**************** OBSŁUGA DWÓCH KLAWISZY za pomocą 2 funkcji

int main(void) {

PORTD |= KEY1|KEY2; // podciągamy linie klawiszy do VCC
d_led_init(); //inicjalizacja wyświetlacza
_delay_ms(10);

sei();

while(1) {

key_press( &key1_lock, &PIND, KEY1, licz_inc );

key_push_up( &key2_lock, &PIND, KEY2, licz_dec );

if(liczba>9999) liczba=9999; //zabezpieczenie przed przekroczeniem zakresu wyświetlacza
if(liczba<0) liczba=0; //zabezpieczenie przed przekroczeniem zakresu wyświetlacza
cy1=liczba/1000; //obliczenie pierwszej cyfry do wyświetlenia
cy2=(liczba-1000*cy1)/100; //obliczenie drugiej cyfry do wyświetlenia
cy3=(liczba-1000*cy1-100*cy2)/10; //obliczenie trzeciej cyfry do wyświetlenia
cy4=(liczba-1000*cy1-100*cy2-10*cy3);
}
}

void key_press(uint8_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) ) {

register uint8_t key_press = (*KPIN & key_mask);

if( !*klock && !key_press ) {
*klock=1;

// reakcja na PRESS (wcinięcie przycisku)
if(kfun) kfun();

} else if( *klock && key_press ) (*klock)++;
}

void key_push_up(uint8_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) ) {

register uint8_t key_press = (*KPIN & key_mask);

if( !*klock && !key_press ) *klock=1;
else if( *klock && key_press ) {
if( !++*klock ) {

// reakcja na PUSH_UP (zwolnienie przycisku)
if(kfun) kfun();
}
}
}[/syntax]

Uwaga na inne piny - u mnie PD0 i PD1 - zmieniasz na swoje.
PORTD i PIND zmieniasz na PORTC i PINC.

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 23:43

Re: Obsługa klawisza

2015-05-14T21:32:20+01:00

Masz Bluebooka, że tak zapytam?
Jak go masz, to masz i biblioteki, możesz zrobić obsługę nieblokującą przycisków wg Mirka.
Jeśli nie, to pozostaje Ci na początek standardowa obsługa "drgań styków" na delay-ach.
Możesz oczywiście znależć bez problemu przykłady w sieci nieblokującej obsługi przycisków na timerach i przerwaniach.
Nie ma siły, aby zrobić nieblokującą obsługę bez tych drugich - oprócz koncepcji Mirka z przekazywaniem zewnętrznych funkcji do funkcji obsługi klawisza:
[syntax=c]key_press( &key1_lock, &PINC, KEY1, change_led1 );[/syntax]
Drugie pytanie, dalej nie rozumiesz, dlaczego zmienia Ci się licznik o ileś tam cyfr przy przytrzymaniu przycisku?

------------------------ [ Dodano po: 27 minutach ]

Ponieważ multipleksujesz wyswietlacze wstaw sobie tam swoją funkcję i będzie ok zapewne (jak jest wystarczająco często odpalane przerwanie).
Oczywiscie musisz dać deklarację volatile dla funkcji i zmiennych tam wykorzystywanych.

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 21:32

Re: Obsługa klawisza

2015-05-14T20:38:49+01:00

Przyciski, tak jak kolega wspomniał, są zadeklarowane w pliku d_led.h i działają poprawnie.

[syntax=c]#define KEY1_PIN (1<#defineKEY2_PIN (1<#defineKEY_PORT PORTC
#defineKEY_DIR DDRC[/syntax]

Co do opóźnień to nie widzę sensu ich użycia - one jedynie opóźnią narastanie/opadanie licznika a nie o to chodzi. Chciałbym żeby przycisk wciśnięty i trzymany powodował tylko jedną zmianę licznika.
Wydaje mi się że wszystko rozumiem z tego kodu i że powinno działać ok, a tak nie jest.

Statystyki: Napisane przez kamer — 14 maja 2015, o 20:38

Re: Obsługa klawisza

2015-05-14T19:48:27+01:00

Przecież przyciski działają, jak Kolega napisał. Definicja pewnie w d_led.h.

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 19:48

Re: Obsługa klawisza

2015-05-14T18:14:54+01:00

kamer napisał(a):

Zwracam 0 tak długo aż licznik key-lock przepełni rozmiar zmiennej czyli 255. Dopiero wtedy key_lock jest 0 i sprawdzany jest przycisk w pierwszym warunku if funkcji.

A pro po - na którym pinie i jakiego portu masz KEY1_PIN bo nie widzę definicji klawiszy - czy to kompiluje się?

Statystyki: Napisane przez Wirnick — 14 maja 2015, o 18:14

Re: Obsługa klawisza

2015-05-14T17:43:31+01:00

Chyba nie zdajesz sobie sprawy z szybkości tych pętli.
Dlatego przy wyzerowaniu masz znowu zwróconą jedynkę i inkrementację/decrementację licznika przy cały czas wciśniętym przycisku.
Daj sobie delay w głównej, to zobaczysz zmianę

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 17:43

Re: Obsługa klawisza

2015-05-14T17:26:10+01:00

Zwracam 0 tak długo aż licznik key-lock przepełni rozmiar zmiennej czyli 255. Dopiero wtedy key_lock jest 0 i sprawdzany jest przycisk w pierwszym warunku if funkcji.

Statystyki: Napisane przez kamer — 14 maja 2015, o 17:26

Re: Obsługa klawisza

2015-05-14T17:13:53+01:00

Masz racje, brak wykrzyknika.
Jednak praca nie jest dobrym miejscem do takich analiz. Potrzeba spokoju.
Zwróć uwagę, że zwracanie 0 przez funkcje nic nie zmienia w Twoim programie.
W kolejnym obiegu masz znowu zwracaną jedynkę. Itd - 0/1/0/1

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 17:13

Re: Obsługa klawisza

2015-05-14T17:00:02+01:00

Żeby drugi warunek był spełniony i licznik key-lock był inkrementowany musi być też zwolniony przycisk ( key_lock && (PINC & klawisz ) ).
Wydaje mi się, że powinno to działać na jednej zmiennej, zakładając że w jednej chwili naciskam tylko jeden przycisk.

Statystyki: Napisane przez kamer — 14 maja 2015, o 17:00

Re: Obsługa klawisza

2015-05-14T16:34:42+01:00

Sorry odnośnie deklaracji, jest ona u Ciebie też poza ciałem funkcji, ale tak wrzucona, że moje stare oczy to przegapiły.
Powinna być dla porządku na górze programu.
Niestety jestem w pracy i rzucam tylko w wolnej chwili okiem na Twój kod.
Zaraz się przyjrzę dokładniej.

------------------------ [ Dodano po: 10 minutach ]

Ale jak key_lock = 1
to zawsze masz spełniony drugi warunek i zawsze masz dekrementację licznika i inkrementację key_lock

------------------------ [ Dodano po: 16 minutach ]

Raczej musisz pójść drogą dwóch zmiennych dla obydwu klawiszy - modyfikacja kodu Mirka:
http://mirekk36.blogspot.com/2012/10/ob ... w-cd2.html

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 16:34

Re: Obsługa klawisza

2015-05-14T16:03:45+01:00

Ale dlaczego program zwraca mi 1 gdy przycisk jest wciśnięty a zmienna key_lock jest różna od 0?
W takim wypadku program nie powinien wejść do pętli if( !key_lock && !(PINC & klawisz)) a powinien zwracać 0.

Statystyki: Napisane przez kamer — 14 maja 2015, o 16:03

Re: Obsługa klawisza

2015-05-14T15:43:12+01:00

Ależ wręcz przeciwnie.
Program zauważa, że masz wciśnięty przycisk cały czas,
a ponieważ wywołuje ciągle w pętli funkcję sprawdzania klawisza - "klawisz_wcisniety",
ciągle zwraca też 1, co powoduje inkrementację licznika.
Nie wykorzystujesz wcale zmiennej key_lock, tak w ogóle, jeśli chcesz z niej korzystać w funkcji main i klawisz_wcisniety,
zadeklaruj ją przed main.

------------------------ [ Dodano po: 2 minutach ]

Przy zmniejszaniu podobnie, dostajesz ciągle 0 i dekrementujesz z szybkoscią obiegu pętli while.

Statystyki: Napisane przez jerrylu — 14 maja 2015, o 15:43

Re: Obsługa klawisza

2015-05-14T14:39:08+01:00

Cześć wszystkim,
Jestem początkujący jeżeli chodzi o programowanie, więc proszę o wyrozumiałość:)
Modyfikuję właśnie program dotyczący obsługi wyświetlacza 7segmentowego z BB. Dopisałem funkcję która obsługuje dwa przyciski i w zależności od ich stanu zwiększa lub zmniejsza wyświetlaną liczbę.
Napotkałem problem przy obsłudze przycisku - program nie zauważa, że przycisk jest ciągle wciśnięty i zmienia mi wartość liczby o wiele więcej niż o 1 podczas jednego naciśnięcia.
Do zabezpieczenia funkcji użyłem fragmentu kogu z artykułu Pana Mirka o obsłudze klawiszy, ale widocznie nie zrozumiałem czegoś do końca bo ciągle występuję ten błąd.
Kod poniżej:

[syntax=c]* main.c
*
* Created on: 13 maj 2015
* Author: Luk
*/

#include //biblioteka systemowa avr
#include //biblioteka opóźnień
#include //biblioteka przerwań

#include "d_led.h"//załącznie pliku nagłówkowego obsługi led

uint8_t klawisz_wcisniety(uint8_t klawisz);//deklaracja funkcji sprawdzającej przycisk

int main(void){

d_led_init(); //inicjalizacja wyświetlacza

uint16_t liczba;//deklaracja zmiennej liczba
liczba=2015; //wartość początkowa zmiennej liczba

sei(); //globalne załączenie przerwań

while(1){

if( klawisz_wcisniety( KEY1_PIN) ) liczba+=1; //sprawdzenie czy klawisz 1 wciśnięty
if( klawisz_wcisniety( KEY2_PIN) ) liczba-=1; //sprawdzenie czy klawisz 2 wciśnięty
if(liczba>9999) liczba=9999;//zabezpieczenie przed przekroczeniem zakresu wyświetlacza
if(liczba<0) liczba=0;//zabezpieczenie przed przekroczeniem zakresu wyświetlacza
cy1=liczba/1000;//obliczenie pierwszej cyfry do wyświetlenia
cy2=(liczba-1000*cy1)/100;//obliczenie drugiej cyfry do wyświetlenia
cy3=(liczba-1000*cy1-100*cy2)/10;//obliczenie trzeciej cyfry do wyświetlenia
cy4=(liczba-1000*cy1-100*cy2-10*cy3);//obliczenie czwartej cyfry do wyświetlenia
}
}

uint8_t key_lock;//deklaracja zmiennej pomocniczej do sprawdzenia stanu przycisku
uint8_t klawisz_wcisniety(uint8_t klawisz){

if( !key_lock && !(PINC & klawisz)){// sprawdzenie czy zmienna pomocnicza=0 oraz czy przycisk jest wciśnięty
key_lock=1;// ustawienie zmiennej pomocniczej na 1
return 1;// funkcja zwraca wartość 1
}

else if( key_lock && (PINC & klawisz ) ) key_lock++;//sprawdzenie czy zmienna pomocnicza jest różna od 0 oraz czy klawisz jest
return 0;// funkcja zwraca wartość 0
}[/syntax]

Czy możecie mi podpowiedzieć co robię źle?

pozdrawiam
Łukasz

Statystyki: Napisane przez kamer — 14 maja 2015, o 14:39

Re: Obsługa klawisza

2014-05-28T10:11:26+01:00

Wypróbuj poniższy prościutki programik.
[syntax=c]/*
* main.c
* F_CPU: 8000000Hz
* MCU: Atmega32
*
* Created on: 2014-05-28
* Author: Jacek
*/

#include
#include
#include

#define LED_PIN (1<#define LED_ONPORTC &=~LED_PIN
#define LED_OFFPORTC |= LED_PIN
#define LED_TOGPORTC ^= LED_PIN

#define KEY_PIN(1<
volatile uint8_t Timer1;//zmienna na potrzeby timera programowego
uint16_t key_lock;//zmienna na potrzeby obsługi klawisza
uint8_t tryb;

void key_press(uint16_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) );
void zmien_tryb(void);

int main(void)
{

DDRC |= LED_PIN; //ustawienie kierunku pinu jako wyjscie
LED_OFF;

DDRC &=~ KEY_PIN; //ustawienie kierunku pinu jako wejscie
PORTC|= KEY_PIN; //podciągnięcie programowe linii klawisza do VCC

//konfiguracja Timer2 - timery programowe, tyknięcie timera co 10ms
TCCR2 |= (1<TCCR2 |= (1<OCR2 = 78;// dodatkowy podział przez 78 (rej. przepełnienia)
TIMSK |= (1<
sei();

while(1)
{
key_press(&key_lock, &PINC, KEY_PIN, zmien_tryb);

if(0 == tryb) LED_OFF;
else if(1 == tryb) LED_ON;
else if(2 == tryb)
{
if(!Timer1)
{
Timer1 = 100;//100 * 10ms = 1s
LED_TOG;
}
}
else if(3 == tryb)
{
if(!Timer1)
{
Timer1 = 10;//10 * 10ms = 100ms
LED_TOG;
}
}

}//end while
}//end main

void zmien_tryb(void)
{
tryb++;
if(tryb > 3) tryb = 0;
}

ISR(TIMER2_COMP_vect)
{
uint16_t n;

n = Timer1;/* 100Hz Timer1 */
if (n) Timer1 = --n;

}

void key_press(uint16_t * klock, volatile uint8_t * KPIN, uint8_t key_mask, void (*kfun)(void) )
{
register uint8_t key_press = (*KPIN & key_mask);

if( !*klock && !key_press )
{
*klock=50000;

// reakcja na PRESS (wcinięcie przycisku)
if(kfun) kfun();

} else if( *klock && key_press ) (*klock)++;
}[/syntax]

Do obsługi klawisza zastosowałem funkcję, którą Mirek opisał na swoim blogu. Zamiast opóźnień typu delay zastosowałem timery programowe.

Potraktuj ten program jako zachętę do własnych eksperymentów Miłej zabawy

Statystyki: Napisane przez jacekk232 — 28 maja 2014, o 10:11

Re: Obsługa klawisza

2014-05-27T00:11:32+01:00

Troszke naprowadzenia masz tutaj:
http://mirekk36.blogspot.com/2012/10/dr ... ak-to.html
To taka wedka Zamiast wylaczania,wlaczania diody zrob po prostu zwiekszanie/zmniejszanie czestotliwosci

W BlueBooku o ile pamietam to w okolicy opisywania SuperDebouncer troche bylo na ten temat (ale moge sie mylic )

Statystyki: Napisane przez VDR — 27 maja 2014, o 00:11