Switch-Variable innerhalb switch ändern

[Walter T. (nicolas)]

Hallo zusammen,
ist es erlaubt, innerhalb eines switch-Statements die switch-Variable zu 
ändern? Konkret habe ich auf einem ATmega48 ein paar Zeilen, die jeweils 
nur einmal durchlaufen werden sollen (label tohigh und tolow):

void setup(void) {
  enum {tohigh,high,tolow,low,last} setup_mode = tohigh;  

  uint8_t shi = eeprom_read_byte(&eeServoHigh);     
  uint8_t slo = eeprom_read_byte(&eeServoLow);
  int8_t enc_this = encoder_get();
  int8_t enc_last = enc_this;
  uint8_t *enc_var =  &shi;  
  
  uint16_t kt = 0;
  
  while(1) {
    // Tastendruck interpretieren: Ausgang oder weiterschalten
    kt = key_time();
    if(kt) {
      if (kt > SHORT_LONG) {
        break;
      } else {
        setup_mode++;
        if (setup_mode==last) setup_mode = 0;
      }        
    } // if(kt)    
    
     // Drehgeber auswerten (nur Aenderungen)
     enc_this = encoder_get();
     *enc_var += enc_this-enc_last;
     enc_last = enc_this;    
    
    switch (setup_mode) {
      tohigh: // High-Stellung einstellen (Start)
        led_mpx_putany((char[]) {0x00,LED_CHAR_I,LED_CHAR_H});
        enc_this = encoder_get();
        enc_last = enc_this;
        enc_var = &shi;
        setup_mode = high;
        /* FALLTHRU */
      high: // High-Stellung einstellen 
        OCR1A = shi;
        break;
        
      tolow: // Low-Stellung einstellen (Start)
        led_mpx_putany((char[]) {0x00,LED_CHAR_O,LED_CHAR_L});
        enc_this = encoder_get();
        enc_last = enc_this;
        enc_var = &slo;
        setup_mode = low;
        /* FALLTHRU */
      low: // Low-Stellung einstellen
        OCR1A = slo;
    }
  } // while(1)
  
  eeprom_write_byte(&eeServoHigh, shi);
  eeprom_write_byte(&eeServoLow, slo);
}

das beobachtete Verhalten weicht jedoch massiv davon ab, was ich 
erreichen will (um genau zu sein: Er hängt sich beim Start der Routine 
auf).

Die Routine led_mpx_putany((char[]) ) stellt beliebige Bitmuster auf 
einer 3-Ziffern-7-Segment-Anzeige dar, die Funktion encoder_get(void) 
liest einen Drehgeber kontinuierlich aus und der Rest sind 
Standardfunktionen.

Die folgenden Warnungen des build-Prozesses machen mich mißstrauisch:
enumeration value 'tohigh' not handled in switch [-Wswitch]
enumeration value 'high' not handled in switch [-Wswitch]
enumeration value 'tolow' not handled in switch [-Wswitch]
enumeration value 'low' not handled in switch [-Wswitch]
enumeration value 'last' not handled in switch [-Wswitch]
label 'low' defined but not used [-Wunused-label]
label 'tolow' defined but not used [-Wunused-label]
label 'high' defined but not used [-Wunused-label]
label 'tohigh' defined but not used [-Wunused-label]

Viele Grüße
Nicolas

[Läubi .. (laeubi)]

Nicolas S. schrieb:
> ist es erlaubt, innerhalb eines switch-Statements die switch-Variable zu
> ändern

Ja, wieso nicht?

Nicolas S. schrieb:
> Er hängt sich beim Start der Routine auf

Was bedeutet das? Neustart? Endlosschleife?

Nicolas S. schrieb:
> as beobachtete Verhalten weicht jedoch massiv davon ab, was ich
> erreichen will

Ein case vor dem enum solltest du noch spendieren, siehe z.B. 
http://crasseux.com/books/ctutorial/enum.html

Nicolas S. schrieb:
> Die Routine [...] die Funktion [...]

Beim debugging immer auf das Wesentliche reduzieren... eine LED an/aus 
reicht oft völlig aus um das Problem einzugrenzen.

[Walter T. (nicolas)]

Läubi .. schrieb:
> Ein case vor dem enum solltest du noch spendieren

Facepalm JA! Danke!

[Peter D. (peda)]

Nicolas S. schrieb:
> ist es erlaubt, innerhalb eines switch-Statements die switch-Variable zu
> ändern?

Ein Switch ist ein indiziertes Goto, die Case sind die Labels.
Du kannst also die Variable jederzeit ändern, evaluiert wird sie nur in 
der Switch-Zeile.

[Walter T. (nicolas)]

Peter Dannegger schrieb:
> Ein Switch ist ein indiziertes Goto

Wie Du siehst, hatte ich schon zuviel GOTO im Kopf. :)

Danke für die Klarstellung!

P.S.: Jetzt geht auch alles. Habe also bald auch wieder ein Foto für den 
Thread über krumme Tischplatten.