switch geht nicht

Ich fürchte, der Code wird funktionieren, nur deine Eingänge
lesen nicht das zurück, was du erwarten würdest.

Reduzier das Programm doch erstmal auf einen einfachen Mirror, der die
Eingänge von PINB auf die Ausgänge von PORTC spiegelt.  Da du ja LEDs
dran hast an C, solltest du auf diese Weise gut erkennen können, was
denn genau an PINB reinkommt.

1

#include <avr/io.h>

2

3

int main(void)

4

{

5

6

  PORTB = 0xFF;    // pull ups for key inputs  PortD2 and portD3

7

8

  DDRC = 0xFF;     // outputs for LEDs

9

  PORTC = 0xFF;    // LEDs off

10

11

  for(;;)

12

  {

13

    PORTC = PINB;

14

  }

15

}

p.s.: Meine Glaskugel sagt übrigens, dass deine Eingänge einfach nur
low-aktiv sind, d. h. du musst entweder mit ~PINB arbeiten oder
statt 0x01, 0x02 usw. auszuwerten 0xFE, 0xFD usw.

siehe:
Beitrag "switch-case Anweisung wird nur im WINAVR-Simulator ausgeführt Nicht auf Hardware ATMega8 (STK500)"
gruß
penie

Ralf schrieb:
> Im DEFAULT fehlt das break. Vielleicht bringt ihn das durcheinander.

Der letzte Label in einem switch benötigt kein break.  Wenn der
,,nach unten durchfällt'', ist die switch-Anweisung ja so oder
so beendet.

Es ist dabei übrigens völlig egal, ob der letzte Label nun "default"
heißt, oder ob es ein benannter Label ist (und default vielleicht
mittendrin geschrieben wird): entscheidend ist dabei nicht der Name,
sondern dass es der letzte innerhalb des switch ist.

ObJoke dazu: http://de.wikipedia.org/wiki/Duff's_Device
Eine sehr kreative Variante, sich die Eigenheiten der Programmier-
sprache C auszunutzen. ;-)  Selbst gestandene C-Hasen werden dabei
nicht auf Anhieb sagen können, was dort eigentlich wirklich passiert.

Jörg Wunsch schrieb:

> ObJoke dazu: http://de.wikipedia.org/wiki/Duff's_Device
> Eine sehr kreative Variante, sich die Eigenheiten der Programmier-
> sprache C auszunutzen. ;-)  Selbst gestandene C-Hasen werden dabei
> nicht auf Anhieb sagen können, was dort eigentlich wirklich passiert.

Weil selbst gestandene C-Hasen gerne verdrängen, was die 
switch-Anweisung eigentlich ist, nämlich ein bedingtes goto. Der Code im 
switch sind eben nicht mehrere Blöcke mit der optionalen Möglichkeit in 
den nächsten Block "durchfallen" zu können. Es ist nur ein einziger 
großer Codeblock mit mehreren Einsprungpunkten und der optionalen 
Möglichkeit, ihn vorzeitig verlassen zu können (per break). Wenn einem 
das klar ist, ist auch sonnenklar, was dort passiert, denn mit folgendem 
hätten diese C-Hasen ja vermutlich kein Verständnisproblem:

1

goto jump_into_loop;

2

3

do {

4

   ...

5

   jump_into_loop:

6

   ...

7

} while (...);

Und das ist es, was dort passiert.

Ja, ich weiß, dir brauch ich das nicht zu erzählen. ;-)
Aber vielleicht hilft es ja jemanden, der hierüber stolpert.

Ja, die Erklärung im Wikipedia fand ich so schlecht auch nicht.

Nur, wenn man das Teil das erste Mal sieht, knallt's einem doch ganz
mächtig an den Kopf. :)