Witam.
Piszę sobie mały programik do obsługi klawisza i ledów z zestawu ATB i mam problem , bo nie wiem czy ja coś źle robię, czy moja funkcja jest abstrakcją jak obrazy niektórych malarzy. Czy to w ogóle ma prawo działać? Przytoczę przykład -ciało funkcji dla lepszego zobrazowania sytuacji:



wszystko jest pisane dosyć liniowo, ponieważ uczę się posługiwania funkcjami i if-ami, ale jeśli moja funkcja migacz_foor(); jest funkcją typu void i jest wpisana do funkcji, która także jest typu void, to czy to w ogóle będzie działaś? Funkcja migacz_foor() jest typowym robakiem, czyli przesuwa zapaloną diodę z lewa na prawo. w

_________________
Nikt nie mówił,że będzie łatwo...

funkcja migacz_foor() zadziała gdy KLAWISZ_WCISNIETY da wartość prawda czyli >0, tylko pytanie kiedy się to wykona...
u ciebie jest tak :




nic nie piszesz o podłaczeniu hardware przyciski diody czy co tam masz dalej pomagać bedzie trudno .....
ps do umieszczenia kodów używaj znaczników syntax=c /syntax

podłączenie diod jak w zestawie ATB czyli katodą do uC, załączają się stanem niskim. kod już poprawiłem.

_________________
Nikt nie mówił,że będzie łatwo...

OK ale podpiąć możesz do dowolnego portu ATMEGI ...
napisz proszę Ledy port B przycisk pin PC2 czy jak ?

Najlepiej daj cały kod ...

funkcja void może być wywoływana w innej funkcji.
przecież funkcja main jest typu void (na AVR oczywiście).

ok, już kopiuję cały kod, chociaż nie chętnie, ponieważ nie chciał bym, aby inni mnie oskarżyli o pójście na łatwiznę . Nie oczekuję konkretnego rozwiązania, ponieważ w taki sposób się nie nauczę pisać kodu.

Pytałem o to , czy funkcje jedna w drugiej będą działać, ponieważ chciałem sobie uprościć to co jest w pętli głównej i zastąpić większość kodu nowymi dwiema funkcjami (od pierwszego klawisza i drugiego). Ogólnie rzecz biorąc, to chcę uzyskać efekt następujący:
Gdy wcisnę klawisz pierwszy, to ma mi się wykonać robal trzy razy, to samo na drugim klawiszu. Jednak nie jest to takie proste dla mnie. Gdy napiszę instrukcję dla jednego klawisza, to działa pięknie, problem mam też z tym, aby w drugim klawiszu to samo się wykonywało.
Podaję przykład jaka jest moja koncepcja na dwa klawisze, niestety po wgraniu kodu nie działa mi to poprawnie :


Czy mogę połączyć obsługę dwóch klawiszy operatorem iloczynu && ?


Gdy klawisze nie są wciśnięte, to PORTB ma stan 0,
gdy wcisnę klawisz i osczekam 80ms , sprawdzę stan klawisza, to robal wykona się trzy razy. i koniec:)

Dodam jeszcze,że myślałem o koncepcji drugiej :
if.........
else if..........
else if.........

------------------------ [ Dodano po: 14 minutach ]

taka koncepcja, ale w niej działa tylko pierwszy klawisz, drugi niestety nie uruchamia robala, tylko dlaczego?


po kompilacji błędy n ie wyskoczyły. a mimo to nie działa.

------------------------ [ Dodano po: 30 minutach ]

:D EUREKA!!!!
znalazłem błąd

źle zdefiniowałem KLAWISZ_2. ufff:))
znów jest satysfakcja zwłaszcza,że jestem bardzo zielony z programowania


Bardzo Ci dziękuję za zainteresowanie moim pytaniem

_________________
Nikt nie mówił,że będzie łatwo...

hejka,

@brat_kamyka
Normalnie wróżka jesteś.
(bez fotki i kompletnego kodu)-...
Jakim cudem domyśliłeś się że problemem jest definicja przycisku. ??

