Witam wszystkich,
Programowaniem zajmuje się od niedługiego czasu(będzie jakiś miesiąc), chciałbym się podzielić ze swoim małym programem napisanym w celach edukacyjnych. Ma on za zadanie odczyt czasu z układu PCF8583 i wyświetlić go na wyświetlaczu LCD oraz przez przez RS232. Pisząc program opierałem się na bibliotekach Pana Mirka "lcd44780" i "IC2".
Chciałem spytać Was co można ewentualnie poprawić, uprościć i ogólnie o "kulturę" mojej pracy.



Mam nadzieję, że kod jest w miarę zrozumiały
Teraz planuje wprowadzić funkcję wprowadzania aktualnego czasu poprzez przyciski.

Aby sobie na przyszłość ułatwiać życie - zdecydowanie powinieneś od razu uczyć się dzielić projekt na pliki, tu masz akurat fajną możliwość bo logiczną grupą funkcji do wydzielenia są te do obsługi UART'a - to jedno

a drugie to ZDECYDOWANIE nie używaj takiego zapisu:

UBRRL = 51;

tylko ustawiaj to za pomocą makra - np zgodnie z tym co na blogu

http://mirekk36.blogspot.com/2013/01/rs ... trick.html

_________________
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 ]

Postaram się podzielić projekt na pliki.
A jeśli chodzi o zapis UBRRL to za pierwszym podejściem chciałem wykorzystać sposoby z linku powyżej, ale nie udało mi się i postanowiłem sobie ułatwić trochę pracę. Teraz na pewno przeanalizuję ten artykuł jeszcze raz

Tak można robić:

Zamiast dwóch znaków końca w postaci 0x0d, 0x0a można prościej napisać nadaj_str("\r\n"); i możesz dodać sobie analogiczną funkcję wysyłającą dane z pamięci programu (flash). Funkcję nadaj_str można znacznie uprościć wiedząc, że łańcuchy znaków są zakończone zerem. Podglądnij sobie kod lcd_str.

_________________
Dragonus Cracovus: Biomagia

Poprawiłem kod zgodnie z zaleceniami i wygląda on teraz znacznie lepiej.
Dzięki chłopaki.