Dynamic switch Fällen

Ich versuche, um eine einfache switch case console-Menü für verschiedene Benutzer: admin, moderator, und user. admin hätte create, delete, modify, show Funktionen moderator - create, modify, show Funktionen, und user - create, show Funktionen, zum von zu wählen.

Admin switch case:

if(userType == "admin")
{
    string i = Console.ReadLine();
    switch(i):
    case "create": Console.WriteLine("Created");
                   break;
    case "modify": Console.WriteLine("Modified");
                   break;
    case "delete":Console.WriteLine("Deleted");
                  break;
    case "show":Console.WriteLine("Showed");
                break;
    default: Console.WriteLine("Default");
             break;
}

Moderator switch case:

if(userType == "moderator")
{
    string i = Console.ReadLine();
    switch(i):
    case "create": Console.WriteLine("Created");
                   break;
    case "modify": Console.WriteLine("Modified");
                   break;
    case "show": Console.WriteLine("Showed");
                 break;
    default: Console.WriteLine("Default");
             break;
}

User-switch-case:

if(userType == "user")
{
    string i = Console.ReadLine();
    switch(i):
    case "create": Console.WriteLine("Created");
                   break;
    case "show": Console.WriteLine("Showed");
                 break;
    default: Console.WriteLine("Default");
             break;
}

Gibt es eine Möglichkeit, zu Formen, diese switch-Case-Anweisungen in einem dynamischen wechseln? Wenn ich zu denken oder zu erklären, etwas falsch ist, bitte korrigieren Sie mich.

InformationsquelleAutor Rimantas Radžiūnas | 2015-10-13

c#switch-statement

Das dynamische äquivalent zu einer switch-case ist ein Wörterbuch-lookup. Zum Beispiel:

Dictionary<string, Action> userActions = {
        { "create", () => Console.WriteLine("created") },
        { "show", () => Console.WriteLine("showed") } };
Dictionary<string, Action> adminActions = {
        { "create", () => Console.WriteLine("created") },
        { "show", () => Console.WriteLine("showed") },
        { "delete", () => Console.WriteLine("deleted") } };

Dictionary<string, Dictionary<string, Action>> roleCapabilities = {
        { "user", userActions },
        { "administrator", adminActions } };

roleCapabilities[userType][action]();

Zur Laufzeit, können Sie ganz einfach hinzufügen und entfernen von Aktionen, die zu jeder Rolle (Gruppe).

Zur Umsetzung der "Standard" - Logik, die Sie verwenden würden, so etwas wie:

Action actionCall;
if (roleCapabilities[userType].TryGetValue(action, out actionCall)) {
   actionCall();
}
else {
   //this is the "default" block, the specified action isn't valid for that role
}

InformationsquelleAutor Ben Voigt

Dies ist ein guter Kandidat für die Strategie-Muster.

In der Strategie-Muster, die Funktionalität wird repräsentiert durch ein Objekt, eine Schnittstelle, die übergeben werden können, um. Andere Implementierungen erlauben das Verhalten dynamisch zu ändern.

Beispiel:

interface ConsoleInteractor
{
    void performAction(string action);
}

class UserConsoleInteractor : ConsoleInteractor
{
    public void performAction(string action)
    {
        switch(i)
        {
            case "create": 
                Console.WriteLine("Created");
                break;
            case "show":
                Console.WriteLine("Showed");
                break;
            default: 
                Console.WriteLine("Default");
                break;
        }
    }
}

InformationsquelleAutor sdgfsdh

sind Sie besser dran mit einer Karte von Funktionen oder Aktionen.

var actionsAdmin = new Dictionary<string, Action>{
  {"create", ()=>Console.WriteLine("create")}
  {"modify", ()=>Console.WriteLine("modify")}
}

var actionsUser = new Dictionary<string, Action>{
  {"show", ()=>Console.WriteLine("show")}
  {"foodle", ()=>Console.WriteLine("foodle")}
}

dann wählen Sie die richtige Karte und führen Sie die angegebene Funktion

var action = actionUser[verb];
action();

Great minds think alike 😉 +1 wobei ich denke, dass ein anderes Wörterbuch zum nachschlagen der Rolle ("wählen Sie die richtige Karte" in deinen Worten) ist nützlich.

InformationsquelleAutor pm100

2

Schalter (kein Wortspiel beabsichtigt) es um do-check für die Rolle auf jeden Fall. Und dann, wenn es nicht erlaubt ist, es zu tun, es zu überspringen.
```
string i = Console.ReadLine();
if (allowed(userType, i)){
  switch(i):
  case "create": Console.WriteLine("Created");
      handleCreate();
  break;
  case "show":Console.WriteLine("Showed");
      handleShow();
  break;
  case "delete":Console.WriteLine("Deleted");
      handleDelete();
  break;
  default: Console.WriteLine("Default");
    handleDefault(userType);
  break;
}
```
- Alle, die code-Duplizierung ist nach ärger. Setzen Sie das Kontrollkästchen Zugriff außerhalb der Schalter, und übergeben Sie die variable i statt einem string-literal. Sonst werden Sie eines Tages versehentlich den code ein test für lese-Berechtigungen beim ausführen einer mass löschen.
- Sie sind richtig, mein Punkt war, um den Fokus auf einen einzigen Punkt zu tun, access control.
- Und in den Prozess eingeführt hat, genau die Art von Fehler ist, ich war besorgt. Ist jetzt behoben 🙂
InformationsquelleAutor ergonaut

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.