Also das mit den Tastern müsste stimmen jedoch würde ich ein else vor den if-Abfragen schreiben (außer der ersten).

Bei den LEDs musst du Transistoren verwenden, da ein Ausgang keine 3 Leds verträgt, vermute ich mal.
Alle LEDs aus wäre dann: r1-r3 = 0 und s1-s3 = 1 .
Was aber egal wäre, wenn du jedes mal beim setzen einer LED alles veränderst.
Der Rest stimmt ebenfalls.