1)
Definice konstant na začátku - použijte buď C styl
Kód: Vybrat vše
#define led 11nebo pro číselné konstanty také ve stylu C++
Kód: Vybrat vše
const int led = 11;Takhle se zbytečně alokuje inicializovaná RAM.
2)
Blok uvozený komentářem //PWM má zbytečné složené závorky. Pokud se proměnná value nepoužívá jinde, pak blok má smysl, ovšem s nadefinováním lokální proměnné value. Platí zásada, že proměnné se mají definovat s minimálním oborem viditelnosti. To samé platí pro proměnnou x. Takže:
Kód: Vybrat vše
// ------------- PWM -------------
{
int value, x;
value = analogRead(analogpin); //precist potenciometr
x = map(value, 0, 1023, 0, 255); //prevod hodnot z potenciometru na PWM rozsah
analogWrite(Ledpin, x);
}
delay(20);
nebo lépe:
Kód: Vybrat vše
// ------------- PWM -------------
{
int x = map(analogRead(analogpin),
0, 1023, 0, 255); //prevod hodnot z potenciometru na PWM rozsah
analogWrite(Ledpin, x);
}
delay(20);
3)
Za delay(20) opět zbytečný blok.
4)
Řádku:
Kód: Vybrat vše
if (svetlo) svetlo = false; else svetlo = true; je lépe napsat takto:
Kód: Vybrat vše
svetlo = !svetlo;