So gruppieren Windows-API

Question

Konstanten Wenn const Werte Windows-API zu definieren, ist es besser, sie zu haben als const

public const int SW_HIDE = 0; 
public const int SW_SHOWNORMAL = 1; 
public const int SW_NORMAL = 1; 
public const int SW_SHOWMINIMIZED = 2; 
public const int SW_SHOWMAXIMIZED = 3; 
public const int SW_MAXIMIZE = 3; 
public const int SW_SHOWNOACTIVATE = 4; 
public const int SW_SHOW = 5; 
public const int SW_MINIMIZE = 6; 
public const int SW_SHOWMINNOACTIVE = 7; 
public const int SW_SHOWNA = 8; 
public const int SW_RESTORE = 9; 
public const int SW_SHOWDEFAULT = 10; 
public const int SW_MAX = 10; 

[DllImport("user32.dll")] 
public static extern bool ShowWindow(HandleRef hWnd, int nCmdShow); 

oder sie zusammen als eine Enumeration zu gruppieren.

public enum SW { 
    SW_HIDE = 0, 
    SW_SHOWNORMAL = 1, 
    SW_NORMAL = 1, 
    SW_SHOWMINIMIZED = 2, 
    SW_SHOWMAXIMIZED = 3, 
    SW_MAXIMIZE = 3, 
    SW_SHOWNOACTIVATE = 4, 
    SW_SHOW = 5, 
    SW_MINIMIZE = 6, 
    SW_SHOWMINNOACTIVE = 7, 
    SW_SHOWNA = 8, 
    SW_RESTORE = 9, 
    SW_SHOWDEFAULT = 10, 
    SW_MAX = 10 
} 

[DllImport("user32.dll")] 
public static extern bool ShowWindow(HandleRef hWnd, SW nCmdShow); 

Answer 1

Gruppieren Sie sie als enums.

Warum? Ints werden überall benutzt und man kann sie dort passieren, wo zum Beispiel auch eine Größe benötigt wird. Das führte an erster Stelle zu der ungarischen Notation (szSomething ..). Das Typsystem fehlte und sie versuchten es mit dem Variablennamenschema zu "reparieren". Sie sind jetzt besser dran, mit einem besseren Typensystem; benutze es.

Definieren Sie enums, gruppieren Sie sie auf eine vernünftige Art und Sie werden nicht Thread.Sleep (WM_User) eines Tages (Ja, ich bin nicht ganz ernst mit diesem Beispiel, aber ich denke, Sie bekommen den Punkt).

Answer 2

Abgesehen von der Wartbarkeit des Codes spielt dies keine Rolle.

Ich empfehle die Verwendung einer enum; Dadurch können Sie IntelliSense beim Aufrufen der Funktion verwenden und Fehler vermeiden.
Sie sollten jedoch Ihrem Enum einen aussagekräftigen Namen geben, z. B. WindowShowType.
Vielleicht möchten Sie auch die Präfixe entfernen und vielleicht die Namen zu CamelCase standardisieren.