Probleme mit Switch/Case Abfrage

Wer B. Gast 07.05.2016 11:50 #4569752

Lesenswert?

•

Hallo,

ich arbeite zurzeit an einem "Protokol" für die Übertragung von 433Mhz 
Signalen, damit es geordnet ist (synchro zwischen Arduinos).

Problem zurzeit: Der Arduino führt bei einer Switch/Case Abfrage ein 
"case" durch, obwohl die Bedingung gar nicht stimmt... ?!
Er macht "case statRec", obwohl er das eig. nicht machen dürfte.

Programmcode im Anhang

Der Fehler tritt ab "case statId" auf, vorher nicht...

Ich würde mich freuen wenn mir einer helfen könnte, denn dieser, zurzeit 
kleine Fehler, könnte später zu einen großen Problem werden...

Viele Grüße

Simon

Angehängte Dateien:

ProtocolReceiver.ino (3,02 KB) | Codeansicht

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Wer B. Gast 07.05.2016 11:56 #4569754

Lesenswert?

•

▲
▼

Mir ist gerade aufgefallen, das ich einen Rechtschreibfehler hatte.

Nicht: starRec
Sonder: statRec

Warum die Arduino IDE da nichts sagt...

Trotz der Korrektur geht es immer noch nicht

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Torsten R. (torstenrobitzki) (Firma: Torrox.de) 07.05.2016 11:58 #4569756

Lesenswert?

•

▲
▼

Wenn Du einen Fehler findest, der Compiler aber nicht, könnte es daran 
liegen, dass Du und der Compiler auf unterschiedlichen Code gucken.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Wer B. Gast 07.05.2016 11:59 #4569758

Lesenswert?

•

▲
▼

Torsten R. schrieb:
> Wenn Du einen Fehler findest, der Compiler aber nicht, könnte es daran
> liegen, dass Du und der Compiler auf unterschiedlichen Code gucken.

Daran liegt es nicht

Antwort Zitieren Bearbeiten Löschen Beitrag melden

le.do Gast 07.05.2016 12:03 #4569760

Lesenswert?

•

▲
▼

StatId hat nur im inneren switch genug  breaks. Das äußere fällt durch 
zu statRec

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Joachim B. (jar) 07.05.2016 13:06 #4569807

Lesenswert?

•

▲
▼

le.do schrieb:
> StatId hat nur im inneren switch genug  breaks

gut gesehen!

   }

break; // fehlt

   case starRec:

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Wer B. Gast 08.05.2016 21:55 #4571247

Lesenswert?

•

▲
▼

Danke, dass war es!
Schade dass die IDE das nicht überprüft...

Vielen Dank

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Programmiersprachentheaterintendant Gast 08.05.2016 22:20 #4571276

Lesenswert?

•

▲
▼

> Schade dass die IDE das nicht überprüft...

Was soll überprüft werden?

Die *switch*-Kontrollstruktur ist in C/C++ nun mal so definiert dass 
v.o.n.u. beim ersten passenden case "eingestiegen" wird und 
anschliessend alles darunter ausgeführt wird, ausser es wird ein 
break angetroffen, dann wird die Kontrollstruktur beendet.

Hat man das Gefühl, switch sei sowas wie "1-aus-n" so liegt man 
falsch.

Man kann mit switch ein "1-aus-n" machen, wenn kein break 
vergessen geht. Jedoch kann kein Compiler/IDE/usw. erraten, wieviele 
breaks richtig sein sollen...

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Peter D. (peda) 09.05.2016 10:31 #4571598

Lesenswert?

•

▲
▼

Der : hinter dem Case verrät eigentlich schon, daß es sich um ein 
Sprung-Label handelt.
Daher ist es nur konsequent, wenn auch dahin gesprungen wird. Und 
vorhergehender Code läuft immer in ein Sprung-Label hinein.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Patrick C. (pcrom) 09.05.2016 10:49 #4571611

Lesenswert?

•

▲
▼

Im zukunft kannst du fuer solche state-machines am besten ein enum fuer 
die states benutzen. Dann kann der kompiler detectieren ob alle 
moegliche states eingefuellt sind, sonnst gibt er ein warning.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Thread beobachten

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

Bestehender Account

oder

Mit Google-Account einloggen

Noch kein Account?

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren