po pierwsze czyż kolega w ogóle NIC a NIC nie czytał na temat pętli ???? przecież to ZGROZA pisać takie coś:
[syntax=c]tablica[0] = 0xff;
tablica[1] = 0xff;
tablica[2] = 0xff;
tablica[3] = 0xff;
tablica[4] = 0x0f;
tablica[5] = 0xff;
tablica[6] = 0xff;
tablica[7] = 0x0f;
tablica[8] = 0xff;
tablica[9] = 0xff;
tablica[10] = 0xff;
tablica[11] = 0xff;
tablica[12] = 0xff;
tablica[13] = 0x0f;
tablica[14] = 0xff;
tablica[15] = 0xff;
tablica[16] = 0xf0;
tablica[17] = 0xff;
tablica[18] = 0xff;
tablica[19] = 0xff;
tablica[20] = 0xff;
tablica[21] = 0xff;
tablica[22] = 0xff;
tablica[23] = 0xff;[/syntax]
Nawet powiedziałbym ZGROZO - MASAKRA , proszę jak najszybciej pozmieniać wszystkie te części kodu które można wrzucić w pętle - bo tego się czytać nawet nie da - toż to można zapisać w 1 linii kodu!!!, pierwszy i ostatni raz udzielam takiej podpowiedzi:
[syntax=c]uint8_t i;
for( i=0 ; i<24 ; i++ ) tablica[i] = 0xFF;[/syntax]
Poza tym TOTALNE szastanie - bo tak trzeba powiedzieć - typami danych. Wynika to albo z niefrasobliwości, która w programowaniu procków doprowadzi kiedyś do szaleństwa
albo co gorsze z braku podstawowej wiedzy na temat typów danych w C ..... oj trzeba to doczytać, i nigdy na drugi raz nie definiować np zmiennej :int licznik; (czyli 16-bitowej) dla zmiennej, która w trakcie pracy przyjmie wartości TYLKO od 0 do 24 - czyli zmieści się w jednym bajcie a zatem spokojnie można użyć typu jedno-bajtowego uint8_t !
proszę się jak najszybciej przestawić na typy AVR GCC czyli:
Code:
uint8_t
uint16_t
uint32_t
uint16_t
uint32_t
a nie tam:
Code:
unsigned short int
unsigned int
unsigned int
itd - bo będą kłopoty później oj będą ..... a zresztą czy kolega lubi tak się katować żeby aż tyle klepać w tą klawiaturę ?
Kolejna rzecz - co to kurczę JEST ?
za fantasmagoria ???????Code:
UBRR1H = (unsigned char)(ubrr >> 5);
No - nie nawet nie podam jak ma być - bo nie dość, że kolega ma podane to jak na patelni i w mojej książce to także WYRAŹNIE w nocie PDF ...... no ale widać, że coś nieteges jeszcze z przesunięciami bitowymi - a jeśli tak? to lepiej dopytać jak się czegoś nie rozumie zamiast zgadywać albo pisać tak długi program - bo błąd błędem pogania
Te zmienne:
Code:
volatile int poziom = 1;
volatile int licznik = 0;
volatile int licznik = 0;
poza tym że mogą być uint8_t a nie int, to są to zmienne globalne .... i o ile poziom można zainicjalizować bo ma mieć wartość inną niż 0 to po co inicjalizacja zerem - zmiennej licznik ?
Skoro zmienna bajty to stała - to po jakiego grzyba ją definiować jako zmienną i jeszcze porównanie robić ze zmienną w przerwaniu ?
Code:
if(licznik == bajty) licznik = 0;
a wszystko jeszcze na deser dzieje się tu na liczbach 16-bitowych, które i tak nigdy nie przekroczą zakresu 8-bitów - a ile na to pamięci RAM jest marnowanej oraz ile czasu jest tracone to aż strach się bać pomyśleć.
----------------
Kolejna rzecz - już komuś w innym wątku pisałem - proszę w kodach zamieszczanych na tym forum używać przesunięć bitowych zamiast tych durnych makr _BV() ..... bo od razu widzę, że jest problem ze zrozumieniem przesunięć - i te makra w niczym nie pomogą tylko będą przeszkadzać...... Zastrzegam sobie na przyszłość, że będę pomagał tylko gdy ktoś będzie pisał tu kody zgodnie z dobrą sztuką programowania OK ?
Następny kwiatek:
jest sobie funkcja:
Code:
int poziomo() {
}
}
Ok już nie będę płakał że nie ma void w nawiasach gdzie brak argumentów - ale na ZEUSA jest określony typ zwracanego rezultatu na int - a gdzie jest zwracany ten rezultat za pomocą return ????? Albo on jest potrzebny albo nie ..... albo piszemy program przejrzyście albo nie .....
ale tu dzieje się widzę gorsza rzecz - kolega TOTALNIE nie zwraca uwagi na WARNINGI ..... !!!!!!
czy kolega programuje już w ECLIPSE ????? sorki jeśli się powtarzam z tym pytaniem - ale w ECLIPSE nie dałoby rady nie zauważyć TYCH WARNIGÓW/BŁĘDZIORÓW bo tak trzeba je nazwać w tym mini kodzie ..... no sorki - ale tak pisać programów to nie można ..... albo tak pisać z setkami warningów i potem zwracać się z pytaniem na forum jakby nigdy nic się nie stało i nikt nie widział nigdzie żadnego warninga
---------------------
no i teraz do meritum...... to co się dzieje w pętli głównej to totalny MISZ-MASZ ..... coś tam się zeruje za pomocą tej funkcji poziomo() a potem w kółko miliony IF'ów sprawdzają trzysta tysięcy razy na sekundę tablicę, która napełniana ma być w przerwaniu zupełnie asynchronicznie - totalnie w oderwaniu od tego co dzieje się w przerwaniu ...... Jak to ma prawo działać - toż to CHAOS i nic więcej
----------------------
ale kolejna masakra to:
Code:
_delay_ms(czas);
NIGDY w życiu nie wolno przekazywać do _delay_ms() argumentów w postaci zmiennej. Wprawdzie masz procek ATmega128 to nawet nie zwracasz uwagi na to że nagle po takiej operacji i po kompilacji przybywa ci parę niepotrzebnych kilobajtów we Flashu. Gdybyś robił na ATtiny2313 to już by wykorzystanie pamięci po tej linijce wyniosło np 120% .....
-----------------------
OK - sporo opisanych tu przeze mnie problemów nie ma bezpośredniego wpływu na złe działanie tego programu - ale nieziemsko utrudnia analizę i pracę - nie tylko mi czy komukolwiek na forum (tak nawet nie wypada przedstawiać kodu) ..... ale co najgorsze tobie samemu sprawia poważny problem w analizie i stąd nawarstwiają się błędy i brak czytelności kodu
proponuję to wszystko popoprawiać i wyprowadzić ładnie a wtedy będziemy się mogli skupić na rzeczywistym problemie OK ?
Statystyki: Napisane przez mirekk36 — 6 kwi 2012, o 23:01
]]>
