Sollen statische Variablen durch Enums ersetzt werden?

Question

So wurde ich zu einem bestimmten Code suchen, der in und ich geprüft habe alle rätseln:

// Amount of days before cancellation can't be done 
enum Cancellation { Limit = 2 }; 

den Mann zu fragen, wer sie eingecheckt er argumentiert, dass es viel besser ist, Aufzählungen zu verwenden anstelle von statischen Variablen, bettern als dies:

private static int CANCELLATION_LIMIT = 2; 

Also begannen wir zu streiten. Mein Argument war, dass er enum als Möglichkeit zum Speichern von Werten verwendete (es würde brechen, wenn es zwei enum-Symbole mit dem gleichen Wert gäbe). Er argumentierte, es sei ein Antipattern, statische Variablen in einer Klasse zu haben.

Meine Frage ist, welche Best Practice sollte für beide verwendet werden?

Answer 1

Aufzählungen eingegeben werden.

Das heißt, wenn Sie eine Methode haben, bei der Sie beispielsweise einen bestimmten "Zustand" an eine Methode übergeben müssen, können Sie nur "gültige" Argumente übergeben. Zum Beispiel:

enum OrderState 
{ 
    pending = 1, 
    shipped = 2 
} 

public IList<Order> GetOrdersInState(OrderState) 
{ 
} 

Dies ist ein gutes Beispiel -imho- Aufzählungen zu verwenden. Wenn OrderState ein Int ist, für den Sie 2 Konstanten erstellen, haben Sie keine Einschränkungen und können ungültige Werte übergeben. Der Compiler wird sich nicht beschweren.

Wie auch immer, der Fall, den Sie ansprechen, ich denke, Enums verwenden ist keine gültige Lösung. Es ist ein Missbrauch der Verwendung eines Int, und ein Const Int sollte verwendet werden.

Enums sind gut, aber sie sollten verwendet werden, wo sie verwendet werden müssen. Sie sind nicht das bevorzugte Werkzeug in jeder Situation. Eine const oder statische Var ist in diesem Fall kein Antipattern.

Answer 2

Ich denke, dass Sie Enums verwenden sollten, wenn Sie eine Reihe von Werten direkt verbunden haben.

So etwas wie: enum Status {Offen = 1, Geschlossen = 2, Warten = 3};

Für alles andere, würde ich sagen, statische Variablen sind der Weg zu gehen.

Answer 3

Ich weiß nicht, dass es ein Anti-Pattern ist, statische Variablen in einer Klasse (?) Zu haben. Die Color-Klasse im .NET-Framework enthält beispielsweise viele statische öffentliche Variablen, z. B. Color.Red. Aus dieser Perspektive würde ich Ihnen zustimmen.

Allerdings kann es einen Kompromiss geben: private const verwenden CANCELLATION_LIMIT = 2; und beide von euch sollten glücklich sein. Für ihn wird es keine globale Variable für die Klasse (?) Geben, da die Constants durch den Compiler ersetzt werden, und Sie erhalten einen einzigen Punkt der Änderung mit einem eindeutigen Namen.

Answer 4

zurück "Ist es logisch eine Reihe von Werten"? "Enum ist angemessen": "Static const ist in Ordnung"

(ich ein großer Fan der logisch konsistent bin)

Answer 5

Nein, wie definieren Sie statische String-Variablen oder Dezimalwerte in enum?

Answer 6

Ich glaube nicht, dass CANELLATION_LIMIT klingt wie eine Enum, die in der Regel eine Reihe von Möglichkeiten ist.

Für etwas anderes, wenn es ein const war, dann vielleicht ... aber derzeit ist es ein veränderbares Feld?

Beachten Sie, dass Aufzählungen beschränkt sind auf ganzzahlige-basierte Typen, so kann es nicht für float, string usw.

Answer 7

Für unveränderliche Werte verwendet werden soll eindeutig sein, Aufzählungen sind der Weg zu gehen. Die Frage ist einfach: Soll das Objekt den Wert selbst statisch speichern? In vielen Fällen, etwa bei der Beschreibung von Fehlern oder Handlungen, lautet die Antwort nein. Denken Sie daran, enum s wurde als Ersatz für #define geboren: es verbindet typische Werte mit Bezeichnern und es bietet einen Typ, es heißt eigentlich nicht "diese Konstante hier speichern".

Ich nehme an, Sie wollen eigentlich nichts speichern, aber solche typischen Werte liefern. statische Konstanten Mitglieder sind nur nützlich, wenn Sie sie als solche verwenden möchten, z. B. wenn Sie sie anhand einer Methode übergeben müssen.