Jeżyk-1: Pokazujesz tylko fragment kodu ale chyba problemem jest to, że zmienna przy_stan ma ustaloną wartość na pewien czas (w procedurza sprawdz_klawiature), a w kodzie



wartość menu zmienia się cały czas dla tej ustalonej wartości (fragment: menu++/menu--). Wartość menu zaczyna zależeć od długości naciśnięcia klawisza, czyli dosyć losowo
Także raczej błąd logiczny a nie problem z drganiami.

I trochę takie masło maślane zaczyna się z tego robić bo to są zlepki różnych kodów. Coraz ciężej się to analizuje. Ja bym obsługę klawiatury w całości wrzucił na jakieś przerwanie (wywoływane z 30-60 Hz) łącznie z eliminacją drgań (być może jeszcze z wyborem poziomu menu). Na drugim przerwaniu obsługa zegarka (ten kwarc 32k). W programie głównym obsługa menu i wyświetlacza.

No robi się masełko ;(
Nie umiem lepiej
A czy to nie będzie za wiele tych przerywań ?
Już by były 2, następnie obsługa grzałki (regulacja mocy grzania) jedno przerwanie, dodatkowo regulacja obrotów wentylatora i pomiar ich obrotów kolejne przerwanie?

Napisz co dokładnie chcesz zrobić i wklej kod, który już napisałeś. Bez tego nikt Ci nie powie czy się zmieścisz

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Teraz to popsułem coś ponieważ zabrałem się za przerwanie na ADC.


To ma być nie skomplikowany układ do regulacji temperatury w malutkim pomieszczeniu dokładniej na makiecie, na której termometr DS18b20, grzałka 230V 200W która będzie płynnie regulowana moc grzania, oraz wentylator wiewający zimne powietrze który będzie miał regulowane obroty wiania i pomiar obrotów wentylatora, wentylator będzie przymocowany do ogniwa peltiera, blok wodny i chłodzenie wodne wodne czyli pompka obiegowa, wentylatory chłodnicy i elektrozawory procesor będzie załączał przekaźnik który będzie uruchamiał poszczególne elementy. trochę sporo tego i niestety mam do 15,04 czas na wykonanie samego programu
Już się obawiam że nie zrobię tego na czas i nie wiem czy nie będzie lepiej zlecić komuś napisanie takiego programu.

Po co tyle pisać? Nie lepiej tak:



A tutaj nie masz zbytniego "rozstrzału" między napięciami, więc możesz zapisać tak:


Dlaczego w 68 linii masz jedyne wywołanie tej funkcji, a nigdzie nie zapisujesz tego co zwraca?

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

o coś takiego chodzi ?


Tylko za bardzo nie działa obsługa klawiszy
Zachowuje się jak byłby wciśnięty cały czas przycisk

A który? I jaką masz wartość napięcia jak żadnego nie wciskasz (wyrzuć sobie na ekran)?

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Gdy wczytuje wartość z przerwania :


To mam cały czas 1023
Ale jak wciskam jakiś przycisk to nic się nie dzieje.

A, czyli program zachowuje się prawidłowo tylko my mamy złe warunki
Powinno więc być coś takiego:



Wcześniej cały czas zgadzał się warunek dla ESC, bo 1023>885

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Czemu nic nie piszesz o ostrzeżeniach kompilatora? Mi po próbie kompilacji kodu wrzuconego o 20:22 wyrzuciło ostrzeżenia o przypisywaniu wartości w IFie.

I tak, np linia 97 powinna najprawdopodobniej brzmieć "if (menu==0)", chyba, że wiesz co robisz poprzednim zapisem.

Dodatkowo linia 54 według mnie uniemożliwia wykonanie przerwania ADC_vect

Czemu w ten sposób rozpisałeś mój kod w liniach 97-123 ?

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Nie zauważyłem tego. Przepraszam, miało być ==



Nie przypatrzyłem się i trochę źle zinterpretowałem co miałeś na myśli. już to poprawiam



Też mi się wydaje że ADRF powinno być włączone ale jak go włączę to wywala mi :
'ADRF' undeclared (first use in this function)
Zaglądałem do noty katalogowej to nie ma takiego bitu w Atmega16 i to chyba dlatego taki błąd wyskakuje.

Nie wgłębiałem się w temat ale niech ktoś powie tylko, czy jest tutaj omawiany regulator PID (lub podobny)?

_________________
ATB 1.03, Win XP SP3, ECLIPSE Indigo 3.7.2

W tej chwili nie
Jeszcze tak daleko nie dotarłem

http://www.atmel.com/Images/doc2466.pdf str 219 i według mnie ten bit nazywa się ADATE.

Ps. ADFR nie jest znajdowane w całym PDFie

Ledes: Nie, chyba nie

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

To już też tam próbowałem go wstawić i nić nie dało:(
Dokładnie cały kod teraz mam taki:


I na wyświetlaczu pod zmienną "pom" wyświetla mi się wartość = 3
A gdy duszę przyciski to zmieniają się wartości
1 przycisk = 0
2 przycisk = 1
3 przycisk = 2
4 przycisk = 4
5 przycisk = 0
6 przycisk = 0

------------------------ [ Dodano po: kilkunastu sekundach ]

A gdy włączyłem
ADMUX |= (1<<ADLAR); //wynik wyrównaj do lewej

to:
1 przycisk = 620
2 przycisk = 110
3 przycisk = 150
4 przycisk = 182
5 przycisk = 209
6 przycisk = 230
nic = 250

coś nie tak chyba ten pierwszy wynik ?

W 280 linii wstaw ADCW zamiast ADCH

------------------------ [ Dodano po: 4 minutach ]

W linii 9: volatile uint16_t ADC_pomiar; //aktualny pomiar ADC

------------------------ [ Dodano po: 7 minutach ]

Linia 84: lcd_int(pom); lcd_str(" ");

Wyłączenia IFa w 103 linii jest celowe ?
Dlaczego masz zdublowaną obsługę wyników ADCka ? Chodzi o linie 73-81 vs. 222.
W linii 280 odczytujesz 8 najstarszych bitów ADCka. W takim razie musisz przeliczyć wartości dla przycisków (teraz masz je dla wszystkich 10 bitów) albo odczytywać ADCW i wyłączyć przesuwanie w lewo.

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Ta zamiana spowodowała ze cyferki sie cały czas zmieniają coś pomiędzy 192 , 122, i 600 coś nie idzie odczytać ;/

Tak na szybko to chyba wszystko, bo takiej klawiaturki 6-klawiszy z rezystorkami to nie mam.
Co jeszcze wg Ciebie nie działa ?





ADC podłączyłem działa poprawnie, u mnie w zakresie 2...1023

W liniach 83-84 wyświetlasz sobie wynik pomiaru, tak ? Może te dziwnie wysokie wyniki są związane z tym, że nie usuwasz starych cyfr z wyświetlacza...
Dodaj tam sobie jeszcze
lcd_str(" ");

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

W sumie teraz działa jak dodałem tą linie :
Linia 84: lcd_int(pom); lcd_str(" ");
musiał zostawać poprzedni wynik i dlatego wychodziło 622 teraz wyskakuje 62.
Ale zastanawia mnie dlaczego wcześniej tamtym kodem miałem wartość ok 1023 przy nie wciśniętym przycisku a teraz 251 ?

IF jest w tej chwili włączony wcześniej go wyłączyłem testowo.
Z tym zdublowaniem to już juto pomyślę dlaczego mi tak to wyszło. Z tego co miało być to jedno zamieniało wartość pomiarową na nr przycisku który wciśnięty a 2 zamieniać miało na cyfrę na nazwę.
Jutro to porawię i zobaczę czy nie będzie mi menu przeskakiwało jak wcześniej

Bo odczytujesz różne rejestry. Na 220 str datasheeta masz to ładnie opisane. Jak chcesz mieć pełną rozdzielczość (10 bit = 1023) to wyłącz bit ADLAR i odczytuj rejestr ADCW (taki rejestr "zbiorczy" - zawiera od razu ADCH + ADCL

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Ok Jest dokładnie tak jak piszecie.

Czas zabrać się za zniwelowanie dublującej funkcji.
Ja bym zostawił ten kod ponieważ jest krótki i zrozumiały dla mnie :


Funkcja zwraca wartości:
1 - pierwszy przyciska
2 - drugi przycisk
.
.
.
.
6 - szósty przycisk.
I kombinowałem coś z trybem wyliczeniowym ale chyba źle to zrozumiałem albo źle to robię.
Czy jeżeli zmienna przy_stan zawiera w sobie danę który przycisk jest wciśnięty to funkcją ENUM zamienię ją na postać znakową (RIGHT, LEFT...) ?

Wygląda ok, ale zamiast unsigned char pisz uint8_t - od razu wtedy wiadomo o co chodzi. Po co warunek w 10 linii?

W 63 linii jeżeli chcesz zwracać numer przycisku to wpisz po prostu
return stan_poprzedni
chyba, że nie chcesz zawsze zwracać zawsze jakiegoś przycisku. Wtedy musisz sobie tak przerobić tą procedurę, aby dla braku zmiany zwracała jakiś przewidziany przez Ciebie, nic nie znaczącą wartość.

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Udało się zrobić obsługę tych klawiszy lecz niestety w pętli głównej
zabrałem się za menu i 1 pozycję menu nastawy zegarka i całkiem mi to nie idzie.
Pewnie jak ktoś ten kod zobaczy to się za głowę złapie co ja tworzę



Najbardziej mnie zastanawia dlaczego jak włączę menu ustaw zegar i przyciskiem ok zatrzymuje odliczanie, i zmieniam pozycje nastawy (godziny, ,minuty, sekundy) ale jak wcisnę góra to wszystkie wartości się zwiększają
potem jak zrobię esc to już nie mogę zmieniać pozycji po głównym menu

Nie wiem jak dla innych, ale dla mnie ten kod robi się niedebugowalny
Sorry, może ja po prostu mam specyficzny sposób pisania, ale ilość tekstu trochę mnie przytłacza.

1. Nie stosuj takich długich przerw - 1 pusta linia jest w sam raz.
2. Poopisuj przy deklaracji zmiennych od czego są.
3. Trzeba odchudzić według mnie funkcję main - za dużo w niej wszystkiego. Według mnie byłoby fajnie, gdybyś:

1. Linie 34-57 wyrzucił sobie do funkcji typu inline.
2. Linie 76-130 też wyrzucił do funkcji. Dodatkowo robisz wiele razy to samo dla różnych klawiszy - zastanów się czy nie lepiej będzie zrobić macierz i ją obsługiwać.
3. Linie 157-297 - switch pod switchem to według mnie strasznie "groźna" konstrukcja - zastanów się czy nie przejrzyściej byłoby co najmniej powyrzucać do funkcji kod dla switcha nadrzędnego (tego "switch(menu)").
4. Trochę za "obfite" komentarze według mnie

jest dla mnie trochę za dobrze skomentowane, wolę coś takiego

ale tu już nie nalegam - to zależy od tego co kto lubi i jak pisze.


Tutaj niepotrzebnie się rozpisujesz, można to zapisać tak:

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Ok zdaje sobie sprawę że jest bałagan więc zabieram się za porządki.

Zaczynając od funkcji inline zawsze mam problem z tym co ma funkcja zwracać a co pobierać.
jak zrobiłem tu i tu void to nie działa:

Jest ok. Funkcja nie musi przecież nic pobierać, ani zwracać
Jak chcesz jej użyć to piszesz
ustawienia();
ten pusty nawias jest potrzebny - jest to informacja, że nic do niej nie przekazujesz.

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.

Tego to chyba nie będę wiedział jak zrobić

Ale że samego wyrzucenia do funkcji czy przeróbki na macierz ?

_________________
Nie pisz komentarzy - dobry kod komentuje się sam.