Cześć wszystkim. Mam mały problem z pewnym programem w C (niestety, problem muszę rozwiązać w tym tygodniu) na Atmega16. Program korzysta z ADC do generowania cyfry pseudolosowej (LSB z pomiaru) i wyświetla 4 losowe cyfry na czterech multipleksowanych wyświetlaczach 7 segmentowych. Poniżej znajduje się kod:



Program docelowo pisany do uruchomienia na zestawie ZL10AVR (którego ja niestety nie posiadam, więc nie mogę na bieżąco testować), jednak program nie działa tak jak powinien. Naciskając przycisk praktycznie wszystkie diody na segmentach się świecą, niektóre słabiej, niektóre mocniej (zdarza się czasem że jakaś całkowicie zgaśnie) i wyświetlanych cyfr po prostu nie można rozpoznać. Mógłby ktoś pomóc?

musisz chyba uyważać na kilka rzeczy:
1. z przerwań nie wywołujemy funkcji.
2. jeżeli zmiennych używasz w przerwaniu a potem gdzieś indziej poza tym przrwaniem to
pownienieneś zastosować volatile.
3. a dla lepszej czytelności kodu - funkcje i przerawnia pisz POD funkcją main

To tak na szybko na pierwszy rzut oka choć to pewnie jeszcze nie rozwiązuje Twojego problemu

Oooo... sorki volatile jest

_________________
Jestem początkujący i moje porady mogą być błędne

Niestety, raczej nie rozwiązuje to problemu . W pracowni z układami do testowania programów na bieżąca edytowałem program na różne sposoby, a mimo to nie działało tak jak powinno.

Brakuje nawiasu; }

to może mieć duże znaczenie

W funkcji select_digit w momencie wyboru segmentu są ustawione linie poszczególnych diód, więc przez chwilę na nowym segmencie palą się tak jak w poprzednim. Dopiero następna instrukcja zmienia stan linii diód, i prawdopodobnie stąd "duszek". Najlepiej przed zmianą segmentu wygaś diody, potem zmień segment i zapal ponownie.

A może podziel program na części?
Najpierw uruchom multipleksowanie.
Jak to Ci ruszy (opanujesz) to dołożysz następny kawałek kodu.
Łatwiej będzie Ci zapanować nad tym co nie działa i gdzie szukać błędu.

BTW w BB masz działający kod do multipleksowania - może zacznij od tego co tam jest.

_________________
Jestem początkujący i moje porady mogą być błędne