Witam.Próbuje napisać program (zamek szyfrowy) na diodach plus dwa micro switch i ostatnio znalazłem kod w internecie który nie do końca mi się podoba.Czy można tak programować i czy to jest dobry styl programowania.Z góry dziękuje za odpowiedź i pozdrawiam.

Styl trochę zły(nieczytelny) w książce pana mirka jest o tym.

------------------------ [ Dodano po: 1 minucie ]


Tuż to niepotrzebne pisanie w binarce + do tego bardzo głupie lepiej zrobić tak
czyż kod nie jest bardziej czytelny i ładniejszy ?

------------------------ [ Dodano po: 6 minutach ]

zamiast pisać

lepiej było by napisać

i tak w drugim przypadku.
Jak zmieniasz dla jednego portu i/o na inny a drugiego używasz jako wejście to użyj stylu

lub do ustawiania stanu na portach

Witam.A co z tymi while-ami to tak można i bez nawiasów i średników { } , ;


------------------------ [ Dodano po: 9 minutach ]

A nie suma bitowa (OR)

Jeżeli po while() jest tylko jedna instrukcja, to można. Znaczy, średnik musi być. To, co masz napisane w jednej linii można zapisać tak:

Jeszcze jedno jak mam tak to buzer się nie wyłącza tylko cały czas brzęczy . Proszę o podpowiedz.

A gdzie masz funkcję main(), w której wywołujesz funkcję blokada()?
W ogóle, skompilowało się to?

Kolego, a jak ma to działać? Popatrz, w linijce 77 masz(celowo pominąłem resztę).
Oznacza to: dopóki "1" jest prawdą (a jest zawsze, bo fałsz to wartość 0) wywołuj funkcję blokada(). Czyli zadziała funkcja (10 zmian stanu na pinie buzera, następnie wyłączenie buzera), a następnie ta funkcja będzie wywołana jeszcze raz... i jeszcze raz... i jeszcze miliard razy.
Jeśli chcesz wywołać tą funkcję raz, a później "zablokować" program, to zrób to tak (linijki od 75):W tym przypadku funkcja blokada() zostanie wywołana raz, a później będzie się wykonywała pusta, zaznaczona linijka

EDIT:
Teraz zauważyłem:
UWAGATo, co jest napisane do końca postu to głupoty... nie stosować, nie powielać!!
Nie, tam ma być suma logiczna ||
Zauważ, że 1<<PB3 ustawia tylko jeden bit w rejestrze. Stosując || ustawiasz ten jeden, konkrety bit pozostawiając pozostałe bez zmian

to z kodu Mirka.Przerwania AVR i jest |

No tak, racja... pośpieszyłem się. Oczywiście, że masz rację. Przepraszam za zamieszanie. Poprzedni post zaraz poprawię

ok.