Switch-Anweisungen sind schlecht?

Ein-Schalter ist wie jedes andere Steuerelement-Struktur. Es gibt Orte, wo es die beste/sauberste Lösung, und vielen weiteren Orten, wo es völlig unangebracht. Es ist nur missbraucht, weit mehr als andere Kontroll-Strukturen.

In OO-design, ist es im Allgemeinen vorzuziehen, als in einer situation, wie Ihnen der Einsatz von verschiedenen message-Typen/Klassen Erben von einer gemeinsamen Klasse Nachricht, dann verwenden Sie überladene Methoden, um "automatisch" die Unterscheidung zwischen den verschiedenen Arten.

In einem Fall wie dem Ihren, man könnte eine Aufzählung, die Karten zu Ihrem action-codes, fügen Sie dann ein Attribut zu jedem nummerierten Wert, das Sie verwenden, Generika oder Typ-Gebäude zu bauen, die verschiedenen sub-class-Objekte, so dass die überlastung Methode funktioniert.

Aber das ist eine echte Schmerzen.

Bewerten, ob es eine design-option, wie die Aufzählung, die möglich ist und Ihre Lösung. Wenn nicht, verwenden Sie einfach den Schalter.

InformationsquelleAutor Toby

Den Strategie Muster in den Sinn kommt.

Ist das strategy-Muster wird gedacht, um ein Mittel zu definieren, die eine Familie von algorithmen zu Kapseln jeweils als ein Objekt, und mache Sie austauschbar. Das Strategie-Muster können die algorithmen unabhängig voneinander variieren von clients, die Sie verwenden.

In diesem Fall die "Familie von algorithmen" sind Ihre verschiedenen Aktionen.

Als für switch-Anweisungen - in "Clean Code" von Robert Martin sagt, dass er versucht, beschränken sich eine switch-Anweisung pro Typ. Nicht beseitigen insgesamt.

Der Grund ist, dass switch-Anweisungen nicht einhalten OCP.

InformationsquelleAutor Oded

IMO switch Aussagen sind nicht bad, sollte aber vermieden werden, wenn möglich. Eine Lösung wäre die Verwendung einer Map wo die Tasten sind die Befehle, und die Werte Command Objekte mit einem execute() Methode. Oder ein List wenn Ihre Befehle sind numerisch und haben keine Lücken.

Jedoch in der Regel, verwenden Sie switch Aussagen, wenn design-Pattern implementiert; ein Beispiel wäre die Verwendung einer Kette der Verantwortung - Muster zu handhaben, die Befehle jeder Befehl "id" oder "Wert". (Die Strategie Muster war auch aufgeführte.) Aber in deinem Fall hast du vielleicht auch einen Blick in die Befehl Muster.

Grundsätzlich in OOP, werden Sie versuchen, andere Lösungen, als sich auf switch Blöcke, die mit einem prozeduralen Programmierparadigma. Jedoch, Wann und wie zu verwenden, entweder etwas ist Ihre Entscheidung. Ich persönlich benutze oft switch Blöcke bei Verwendung der Fabrik Muster etc.

Definition von code-organisation :

• ein Paket ist eine Gruppe von Klassen mit coherant API (ex: Collection - API in vielen frameworks)
• eine Klasse ist ein Satz von kohärenten Funktionalitäten (ex: Math Klasse...
• eine Methode ist eine Funktionalität; Sie sollte das tun, eine Sache und eine Sache nur. (ex: hinzufügen eines Elements in eine Liste benötigen, um zu vergrößern, sagte der Liste, in welchem Fall die add Methode wird verlassen sich auf andere Methoden zu tun, der sich nicht erfüllen wird, die operation selbst, weil es nicht, es ist Vertrag.)

Deshalb, wenn Sie Ihre switch - Anweisung ausführen unterschiedlicher Arten von Operationen, Sie sind "gegen" , definition; in der Erwägung, dass die Verwendung eines design-Muster nicht wie jede operation definiert ist, in seiner eigenen Klasse (eigene Funktionen).

InformationsquelleAutor Yanick Rochon

Befehle verwenden. Wickeln Sie die Aktion in ein Objekt und lassen Polymorphismus tun, der Schalter für Sie. In C++ (shared_ptr ist einfach ein Zeiger oder eine Referenz in Java-Bezug. Es ermöglicht eine dynamic dispatch):

void GameServer::perform_action(shared_ptr<Action> op) {
    op->execute();
}

Kunden wählen eine Aktion aus, und sobald Sie tun, Sie senden, dass die Aktion selbst auf den server, damit der server muss nicht jedem Parsen:

void BlueClient::play() {
    shared_ptr<Action> a;
    if( should_move() ) a = new Move(this, NORTHWEST);
    else if( should_attack() ) a = new Attack(this, EAST);
    else a = Wait(this);
    server.perform_action(a);
}

InformationsquelleAutor wilhelmtell