Also eure Programme laufen genau andersrum, während du alle ports einzelnd anmachst, also high machst, mach er sie Low.
Er hat seine Dioden ander angeschlossen, nämlich gegen Vcc, also leuchten sie, wenn am Portx.x 0 anliegt, das ist sinnvoll, weil die AVR so höhere Strom fließen lasse, also mach es am besten auch so, un schalte sie nicht gegen GND, vielleicht leuchten sie ja shcon, nur eben schwach.
portc = 255 heißt dann eben, alle Pins werden high gesetzt, also die Zahl 255 angelegt, genauso könntest du schreiben portc = &B11111111.

Schalte deine Dioden an Vcc (umdrehen nciht vergessen) und benutze mal das Script von schnuppel, allerdings verstehe ich nciht, warum er dann mitten im Script vom Setzen auf 0 zum setzten auf 1 springt...

MFg Moritz