我如何对以下意外事件进行编程？

Question

我有一个包含宠物名称的数组和另一个包含它们发出的声音名称的并行数组。 我正在编写一个ShowSounds()方法，它要求用户输入动物名称，然后显示相应的声音。

如果用户随机输入某些内容，我如何在错误消息中编码“抱歉，该动物不在我们的列表中” ？

我现在遇到的问题是，对于if语句，即使我输入了正确的动物，它也会显示四次错误消息。

public static void ShowSound(string userInput2, string[] localPets4, string[] localSounds2)
{
    for (int l = 0; l < localPets4.Length; l++)
    {
        if (userInput2 == localPets4[l])
        {
            Console.WriteLine("{0} makes the sound {1}", localPets4[l], localSounds2[l]);
            Console.WriteLine();
        }
        else
        {
            Console.WriteLine("Sorry that item isn't in our list of animals");
        }
    }
}

Answer 1

试试这个：

public static void ShowSound(string userInput2, string[] localPets4, string[] localSounds2)
{
    var result =
        localPets4
            .Zip(localSounds2, (pet, sound) => new { pet, sound })
            .Where(x => x.pet == userInput2)
            .FirstOrDefault();

    if (result != null)
    {
        Console.WriteLine($"{result.pet} makes the sound {result.sound}");
        Console.WriteLine();
    }
    else
    {
        Console.WriteLine($"Sorry, {userInput2} isn't in our list of animals");
    }
}

localPets4.Zip(localSounds2, (pet, sound) => new { pet, sound })行是localPets4和localSounds2数组中元素的匹配元素，并以new { pet, sound }的形式创建单个枚举。

.Where(x => x.pet == userInput2)只保留pet等于userInput2的元素。

这应该意味着，如果有匹配项，则只有一个匹配项，如果没有匹配项，则可枚举项为空。 调用.FirstOrDefault()然后你有一个new { pet, sound }来管理输入或者你有null 。

其余代码只是根据result生成输出。

Answer 2

你可以使用Array.Exists(localPets4, x => x.Equals(userInput2)); 检查输入是否有效，然后是否循环查找声音。

if (Array.Exists(localPets4, x => x.Equals(userInput2))) {
    for (int l = 0; l < localPets4.Length; l++)
    {
        if (localPets4[l].Equals(userInput2) {
            Console.WriteLine("{0} makes the sound {1}", localPets4[l], localSounds2[l]);
            Console.WriteLine();
            break;
        }
    }
}
else
    {
         Console.WriteLine("Sorry that item isn't in our list of animals");
    }

但这只会检查 unserInput2 是否完全匹配。 如果您需要近似匹配，则必须更详细一些。

Answer 3

由于您只想打印该语句一次（如果未找到任何项目），则需要将其从循环中删除：

public static void ShowSound(string userInput2, string[] localPets4, string[] localSounds2) 
{ 
    bool itemFound = false;

    for (int l = 0; l < localPets4.Length; l++) 
    { 
        if (userInput2 == localPets4[l])
        { 
            itemFound = true;
            Console.WriteLine("{0} makes the sound {1}", localPets4[l], localSounds2[l]);
            Console.WriteLine(); 
        }
    } 

    if (!itemFound)
    {
        Console.WriteLine("Sorry that item isn't in our list of animals"); 
    }
}

Answer 4

这与数组的遍历有关。 如果string[] localPets4包含 4 个元素，您的for loop将迭代 4 次 - 因此会出现 4 条错误消息。 因此，在您的 for 循环中，您应该只进行搜索。 然后反馈用户的逻辑 - 成功/失败 - 将在for loop之外处理。

int foundAtIndex = -1;

for (int l = 0; l < localPets4.Length; l++) 
{ 
    if (userInput2 == localPets4[l])
    { 
        foundAtIndex = l;
    }
}
if (foundAtIndex != -1)
{
        Console.WriteLine("{0} makes the sound {1}", localPets4[foundAtIndex ],localSounds2[foundAtIndex ]);
        Console.WriteLine(); 
}
else 
{
    Console.WriteLine($"Sorry, {userInput2} isn't in our list of animals");
    Console.WriteLine(); 
}

由于这是一对一的关系，您可以使用[Dictionary][1] ，但我不确定这是否超出了您的应用程序范围。