摆脱为每种可能的条件编写 if/else if 的习惯

当解析 xml 文档的节点或属性时，如果文档很大，我会有一堆 if 和 else 语句。

显然，从长远来看，100+ ifs 并不能构成可维护的代码。

除了这样做之外，还有其他更好的方法吗？我在 Hanselman 的博客上读到，他的一位朋友也有同样的情况，并且编写了大量 ifs/else if 以及通常很差的代码。汉塞尔曼提供了一些更易于维护的方式的片段，但整个代码不可用，因此很难准确理解（整个情况）正在发生的事情。if、else之后的生活 http://www.hanselman.com/blog/BackToBasicsLifeAfterIfForAndSwitchLikeADataStructuresReminder.aspx

我正在使用 .NET 3.5，因此我可以使用扩展方法和 LINQ 的全部功能。然而，我使用 .NET 2.0 工作，所以也很感激 v2.0 中的任何解决方案。 :)

我的代码看起来与 Hanselman 网站上的问题非常相似：

if (xmlNode.Attributes["a"].Value == "abc" {

}
else if (xmlNode.Attributes["b"].Value == "xyz"
{
wt = MyEnum.Haze;
}

我可以有一个字典，将我正在寻找的值存储为键，也许还有值中的委托（或者我希望在查找所需值时发生的任何事情），所以我可以说 if (containskey) 获取委托并执行它，在伪代码中。

这样的事情一直持续着。显然是非常幼稚的编码方式。我在解析文本文档中的值等时遇到同样的问题。

Thanks

如果您需要地图<condition on xml node> to <change of state> 没有办法避免定义该映射某处。这完全取决于您可以对条件做出多少假设以及在这些条件下您会做什么。我认为字典的想法很好。为了提供尽可能多的灵活性，我会这样开始：

Dictionary<Predicate<XmlNode>, Action> mappings;

然后开始尽可能地简化。例如，您是否经常像示例中那样将 wt 设置为 MyEnum 的值？如果是这样，你想要这样的东西：

Func<MyEnum, Action> setWt = val => 
    () => wt = val;

对于可能常见的情况，您只需检查属性是否具有特定值，您也需要一些便利：

Func<string, string, Predicate<XmlNode>> checkAttr = (attr, val) => 
    node => node.Attributes[attr] == val;

现在您的字典可以包含以下项目：

 ...
 {checkAttr("a", "abc"), setWt(MyEnum.Haze)},
 ...

这很好又简洁，但也不限于简单<attribute, value> to <enum> 映射。好的，现在你有了这些条件-动作对的大字典，你只需说：

foreach(DictionaryEntry<Predicate<XmlNode>, Action> mapping in mappings)
{
     if (mapping.Key(xmlNode))
     {
         mapping.Value();
         break;
      }
}

如果您避免使用 lambda 语法和字典初始值设定项，您应该能够在 2.0 中做到这一点。