Wenn Bedingung in Switch-Fall [Duplizieren]

Der Compiler wird nicht verstehen, was Sie hier meinen.

Der Compiler verbindet die Punkte nicht und versteht, dass die break; -Anweisung innerhalb Ihrer if -Anweisung mit der switch -Anweisung verknüpft ist. Stattdessen versucht es, es mit einer Schleife zu verknüpfen, da break; -Anweisungen allein nur mit Schleifen verwendet werden können, um daraus auszubrechen.

Das bedeutet, dass Ihr case -Baustein seine break -Anweisung zur Vervollständigung fehlt, und der Compiler sich beschwert.

Anstatt zu versuchen, den Code aus einer switch -Anweisung zu entfernen, würde ich stattdessen Ihre ursprüngliche if -Anweisung auflösen.

Das gehört Ihnen:

So würde ich es schreiben:

Sie haben nicht , um alles in einer Zeile zu packen.

Ich würde auch versuchen, Eigenschaften so zu benennen, dass sie einfacher zu lesen sind. Ich würde die Eigenschaft EXPENSE in IsExpense umbenennen, so dass der obige Code wie folgt gelesen würde:

Dann würde ich im Idealfall die Sub-Ausdrücke zu Methoden umdefinieren:

Dann können Sie den Ausdruck wieder in die if-Anweisung einfügen:

Dies sollte einfacher zu lesen und später ändern.

Zunächst bemerke ich, dass Sie vergessen haben, in Ihrem zweiten Punkt eine Frage zu stellen. Also werde ich einige Fragen stellen, die Sie an Ihren zweiten Punkt richten:

  

Was bedeutet der Fehler "Kann nicht durchfallen"?

Im Gegensatz zu C und C ++ erlaubt C # kein versehentliches Durchlaufen von einem Switch-Bereich zu einem anderen. Jeder Schaltabschnitt muss einen "unerreichbaren Endpunkt" haben; Es sollte mit einem Break, Goto, Return, Throw oder (selten) Endlosschleife enden.

Dies verhindert den häufigen Fehler des Vergessens, die Pause einzulegen und versehentlich "durchzufallen".

Sie haben Ihren Code so geschrieben, als ob der Durchgriff legal ist; Meine Vermutung ist, dass Sie ein C-Programmierer sind.

  

Wie kann ich Durchgriff in C # erzwingen?

So:

Nun kann der Erreichbarkeitsanalysator feststellen, dass unabhängig davon, welcher Zweig des "if" ist, der Endpunkt des Schalterabschnitts nicht erreichbar ist.

  

Ist das ein guter Stil?

Nein. Ihr ursprünglicher Code war viel besser lesbar.

  

Ich habe gelesen, dass Switch-Statements im Allgemeinen schrecklich sind - Stimmt das?

Meinungen variieren. Switch-Anweisungen sind sehr nützlich, wenn es eine kleine Anzahl sehr "knackiger" Alternativen gibt, deren Verhalten nicht auf komplexe Weise interagiert. Einige Leute werden Ihnen sagen, dass Switched Logik sollte stattdessen durch virtuelle Methoden oder Besucher Muster behandelt werden, aber das kann auch missbraucht werden.

  

Sollte ich in diesem speziellen Fall einen Schalter verwenden?

Ich würde nicht.

  

Wie würden Sie meinen Code verbessern?

Als Erstes sollten Sie in C # keine Dinge IN ALLEN CAPs nennen.

Zweitens, boolesche Werte nicht mit wahr und falsch vergleichen. Sie sind schon Booleans! Wenn du die Wahrheit von Aussage X wissen willst, würdest du nicht auf Englisch sagen "ist es wahr, dass X wahr ist?" Du würdest sagen "Ist X wahr?"

Ich würde wahrscheinlich schreiben:

Oder, noch besser, ich würde den Test in eine eigene Methode abstrahieren:

Oder abstrahiere das Ganze weg:

Verwenden Sie if-Anweisungen und extrahieren Sie komplexe Bedingungen in Methoden, z. B.

Denken Sie immer an OOP und Polymorphismus, wenn Sie einen solchen "Schalter" schreiben und einen weiteren hinzufügen Fall zu diesem Code wird ein Albtraum

siehe dies und ähnliche (C ++) Anweisungen zum Konvertieren von Schaltern

PS Wenn Sie daran interessiert sind, Ihren Code sauber und lesbar zu machen, lesen Sie Smalltalk Best Practice Patterns von Kent Beck und / oder Clean Code von Uncle Bob Ich habe beide sehr genossen, sehr zu empfehlen.

Wenn Sie Lesbarkeit möchten, werfen Sie einfach Ihren Syntax-Papierkorb weg:

Geben Sie den else -Teil für jeden von ihnen an, damit es keinen Fehler auslöst, aber wie andere sagen, brauchen Sie in diesem Fall tatsächlich nicht switch .

Der Grund, warum Sie diesen Fehler erhalten, besteht darin, dass Sie break -Anweisungen nicht definieren.

Sie haben break bedingt definiert.

Stellen Sie entweder sicher, dass jede case-Anweisung ihre eigene break hat oder gruppieren Sie die case-Anweisungen wie folgt.

Refaktorieren Sie die if -Anweisungen, damit Sie es so ausdrücken können:

Die extrahierte Logik:

Stimmen Sie mit Dennis überein, Sie möchten keinen Schalter für dieses Problem.

Obwohl wahrscheinlich weniger lesbar, können Sie auch kürzere verwenden: