[C#] 3 statussen met enum

[Verwijderd]

Hallo,

Normaal als ik van een product, object, wat dan ook aan probeer te geven of het actief is of niet is dat makkelijk, aangezien er maar 2 statussen zijn. In mijn geval specifiek:

code:

h => h.IsActive ? "Actief" : "Inactief"

Echter nu is het zo dat er een status bijgekomen is genaamd "In behandeling". Dan werkt mijn normale oplossing uiteraard niet.

Na wat zoeken kwam ik op de suggestie om 'enum' te gebruiken. Dus in dit geval:

code:

public enum IsActive
{
Actief = 0,
In behandeling = 1,
Inactief = 2
}

Verder dan dit kom ik echter niet. Ik wil, afhankelijk van de status, iets uitvoeren. Net zoals ik deed toen ik bool gebruikte als er maar 2 statussen waren.

Ongetwijfeld een hele simpele oplossing maar ik kom er niet meer uit.

Ik hoop dat jullie mij kunnen helpen hiermee.

[Cloud]

Ik zou de naamgeving van je enum wel wat aanpassen, want de huidige naam doet vermoeden dat er maar twee statussen mogelijk zijn (zoals je vroeger had).

Stel je noemt het veld van het object waarin de enum staat State en je noemt je enum States.

C#:

switch (h.State) 
{
  case Actief:
    // doe actief ding
    break;
  case InBehandeling:
    //doe behandeling ding
    break;
  case Inactief:
    // doe inactief ding
    break;
}

Of als je je oude vraag in de nieuwe situatie opnieuw wil stellen:

C#:

if (h.State == States.Actief)
  // doe je actieve ding

Enums zijn ideaal voor wat jij nodig hebt en best eenvoudig in gebruik Het is daarnaast slim om per enum meteen al de integer waarde op te geven, zodat die vast staat mochten er in de toekomst wijzigingen gedaan worden.

[RobIII]

Switch statement gebruiken.

C#:

switch (h.Status)
  case Status.Actief:
    //blablabla
    break;
  case Status.In_behandeling:
    //blablabla
    break;
  case Status.Inactief:
    //blablabla
    break;
}

Ah, cloud was/is me voor

Als je dan toch staat op een ternary:

C#:

h => h.State == State.In_behandeling ? "In behandeling" : h.State == State.InActive ? "Inactief" : "Actief"

Zoiets Puur onleesbare code
Beter gebruik je, zoals Woy hieronder zegt, voor weergave doeleinden een DisplayNameAttribute (die overigens, helaas, niet op Enums van toepassing is, maar makkelijk zelf te implementeren) of desnoods gewoon een .ToString() op je enum.

[ Voor 83% gewijzigd door RobIII op 04-06-2014 12:19 ]

[Woy]

Sowieso is het niet mogelijk om spaties in je enum waardes op te nemen, maar op een enums kan je gewoon vergelijken met bijvoorbeeld de == operator, of je kunt ze in switches gebruiken.

C#:

enum Status { Actief, InBehandeling, InActief, .... }

var myStatus = Status.Actief;

if( myStatus == Status.Actief )
    //DoSomething;

switch(myStatus)
{
   case Status.Actief:
       //DoSomething
        break;
   case Status.InBehandeling:
       //DoSomething
        break;
   case Status.InActief:
       //DoSomething
        break;
/* Als de vorige cases niet geraakt zijn */
    default:
         break;
}

Als het puur om het weergeven van de naam van de status gaat kun je eens kijken naar het gebruik van Attributes zoals DisplayNameAttribute

edit:

/spuit 11

[ Voor 10% gewijzigd door Woy op 04-06-2014 12:13 ]

[Vishari Beduk]

C#:

public string AfhankelijkVanDeStatusIetsUitvoeren(IsActive deStatus)
{
    switch (deStatus)
    {
        case IsActive.Actief: return "Actief";
        case IsActive.InBehandeling: return "InBehandeling";
        case IsActive.Inactief: return "Inactief";
    }

    throw new NotImplementedException();
}

te laat

[ Voor 4% gewijzigd door Vishari Beduk op 04-06-2014 12:21 . Reden: nog een spuit 11 ]

[farlane]

Als je merkt dat je op veel plekken staat te switchen kun je de constructie ook vervangen door polymorfisme, dwz voor elke case een aparte class die een interface implementeren.

[Verwijderd]

- Ik zag de updates niet -

[ Voor 96% gewijzigd door Verwijderd op 04-06-2014 13:32 ]

[Guldan]

Eens met de antwoorden hierboven maar ik zou dan wel in de default een ArgumentOutOfRange exception gooien. Dan krijg je namelijk wat sneller door dat iemand je enum heeft gewijzigd:

C#:

switch(myStatus)
{
   case Status.Actief:
       //DoSomething
        break;
//snip
/* Als de vorige cases niet geraakt zijn */
    default:
      throw new ArgumentOutOfRangeException("myStatus");
     break;
}

De enige opmerking die ik heb over mijn eigen code is dat ik het mooier zou vinden om iets meer uitleg te geven in de argumentOutOfRangeException maarja resharper wil het graag zo .

[BasieP]

Een nog betere oplossing is een status 'unknown' (= 0)' toevoegen, en deze ook in je switch statement zetten.
Je voldoet dan aan:
MSDN: CA1008: Enums should have zero value

en een niet gesette enum is dan de waarde 'unknown' en niet 'actief' wat bij jou het geval is

[D-Raven]

Een nullable enum is ook een optie. Deze gebruik ik persoonlijk liever. Ben sowieso geen fan van Enum's. Omdat deze vaak leiden tot oneindige aantal switch statements.

[Corniel]

De labels in een resource file of attribuut van je enum zetten (eventueel in een dictionary). Geen switches meer, een duidelijke scheiden van layout en logica.