Brechen Sie auf Standardfall in Schalter

Ich bin ein bisschen verwirrt, wann immer oder wann nicht breaknach dem letzten Fall, oft default.

switch (type) {
    case 'product':

        // Do behavior

        break;
    default:

        // Do default behavior

        break; // Is it considered to be needed?
}

breakMeines switchErachtens besteht der einzige Zweck darin, zu verhindern, dass der Code im Rest des Falls ausgeführt wird.

Wird es dann als logischer angesehen, einen breakLeisten aufgrund von Konsistenz zu haben oder ihn zu überspringen, weil breaküberhaupt keine funktionale Verwendung angewendet wird? Beides ist meiner Meinung nach auf unterschiedliche Weise logisch.

Dies könnte bis zu einem gewissen Grad mit dem Beenden einer .phpDatei mit verglichen werden ?>. Ich ende nie mit, ?>hauptsächlich wegen des Risikos, Leerzeichen auszugeben, aber man könnte argumentieren, dass es die logische Sache wäre, die Datei mit zu beenden.

breakwird nach der letzten Alternative technisch nicht benötigt (was wohlgemerkt nicht sein muss default: Es ist vollkommen legal und manchmal sogar nützlich, den defaultZweig an die erste Stelle zu setzen ); ob Ihr Code durch das Ende der switchAnweisung fällt oder breaksam Ende seiner letzten Verzweigung heraus, hat das gleiche Ergebnis.

Aus drei Gründen würde ich dennoch jeden Zweig, einschließlich des letzten, mit einem returnoder beenden break:

1. Refactorability. Wenn alle Ihre Zweige mit breakoder enden return, können Sie sie neu anordnen, ohne die Bedeutung zu ändern. Dies macht es weniger wahrscheinlich, dass eine solche Neuordnung eine Regression einführt.
2. Konsistenz und geringste Überraschung. Konsistenz besagt, dass Ihre Zweige konsistent enden sollten, es sei denn, sie haben tatsächlich eine andere Bedeutung. Das Prinzip der geringsten Überraschung schreibt vor, dass ähnliche Dinge ähnlich aussehen sollten. Das Beenden des letzten Zweigs eines switchBlocks, genau wie der vorhergehende, erfüllt beides, was das Lesen und Verstehen erleichtert. Wenn Sie das Explizite weglassen break, wird der letzte Zweig optisch anders sein (was besonders für das schnelle Scannen wichtig ist), und um zu sehen, dass es wirklich nicht anders ist, muss der Leser zum Kern des Lesens einzelner Aussagen absteigen .
3. Schütze dich. Wenn Sie es sich zur Gewohnheit machen, alle Ihre switchZweige mit einem zu beenden break, wird es nach einer Weile automatisch und Sie werden es seltener versehentlich vergessen, wenn es darauf ankommt. Wenn Sie sich schulen, breakum das Ende jedes Zweigs zu erwarten, können Sie auch fehlende breakAnweisungen erkennen, was sich hervorragend zum Debuggen und zur Fehlerbehebung eignet.

In Anbetracht der Mehrdeutigkeit, die bei der Verwendung von switch-casein den meisten Sprachen besteht, würde ichbreak bei der Verwendung vorschlagen, immer eine Anweisung zu verwenden, es sei denn, dies ist ausdrücklich und beabsichtigt unerwünscht .

Dies liegt zum Teil daran, dass jeder caseAnruf gleich aussieht, was meiner Meinung nach die Lesbarkeit verbessern würde. Es bedeutet aber auch, dass sich jemand (auch Sie), caseder zu einem späteren Zeitpunkt ein nach dem letzten einfügen möchte, nicht mit der Überprüfung des vorherigen Blocks befassen muss, was dazu beitragen kann, Fehler beim Hinzufügen von neuem Code zu reduzieren.

Es ist breaknach dem letzten Fall nicht erforderlich . Ich benutze das Wort " last " (nicht Standard ), weil es nicht notwendig ist, Standard ist der letzte Fall.

switch(x)
{
case 1:
//do stuff
break;

default:
//do default work
break;

case 3:
//do stuff

}

Und wir wissen, a break ist zwischen zwei aufeinanderfolgenden cases notwendig . Manchmal verwende ich if(a!=0)in meinem Code, um die Lesbarkeit zu verbessern, wenn andere auf meinen Code verweisen. Ich kann mich für die Verwendung entscheiden if(a), das wäre eine Frage meiner Wahl