Ale najpierw kod:
Zakładamy, że tryb=='M'
[syntax=c]
while(1){
if (tryb=='M'){ OCR0=red; OCR1=green; OCR2=blue; } //jesli tryb manualny to ustal piny PWM
if (tryb=='A'){} //jesli tryb auto to cos
if (flaga==1){ //jesi przyszedl string
flaga=0; //skasuj flage
odczyt(); //odczyt z rsa
}
[/syntax]
No i teraz tak,
Moja dioda będzie pracowała w 2 trybach Manualny(czyli my za pomocą suwaga zmieniamy wartość koloru), albo tryb automatyczny, czyli leci sobie tam tęcza..
No i teraz mam taki dylemat w jaki sposób lepiej napisać kod:
Bo jeśli mamy tryb manualny, to raczej często kolory się zmieniać nie będą (ustawimy jakiś dany kolor) i lampka będzie świeciła tak np. przez 30min. to po co mają być cały czas ustawiane końcówki OCR0=red; OCR1=green; OCR2=blue;
Czy nie lepiej zrobić tak, że jeśli będzie tryb manualny, to ustawiamy sobie jakąś flagę (bool Tryb_manualny)
I kod piszemy wtedy tak:[syntax=c]
while(1){
if (Tryb_manualny){
OCR0=red; OCR1=green; OCR2=blue;
Tryb_manualny=0;} //jesli tryb manualny to ustal piny PWM
if (tryb=='A'){} //jesli tryb auto to cos
if (flaga==1){ //jesi przyszedl string
flaga=0; //skasuj flage
odczyt(); //odczyt z rsa
}
[/syntax]
No i przy takiej postaci to uC nie będzie cały czas niepotrzebnie ustalał tych pinów, tylko będzie sprawdzał w kółko warunki...
Która wersja kodu jest lepsza?
Czy to ma jakieś znaczenie?
Chciałbym nauczyć się dobrych praktyk
)Statystyki: Napisane przez xamrex — 13 wrz 2012, o 22:55
]]>