switch Anweisung default Zweig (Kompilerfehler?)

[Matthias R. (matsch)]

Hallo Forum,
ich habe ein Problem mit folgender switch-case Anweisung:

switch (Befehl){
 case 'D' :                      
   uart_puts("ein D\r");
   break;
 case 'R' :                      
   uart_puts("ein R\r");
   break;
 case 'W' :
   uart_puts("ein W\r");
   break;
 case 'P' :                      
   uart_puts("ein P\r");
   break;
 default:
   uart_puts("nop\r");
   break;
}

In 'Befehl' steht ein Zeichen, welches ich über RS232 mit meinem 
ATMega88 empfange. Je nach empfangenem Zeichen wird eine Antwort über 
RS232 an den PC gesendet (mit der UART library von P. Fleury).
Das ganze funktioniert auch sehr gut.

Nun habe ich neu den default-Zweig eingefügt. Wenn jetzt ein ungueltiges 
Zeiche empfangen wird, soll ein 'nop' gesendet werden - funktioniert 
auch.

ABER: Wenn ein gültiges Zeichen empfangen wird (D, R, W oder P) wird 
neben dem zugehörigen Text auch immer noch das 'nop' aus dem default 
Zweig gesendet.
Nach meinem Verständnis soll der default Zweig aber nur durchlaufen 
werden, wenn keine andere Bedingung erfüllt ist, und nicht bei jeder 
richtigen UND falschen Bedingung.

Ist das ein Kompilerfehler oder habe ich einen Fehler zwischen den 
Ohren?
Wie erreiche ich, dass der default Zweig NUR bei ungültigen Zeichen 
durchlaufen wird?

P.S.
Ich verwende avr-gcc (WinAVR 20080610) 4.3.0

[Sven (Gast)]

Das sieht eher nach einem Problem in der Datenübertragung aus. Z.b. 
verwendet dein Terminal 9 Bit, statt 8. o.ä. so dass dein µC denkt, es 
würde 2 Zeichen empfangen. setze mal dort an...

Zum debuggen: Einfach mal vor das Switch ein:

char str[8];
sprintf(str, "RX: %02X\n\r", Befehl);
printf(str);

[Sven P. (Gast)]

Wie sendest du denn die Zeichen an den Controller? Am Terminal das 
Zeichen eigeben und Enter drücken..? --> Zwei Zeichen!

[Karl H. (kbuchegg)]

Jede Wette du empfängst in Wirklichkeit nicht 1 Zeichen sondern 2. Zb 
ein abschliessendes Return nach dem Buchstaben.

Lass dir doch einfach mal den 'Befehlsbuchstaben' zurückgeben (als 
Zeichen und zusätzlich den ASCII Code in Zahlenform) dann siehst du was 
dein Programm wirklich zu Gesicht bekommt.

[Klaus (Gast)]

Ich bin immer wieder erstaunt, das fast alle Kompilerfehler von 
Anfängern gefunden werden...

Oder ist es vielleicht doch einfach nur die Selbstüberschätzung der 
Anfänger, die immer zuerst dem Kompiler die Schuld geben?

[Karl H. (kbuchegg)]

Klaus schrieb:
> Oder ist es vielleicht doch einfach nur die Selbstüberschätzung der
> Anfänger, die immer zuerst dem Kompiler die Schuld geben?

Ich unterstelle niemandem in dem Bereich eine böse Absicht.
Es ist einfach der Lernprozess, dass unser vermeintlich gut 
funktionierendes Gehirn ganz schnell den Überblick über vermeintlich 
einfache Abläufe verliert.

[(nein) (ich) (bin) (kein) (Gast)]

versuche es mit einem lokalen funktionsblock, also {} nach jedem case


quasi case 'a': {//weil hier mehr als ein Befehl kommt}

[(nein) (ich) (bin) (kein) (Gast)]

wenn du ein oszi hast, siehst du, ob zwei zeichen gesendet werden...
ist noch viel wahrscheinlicher, als dass du {} verwenden musst.

[Matthias R. (matsch)]

> Ich bin immer wieder erstaunt, das fast alle Kompilerfehler von
> Anfängern gefunden werden...
@Klaus
OK, ich bin nicht der Profi, und habe auch nicht wirklich an einen 
Kompilierfehler geglaubt. Die echten Fehler des Kompilers sind bestimmt 
nicht von so einfacher Natur. Darum auch das '?'. Dafür kann ich besser 
Minigolf ;-)

