CODEWITHSUNDEEP
c#enumsswitch-statement.net-6.0
K.Z
K.Z

Reputation: 5075

c# Using enum in Switch Case throwing exception

I am working on .NET 6.0 application, I have enum that I am trying to use in switch as to compare with string value but getting exception.

error

![enter code here

private static bool ValidateAlphanumericCase(string text, string fieldName)
    {
        if (!string.IsNullOrWhiteSpace(fieldName))
        {
            var rule = GetRule(fieldName).TxtFieldFormat; // string value

            switch (rule)
            {
                case TextFieldFormat.AlphanumericUpperCase.ToString():
                    break;

                case TextFieldFormat.AlphanumericLowerCase.ToString():
                    break;
            }

        }
        else
        {
            new EmptyFieldNameException();
        }

        return false;
    }

enum

 public enum TextFieldFormat
{
    AlphanumericUpperCase = 0,
    AlphanumericLowerCase = 1,
}

Upvotes: 0

Views: 1192

Answers (3)

Uğur Demirel
Uğur Demirel

Reputation: 81

You can also use it this way

TextFieldFormat.AlphanumericUpperCase.ToString("g" or "G")

Upvotes: 0

sihirbazzz
sihirbazzz

Reputation: 718

Seems you understood switch-case block a little bit different.

A switch block is simply a shortcut (but more performant shortcut) of many if - else if - else blocks. Of course they are not the same, but their working logic almost the same.

With this very little info, you can easily think about what's wrong in your code.

Bingo, you're right! Case blocks should check the state. (Boolean value..Just interests with either the given statement results true or false..)

After checking the boolean result, Which case's statement match, code continues on that case block.

So, in your situation your code could be like this :

switch (rule)
{
    
/// Some statements need to put in paranthesis. Also you would need put business codes of cases into curly braces.
/// I write from my mind.
/// So please try paranthesis and/or braces if this code break.

case rule==TextFieldFormat.AlphanumericUpperCase.ToString():
DoSomethingWithFirstCase(); break;

case rule==TextFieldFormat.AlphanumericLowerCase.ToString():
DoSomethingWitSecondCase(); break;

default: DoSomethingWhenNoMatchWithOtherCases();
}

Hope this helps.

Upvotes: 1

shingo
shingo

Reputation: 27011

TextFieldFormat.AlphanumericUpperCase.ToString()

This is a method invocation expression and it is not a valid pattern for swith statement.

You can find all valid patterns here: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/operators/patterns

The closest pattern is type pattern or constant pattern, I guess the compiler recognizes AlphanumericUpperCase as a nested class of TextFieldFormat and fails.

In this case you can use nameof operator.

 switch (rule)
 {
      case nameof(TextFieldFormat.AlphanumericUpperCase):
          break;
      case nameof(TextFieldFormat.AlphanumericLowerCase):
          break;
 }

Upvotes: 1

Related Questions