@Wojtek,
Widzę że zaczynasz ogarniać pisanie własnych funkcji. - super.
Przy najbliższej okazji proponuję zainteresować się timerami , bo widzę że używasz jeszcze delaye.
Wbrew pozorom timery to nie straszydło - tylko tak wygląda na pierwszy rzut oka. a są bardzo przydatne.
Jak wejdą w krew,
to nawet mruganie jedną diodą chce się pisać z wykorzystaniem timera.

Pozdrawiam Darek.

oo cześć Darku. na timerki przyjdzie czas

_________________
Nikt nie mówił,że będzie łatwo...

bo błędów logicznych w funkcjach nie znalazłem ...
to jak nie kod ???? to co ???? HARDWARE !!!

Teraz pytanie do autora wątku po co tworzy się funkcje???
Odp. aby poprawić czytelność kodu,
funkcja to cześć kodu realizująca konkretne zadanie ( np. lcd_init(); itd....),
ja przeważnie jak pisze to bardzo podobnie do Mirka,
wpierw wszystko {co dotyczy jednego problemu} w funkcji main testuję czy działa zgodnie z założeniami,
potem przenoszę kod do osobnej funkcji ale uwaga nazwa funkcji musi mi mówić co ona robi (np: send_spi(string mojstring);)
i wszystko jasne nie.
Innymi słowy :
Programowanie to jak budowanie z klocków LEGO wpierw małe dzieci uczą składać klocki DUPLO (każdy klocek to prosta funkcja na przykład: void zapal_diode1(void);)
potem przychodzi czas na zwykłe klocki LEGO (np viod send_spi(string mojstring); która używa klocka DUPLO viod sendchar_spi(char A);)
Następnie dostajemy zestaw Lego Technic (np funkcja która ma za zadanie wysłanie napisów do wyświtlacza graficznego podpiętego po SPI ....)
a LEGO mindstorm to już wszystko (np prosty system czasu pseudo rzeczywistego -- timery programowe Mirka czy inne sposoby uruchamiania klocków lego Technic).



Pozdrawiam brat_kamyka.

dziękuję za cenne rady

_________________
Nikt nie mówił,że będzie łatwo...

witaj ponownie,

do klawiszy używasz preprocesora dyrektywa:)

a diodami bawisz się w kodzie bezpośrednio --- ciężko się to czyta,
spróbuj użyć tej samej techniki programowania ( preprocesora )




da to takie zalety w jednym miejscu definiujesz "hardware" w sposób bardzo przejrzysty (zwiększasz przenośność kodu na inne połączenia (poprawiasz tylko deklaracje #define )
po za tym łatwiej będzie ten kod analizować
później w kodzie używasz


Proszę poczytaj w BB rozdział 4.4 strona 187 do 192.
tam masz wszystko jak na tacy podane ..... ( rozbuduj o inne LEDY do robala)




Między wierszami czytam że diody na PORTB masz zapięte tylko pytanie co jest na PORTC pinie PC0

na C był buzer, ale go zlikwidowałem, to jest kod gdzie ćwiczę język C, więc jest trochę bałaganu. Dzisiaj bawiłem się z wyświetlaczem LCD i prawie pół dnia mi to zajęło. muszę to dokładnie rozgryźć to strasznie wciąga tak, tak że prawie cała niedziela przeleciała hehe. teraz chcę sobie coś wyświetlać na lcd, ale muszę jeszcze o tym poczytać. Ten kod , który jest wyżej sam sobie pisałem, więc jest w miarę czytelny, najgorsze jest to,że strasznie się męczyłem z drugim klawiszem i szukałem dlaczego nie działa, kombinowałem z kodem,a okazało się ,że miałem źle zdefiniowany pin. i na tym człowiek najwięcej traci czasu. Ale są tego plusy, ponieważ po drodze można się nauczyć czegoś innego. I tak powoli do przodu

_________________
Nikt nie mówił,że będzie łatwo...