c# - 如何重构一组丑陋的 if 语句？

Question

我有这组 If 语句，它工作正常，但看起来有点难看。有什么办法可以让我开始而不是使用大而难看的 if 语句吗？问题是它使用 string.contains 这使得实现字典有点困难，我尝试过但失败了:(

这里是:

foreach (var Item in Stuff)
            {
                var loweredQuoteItem = quoteItem.Name.ToLower();

                if (loweredQuoteItem.Contains("one"))
                    Item.InsurerNameShort = "Item One";

                if (loweredQuoteItem.Contains("two"))
                    Item.InsurerNameShort = "Item Two";

                if (loweredQuoteItem.Contains("chaucer"))
                    Item.InsurerNameShort = "Chaucer";

                if (loweredQuoteItem.Contains("three"))
                    Item.InsurerNameShort = "Item Three";

                if (loweredQuoteItem.Contains("four"))
                    Item.InsurerNameShort = "Item Four";

                if (loweredQuoteItem.Contains("five"))
                    Item.InsurerNameShort = "Item Five";

               }

Answer 1

The problem is that it uses string.contains which makes it a bit difficult to implement a dictionary, which I tried and failed :(

那我猜你的实现方式不正确。这是你应该做的:

• 构建字典。
• 将“Item One”作为键“one”的值
• 将“Item Two”作为键“two”的值
• 将“Item Two”作为键“chaucer”的值
• 等等...
• 在当前循环中，一旦获得了 loweredQuoteItem，就循环遍历字典。
  • 如果字典包含 innerLoopKey，请将 Item.InsurerNameShort 设置为 innerLoopValue(并可选择中断)

确保在 foreach (var Item in Stuff) 循环之外构造此字典，以获得更好的性能。