Question

我有以下C＃代码：

AnimalTypeEnum animal;
string s = Console.ReadLine();
switch (s.ToLower())
{
case "dog":
    animal = AnimalTypeEnum.DOG;
    break;
case "cat":
    animal = AnimalTypeEnum.CAT;
    break;
case "rabbit":
    animal = AnimalTypeEnum.RABBIT;
    break;
}

Console.WriteLine(animal); #compiler error here

我在最后一行收到此错误：Use of unassigned local variable 'animal'。我知道这是因为animal可能没有取决于用户输入的值，所以我该如何解决？

理想情况下，如果输入了未知的动物类型，我想显示一条错误消息，并让用户再次输入该值。

感谢。

Answer 1

这是修复它的一种方法，使用递归调用而不是需要捕获并抛出异常，或者使用循环（在这种情况下循环会混淆我认为的意思;太多关于你如何＆＃ 39;重新做而不是你正在做的事情：

private static AnimalTypeEnum GetAnimalFromInput()
{
    AnimalTypeEnum animal;
    string s = Console.ReadLine();
    switch (s.ToLower())
    {
        case "dog":
            animal = AnimalTypeEnum.DOG;
            break;
        case "cat":
            animal = AnimalTypeEnum.CAT;
            break;
        case "rabbit":
            animal = AnimalTypeEnum.RABBIT;
            break;
        default:
            Console.WriteLine(s + " is not valid, please try again");
            animal = GetAnimalFromInput();
            break;
    }
    return animal;
}
static void Main(string[] args)
{
    AnimalTypeEnum animal = GetAnimalFromInput();

    Console.WriteLine(animal);
}

我还要注意，将您的交换机重构为if / else链是一种很好的做法，使用if (s.Equals("dog", StringComparison.CurrentCultureIgnoreCase))（或相应的不区分大小写的比较）来保持其在其他地方工作文化。当然，这可能不适用于您的场景（例如测试/家庭作业应用程序，或者只能在您的文化中使用的内容）。

更新：感谢Mennan Kara的想法，如果您的值（例如"dog"）始终与枚举值相匹配（例如DOG），然后您可以使用Enum.TryParse来改进代码：

private static AnimalTypeEnum GetAnimalFromInput()
{
    AnimalTypeEnum animal;
    string s = Console.ReadLine();
    if (Enum.TryParse(s, true, out animal))
        return animal;
    else
    {
        Console.WriteLine(s + " is not valid, please try again");
        return GetAnimalFromInput();
    }
}

如果您需要灵活地将它们分开，请保留现有的开关。

Answer 2

如果s.ToLower()是其他dog，cat或rabbit，则animal没有价值。

对于这种情况，您应该在交换机中添加默认值：

switch (s.ToLower())
{
case "dog":
    animal = AnimalTypeEnum.DOG;
    break;
case "cat":
    animal = AnimalTypeEnum.CAT;
    break;
case "rabbit":
    animal = AnimalTypeEnum.RABBIT;
    break;
default:
    animal = ...
    break;
}

Answer 3

AnimalTypeEnum animal;
var s = Console.ReadLine();
Console.WriteLine(!Enum.TryParse(s, true, out animal) ? "Not a valid animal" : animal.ToString());

Answer 4

对于您的代码未知的动物，您应该有一个默认ENUM。你甚至可以让你的代码学习新的动物。例如。

switch (s.ToLower())
{
default:
    animal = AnimalType.Unkown;
    break;
}

或

default:
    animal = new MakeEnum(s.ToLower());
    myEnumList.Add(animal);
    break;

您的MakeEnum基本上只需要检查当前枚举数的长度，并使用数字或其他参数创建新的枚举。

C＃中switch语句的“使用未分配的局部变量”编译器错误？

4 个答案: