Odpowiadając: warunek z linii nr.2 nie zostanie spełniony ponieważ key_state zostało z wartością 1.
tak tak - dlatego skasowałem wcześniejszą wypowiedź zanim napisałeś swoją ... teraz jeszcze patrzę na to i za chwilę napiszę kilka słów
------------------------ [ Dodano po: 13 minutach ]
Po pierwsze to ogromna radość, że TAK głęboko wszedłeś w tę funkcję ... to mnie bardzo cieszy.
Po drugie - rzeczywiście w pewnych sytuacjach może dojść do tego co mówisz - chociaż ja tu założyłem właśnie możliwości REGULACJI tym doborem czasu na różnych etapach .... Nie mniej jednak biorąc pod uwagę aż takie coś:
haloon napisał(a):
a w wersji hardcorowej przy kątowym atakowaniu słicza z dużą szybkością, nawet poniżej 10ms.
No to rzeczywiście .... jak się "postarać" to można.
Ja bym może dodał nieco krótszy ten warunek .... bo nie musimy czekać aż Timer1 będzie równy 0, tylko od razu i ZAWSZE gdy klawisz zwolniony a stan jest = debounce to .... zrobiłbym to tak
[syntax="c"]
...
if( key_press && !*key_state ) {
*key_state = debounce;
Timer1 = 15;
} else
if( *key_state ) {
if (!key_press && debounce==*key_state ) {
Timer1 = 15; // zamiast idle - podtrzymujemy czas debounce, dzięki czemu można go chyba fajnie skrócić do minimum
} else
if( key_press && debounce==*key_state && !Timer1 ) {
*key_state = 2;
Timer1=5;
} else
if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc(); /* KEY_UP */
*key_state=idle;
} else
...[/syntax]
------------------------ [ Dodano po: 18 minutach ]
Generalnie - to powiem tak - CELEM albo inaczej GŁÓWNE CELE jakie mi przyświecały pisząc tę funkcję to:
1. Pokazać czytelnikowi jak działają timery programowe
2. pokazać jak należałoby próbować podchodzić do obsługi klawisza (szczególnie chodzi o repetycję)
3. pokazać jak korzystać w prosty sposób ze wskaźników na funkcje przekazywanych do innej funkcji - czyli pierwsze zetknięcie z TAK WAŻNYMI z mojego punktu widzenia CALLBACKAMI (czyli mechanizmem funkcji zwrotnych)
4. pokazać jak pisać funkcje nieblokujące z nieco rozbudowaną tzw maszyną stanów
5. Na pewno nie miała to być NAJLEPSZA FUNKCJA na świecie do obsługi klawiszy i nawet pomimo tej poprawki nadal wg mnie nie będzie pretendować do tego tytułu .... bo obsługuje wciskanie tylko pojedynczych klawiszy (dwóch naraz nie da się wykryć)
a też np by się przydało w wielu sytuacjachJak dla mnie punkty od 1 do 4 kolega zaliczył na piątkę ! ... dlatego piszę, że to miło słyszeć - że się przydało to co napisałem
------------------------ [ Dodano po: 22 minutach ]
O! nawet jeszcze inaczej nieco bym to zrobił - tego IF'a zabezpieczającego umieściłbym na końcu łańcuszka - aby nie tracić cennego czasu funkcji nieblokującej!
[syntax="c"]
...
if( key_press && !*key_state ) {
*key_state = debounce;
Timer1 = 15;
} else
if( *key_state ) {
if( key_press && debounce==*key_state && !Timer1 ) {
*key_state = 2;
Timer1=5;
} else
if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc();/* KEY_UP */
*key_state=idle;
} else
if( key_press && go_rep==*key_state && !Timer1 ) {
*key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==*key_state && !Timer1 ) {
*key_state = rep;
} else
if( key_press && rep==*key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc();/* KEY_REP */
} else
if (!key_press && debounce==*key_state ) {
Timer1 = 15; // ***** podtrzymujemy czas debounce na wypadek dłuższej niezamierzonej przerwy *********
}
[/syntax]
Statystyki: Napisane przez mirekk36 — 29 mar 2015, o 23:39
]]>