@Sven, K-H.
Richtig, ich sende nicht nur ein Zeichen sondern eine Zeichenkette mit 
abschießendem 'CR' (ist die Ende Kennung). Zusammen bis zu 4 Zeichen 
(das CR mitgezählt.
Die Routine wird eigentlich aber erst aufgerufen, wenn die Ende Kennung 
gefunden wurde (so hofft der Anfänger).
Ich werde eure Tipps mal verfolgen. Danke schon mal. Ist echt gut, dass 
einige von euch hier auf andere Bäume hinweisen, wenn mann den Wald 
nicht mehr sieht.... Wobei ich auch immer für eine spitzfindige 
Bemerkung offen bin.

MfG
Matthias

[Matthias R. (matsch)]

Karl heinz Buchegger schrieb:
> Jede Wette du empfängst in Wirklichkeit nicht 1 Zeichen sondern 2. Zb
> ein abschliessendes Return nach dem Buchstaben.

Vor der Switch Anweisung kopiere ich mir das erste Zeichen des Strings 
in die Variable 'Befehl':

Befehl  = String[0]; // erstet Zeichen aus dem String holen
Parameter1 = String[1]; // zweites Zeichen aus dem String holen

switch (Befehl){
... siehe oben ...
}

der default Zweig wird aber wie beschrieben durchlaufen.

matsch

[Karl H. (kbuchegg)]

Matthias R. schrieb:

> Ich werde eure Tipps mal verfolgen.

Der Haupttip ist immer:
Spick dein Programm mit Ausgaben. Wenn du nicht weißt, warum eine 
bestimmte Aktion ausgelöst wird, dann mach vor den entscheidenden if 
(oder in deinem Fall switch) eine Ausgabe rein, die dir den 
'Entscheidungsträger' anzeigt.

In deinem Fall hätte sich auch angeboten ...

 default:
   uart_putc("->'");
   uart_putc(Befehl);
   uart_putc("'"); 
   uart_puts("nop\r");
   break;

... damit du nicht nur siehst, dass etwas nicht verarbeitbares in den 
Verteiler gekommen ist, sondern auch noch siehst was das den war.
Bei einer 'Fehlermeldung' ist es immer gut, den Auslöser auch mit 
auszugeben. Die Anzahl der von Micorosoft mit der simplen Fehlermeldung 
in die Verzweiflung getriebenen Programmierer "Eine DLL konnte nicht 
gefunden werden" ist legendär  .... Verdammt - WELCHE?

[Karl H. (kbuchegg)]

Matthias R. schrieb:

> der default Zweig wird aber wie beschrieben durchlaufen.

Der ganze switch-case wird dann aber wahrscheinlich in einer 
übergeordneten Schleifenstruktur ein zweites mal aufgerufen.

Mach mal folgendes:

switch (Befehl){
 case 'D' :                      
   uart_puts("ein D\r");
   break;
 case 'R' :                      
   uart_puts("ein R\r");
   break;
 case 'W' :
   uart_puts("ein W\r");
   break;
 case 'P' :                      
   uart_puts("ein P\r");
   break;
 default:
   uart_puts("nop\r");
   break;
}

uart_puts( "Fertig\r" );   // <-

[Matthias R. (matsch)]

Karl heinz Buchegger schrieb:
> ... sondern auch noch siehst was das den war.
Danke für den Tipp, den Fehler mit auszugeben.

Die Funktion wurde tatsächlich immer zwei mal aufgerufen. Fehler 
erkannt, gefunden, behoben ... und der Anfänger hat sogar das 'warum' 
verstanden.

Danke
Matsch

[Simon K. (simon)]

(nein) (ich) (bin) (kein) schrieb:
> versuche es mit einem lokalen funktionsblock, also {} nach jedem case
>
>
> quasi case 'a': {//weil hier mehr als ein Befehl kommt}

So ein Quatsch!

[Εrnst B. (ernst)]

das mit dem "Kompilerfehler" im Betreff ist schon richtig...

Auf keine andere Weise kriegt man im GCC-Forum so schnell gute und 
fundierte Antworten.
Mit dem Betreff "Wie funktioniert default: in switch" wären nur 
haufenweise Antworten wie "Kauf dir ein C-Buch" gekommen