... magari usare due variabili diverse nel ciclo di FOR aiuta ... XD XD XD
Non vedo dove dichiari "i" ... ritengo quindi sia dichiarata globalmente e ... NON puoi non aspettarti strani comportamenti se usi la stessa variabile !!! 
Creati una variabile locale solo per la funzione spegniLed() ...
void spegniLed(){
byte j;
for(j=0; j<4; j++){
digitalWrite(j,LOW);
}
}
Guglielmo
@nid : ma io e te siamo telepatici che postiamo sempre assieme ??? XD XD XD