pinMode(1,OUTPUT);
Vorsicht Pin 1 ist in der Regel von der Seriellen belegt.
Und ich stimme meinen Vorrednern in Grenzen zu.
Richtig ist:
Stecke die LEDs (ihre Pinnummern) in ein Array!
Fange bei 0 an zu zählen, wie es sich für anständige C++ Arrays gehört.
Keine #define verwenden.
Verwende const byte für die Definition.
Es gibt sinnvolle Einsätze für #define. Aber nicht hier
Das delay() kann man entfernen, wenn man unsere "Nachtwächter Erklärung" verstanden hat.