Witam wszystkich!

Zafascynowała mnie funkcja do obsługi drgań styków z forum Mirka.
Chciałbym ją zmodyfikować w ten sposób żeby zwracała mi rezultat w zależności od wciśniętego klawisza. Jednak napotkałem problem którego chyba nie uda mi się rozwiązać bez Waszej pomocy.
Otóż zmodyfikowana funkcja wygląda jak poniżej:



Problem jest w tym, że przycisk nie reaguje prawidłowo. Tzn. dioda zmienia stan na przeciwny, ale nie za każdym razem np po dwóch naciśnięciach.

Postanowiłem zrobić mały test.
Użyłem dodatkowo jednej diody żeby zobaczeć co się dzieje w poszczególnych fragmentach funkcji


w tym przypadku dioda pomocnicza działa prawidłowo, za każdym wciśnięciem reaguje. A w funkcji głównej nie działa dioda 2. tzn działa ale dioda pomocnicza zmienia się tak jak powinna (za każdym nacisnieciem) a dioda led1 za np 2 albo 3

Poniżej funkcja main



Proszę, nakierujcie mnie gdzie robię błąd:)

Mam pytanie masz zdefiniowany jeden ,czy dwa przyciski?

zdefiniowane oba. Ustawione jako wejscia i podciagniete

Sorry z kodu wynika , że masz dwa klawisze .A diody nie zmieniają stanu jak naciskasz klawisz szybko , wolno , czy to nie ma znaczenia ?.

chyba to nie jasno opisalem. dioda 3 reaguje prawidlowo. tzn za kazdym przycisnieciem zmienia stan na przeciwny. diody 1 i 2 reaguja ale za dwoma albo trzema nacisnieciami. czy szybko czy wolno naciskam jest to samo

Z ciekawości przetestowałem twój program i faktycznie objawy są takie jak opisałeś. W czasie szukania przyczyn takiego działania zakomentowałem jeden warunek w pętli while i dioda z drugiego warunku zaczęła działać prawidłowo. To nasunęło mi odpowiedź dlaczego jeden warunek działa dobrze a dwa już nie.

Załóżmy, że wciśnięty jest klawisz 1 a program zaczyna sprawdzać drugi warunek w pętli while. Po wejściu do funkcji wykona się pierwszy warunek if

Zmienna wynik zostanie ustawiona na jeden i tą wartość zwróci funkcja.
Po wyjściu z funkcji ta wartość zostanie przyrównana do 2 (bo program sprawdzał drugi warunek w while).
Ten if się nie wykona bo funkcja zwróciła wartość 1.

Chwilę później program zacznie sprawdzać pierwszy warunek w while. Wewnątrz funkcji, pierwszy if już nie będzie spełniony bo chwilę wcześnie zmienna key_lock1 została ustawiona na 1. Klawisz jest nadal wciśnięty więc żaden warunek wewnątrz funkcji nie będzie spełniony i funkcja zwróci 0.

A dlaczego zero skoro wcześniej zmienna wynik została ustawiona na 1? Ano dlatego, że zmienna wynik to zmienna lokalna, której wartość jest tracona po wyjściu z funkcji.

W poprzednim wywołaniu funkcji jej wartość została ustawiona na 1. I ta wartość została zwrócona. Jednak po ponownym wywołaniu funkcji zmienna wynik została utworzona na nowo i zainicjowana wartością 0. Skoro żaden warunek wewnątrz funkcji się nie wykona to funkcja zwróci 0. Ta wartość zostanie porównana z wartością równą 1 (pierwszy warunek w while).
I tym sposobem żadna dioda nie zmieni swego stanu.

Program dobrze zadziała jeśli podczas wciśnięcia któregoś z klawiszy najpierw zostanie sprawdzony warunek (w pętli while) odpowiadający temu klawiszowi.

W pętli głównej.
Myślę, że powinieneś powołać sobie jeszcze jedną zmienną i do niej przypisywać to co zwraca funkcja press1().
Dopiero po tym "zabiegu" sprawdzać warunki.

Ja bym to zrobił tak:


Poza tym dla lepszej czytelności kodu #define LED1_TOG zamiast #define led1_tog, oraz if(1 == temp) zamiast if(temp == 1).
Myślę, że lepiej jest również zastosować uint16_t dla key_lock1 i key_lock2 zamiast uint8_t.

Dzieki za pomoc!
Jak wrócę do domu to pokombinuję i zobaczę co się dzieje.

tylko, że takie coś:



spowoduje, że albo jedna albo druga dioda będzie świecić połową swojej jasności czyli 50% PWM, zamiast zmienić swój stan po wciśnięciu klawisza.....

trzeba jeszcze ew kasować stan:



to ma szanse działać być może zgodnie z oczekiwaniami autora, ale powiem tak .... te funkcje z bloga były myślane tak aby stosować je nieco inaczej ... przede wszystkim do obsługi kilku pojedynczych klawiszy, a robienie klawiatury - hmmm myślę że warto byłoby wtedy podejść całkowicie inaczej ...

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Przetestowałem i diody świecą zgodnie z oczekiwaniem. Wciskamy klawisz pierwszy i dioda zmienia stan na przeciwny. Wciskamy klawisz drugi i druga dioda zmienia stan na przeciwny. Diody świecą pełnym światłem.



W funkcji press1() jest zerowanie zmiennej wynik, gdy żaden z klawiszy nie zostanie wciśnięty to funkcja ta zwraca nam wartość 0. Czyli zmienna temp w wyniku wywołania funkcji otrzyma wartość 0. Żaden z warunków nie będzie spełniony, diody zmienią stan dopiero po wciśnięciu klawiszy.

Panie Mirku, dlaczego pana zdaniem diody powinny świecić połową jasnośći?
Wydaje mi się, że świecą pełną jasnością. No chyba, że mi się tylko wydaje!

bo nie zauważyłem tego



Wcale ci się nie wydaje ... masz rację

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Dziękuje! Pozdrawiam serdecznie!