Hej, nadszedł czas abym poddał się ocenie Kolegów C-owców.

Do rzeczy: projekt mikroprocesorowego klasyfikatora kolorów opartego na ATmedze32A i czujniku TCS3200.

main.c


colorClass.h


colorClass.c:


Nieco tabulacja komentarzy się rozwaliła - w Eclipse jest większość optycznie wyrównana, tutaj nierówno, chyba system forum po prostu inaczej wyświetla.

Jako, że jestem początkujący proszę o ocenę kodu, sugestie co można by było ulepszyć, zmienić.
Funkcja TCSMeasure jest mocno wzorowana na podobnym projekcie z sieci. Dostosowałem ją nieco do swoich potrzeb.
Wiem, że dostanie mi się za to i za użycie floatów. Można było się bez nich obyć, no ale używam ich w wielu miejscach a takie przeróbki tego na typ całkowitoliczbowy to chyba mają sens w jednym, dwóch miejscach w kodzie, ale nie co chwila. Łatwiej porównywać przecież 34.1 z 35.2 niż mnożyć to razy 10 czy 100 za każdym razem... Ale może się mylę...
Pewnie tego delaya w pętli głównej też warto zastąpić timerem programowym (?).

Będę wdzięczny za wszelkie uwagi i sugestie co do tego kodu.

Więcej o moim projekcie jakby ktoś był ciekawy:
https://dl.dropbox.com/u/6082561/_9%20s ... KOLORU.pdf
https://dl.dropbox.com/u/6082561/_9%20s ... tacja2.pdf
https://dl.dropbox.com/u/6082561/_9%20s ... 0kons1.pdf

_________________
ATB 1.03, Win XP SP3, ECLIPSE Indigo 3.7.2

Te wszystkie stałe (const) możesz umieścić tylko w pamięci programu i zwolnisz z 300 bajtów pamięci RAM, a to jakieś 15% dla ATMEGA32 co może pozwolić na wykorzystanie ATMEGA16. Wiąże się to oczywiście z dużą ilością modyfikacji, ale sam pytałeś. Co do stringów i wskaźników do stringów w pamięci programu poczytaj tutaj http://www.nongnu.org/avr-libc/user-man ... _rom_array

_________________
Dragonus Cracovus: Biomagia

W tej chwili mam takie staty:

Program: 7140 bytes (21.8% Full)
(.text + .data + .bootloader)

Data: 652 bytes (31.8% Full)
(.data + .bss + .noinit)

Procka nie będę zmieniał, bo to działa w dwóch płytkach:
1) ATB
2) moje własne PCB które ma już procka ATmega32A wpiętego w podstawkę.

Ale dzięki za radę, nie wpadłem na to.

_________________
ATB 1.03, Win XP SP3, ECLIPSE Indigo 3.7.2

Dodam jeszcze, że:

spowoduje błędne działanie programu przy wywołaniu typowej funkcji, bo to podsumowanie nie uwzględnia przecież stosu.

_________________
Dragonus Cracovus: Biomagia

Tak, wiem. Dlatego przy 80% już się powinna zapalić lampka. Ale u mnie do 80% daleko...

_________________
ATB 1.03, Win XP SP3, ECLIPSE Indigo 3.7.2

Ledes - przede wszystkim - to aż miło się czyta taki kod. O tej ważnej rzeczy o której napisał Krauser - nawet ja w swoich kodach często zapominam - a przypominam sobie zwykle dopiero wtedy gdy zaczyna brakować RAM'u i trzeba szybko go zaoszczędzić - zoptymalizować kod. Jak zwykle - kolega Krauser - Sokole OKO

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

Dziękuję Mirek za dobre słowo. Cieszę się, że mój kod jest czytelny. Chociaż jak tak teraz dokładniej patrzę, to jest jeszcze kilka baboli jeśli chodzi o notację czy formatowanie kodu. ;P

_________________
ATB 1.03, Win XP SP3, ECLIPSE Indigo 3.7.2

ajtam ajtam ... weź przestań - po tych ostatnich kodach w jednym pliku liczących sobie och-set linii kodu - to to co zaprezentowałeś to miód na moje serce ale - masz rację, że warto po swojemu dopieszczać zawsze kodzik.

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

Witaj.
Czy tych stałych z pliku ColorClass.h nie da się zmienić z float na jakiś typ całkowity?

_________________
sig off ;(