if-less 编程（基本上没有条件）[关闭]

Question

我有一位同事告诉我，他曾经在一家公司工作，该公司的政策是在代码中永远不要有条件（“if”和“switch”语句），并且他们让代码中的所有决定使用多态性和（我猜）其他一些 OO 原则来完成。

我有点理解这背后的原因，即拥有更干燥且更容易更新的代码，但我正在寻找对这个概念的更深入的解释。或者，它可能是更通用的设计方法的一部分。

如果有人对此有任何资源，或者愿意解释，甚至有更多与此相关的术语，我可以用它来找到更多答案，我将非常感激。

我发现 one question on SO 有点相关，但我不熟悉 C++，所以我不太了解那里的答案。

（顺便说一句，我不是 OO 大师，但我可以做到）

我最精通 PHP，然后是 Python，所以我更喜欢使用这些语言的信息。

更新：我会向我的同事询问更多信息，了解他的确切含义。

2015 年更新：经过多年的编程经验，我现在看到，这项政策的目的可能是防止程序员通过在某些地方添加条件（if 语句）以随意的方式添加功能。扩展软件的更好方法是使用"Open/Closed principle"，其中通过使用继承和多态来扩展软件。我强烈怀疑该政策是否对所有条件都非常严格，因为没有它们很难完全做到。

Answer 1

Anti-IF Campaign网站上有一些资源，比如this article。

我相信这是一个程度的问题。条件并不总是不好，但它们可能（并且经常）被滥用。

其他想法（一天后）

Refactoring: Improving the Design of Existing Code 是关于这个主题（以及许多其他主题）的一个很好的参考。它涵盖Replace Conditional with Polymorphism。网站上还有一个新的 Replace Conditional with Visitor。

我看重简单性和single responsibility，而不是删除所有if 语句。这三个目标通常是一致的。支持cyclomatic complexity 指标的静态分析工具可以快速指出带有嵌套或串行条件的代码。 if 语句可能会保留在重构后，但可以分解为更小的方法和/或多个类。

更新：Michael Feathers 在Unconditional Programming 上写了一篇文章。

这是一个热门话题：Phil Haack on Death to the IF statement！

Answer 2

我有个同事告诉我他曾经在一家公司工作 作为一项政策，永远不要有条件（“if”和“switch” 语句）在代码中，他们让所有的决定在 使用多态性和（我猜）其他一些 OO 来完成代码 原则。

我认为您的同事误解了某些内容或使用了错误的词语来解释它。 而且你不能完全避免条件语句。

有句话要说：OOP 中 if 语句的激增可能是糟糕编程的征兆。一些例子：

不要像旧的 C 风格编程那样使用 if 来检查函数的返回值：

int ret = some_func();
if (ret != null)
   //do something

这在 C 代码中很典型，但对于 OOP，您应该使用异常：

try{
    do_something();
}catch(Exception e){
    e.printStackTrace(); //why I was not able to do something
    handle(e); //there is something else I could do to handle the occurred error
}

有时 if 语句的扩散与糟糕的设计有关。考虑下面的 Java 示例：

BaseClass base;
if (base instanceof DerivedClassOneFromBase){
    DerivedClassOneFromBase d = (DerivedClassOneFromBase)base;
    d.methodOne();
}else if (base instanceof DerivedClassOneFromBase){
    DerivedClassTwoFromBase d = (DerivedClassTwoFromBase)base;
    d.methodTwo();
}

这是另一个糟糕的 if 语句示例，可能与糟糕的设计有关。如果这两个派生对象在它们的基类 BaseClass 中定义了一个公共方法，那么您可以调用该方法而不是检查它们的具体类型并进行强制转换：

base.commonMethod();

Answer 3

我阅读了您链接的帖子，似乎他们主要是在谈论消除类中对条件的需求，不要与所有代码混淆。这个想法是，如果您需要检查一个对象的状态（使用条件）以确定它是否具有某些功能，那么实际上您有两个对象（一个支持该功能，一个不支持该功能）并且应该将它们定义为两个相关的类。

Answer 4

有时方法中的条件是不好的，因为它们表明您只是在一个方法中执行多个功能或多种类型的方法。

如果您有一个名为汽车的类和子类（例如 Car 和 Bike），以及一个方法，例如：

drive(Automobile a)
   if (a.isCar)
      // do stuff
   else if (a.isBike)
      // do stuff

你很可能做错了什么。即使它不是基于类型的开关，它也经常是错误的。如果该方法根据某个变量执行多个功能，则它通常会尝试做不止一件事，并且可能应该分成多个方法。

例如：

save(Car c)
   if (c is new)
      // do some stuff
   else if (c is old)
      // do some stuff

可能会分为保存和更新，因为它们是两个不同的功能。虽然它确实取决于。

完全禁止 if 语句会很愚蠢，因为它们有很多有效的用例。

Answer 5

经过几年的编程，我回到了我自己的问题，我现在对它的上下文有了更好的理解。

Sandi Metz 进行了一场精彩的演讲，她将一个非常多毛的 if 语句重构为一些毛茸茸的东西：https://www.youtube.com/watch?v=8bZh5LMaSmE

Answer 6

避免条件并不一定意味着您需要通过多态性或继承来完成它，例如：

您有 3 个不同的文件夹来存储上传的图片、上传的视频和上传的 pdf

你可以这样写代码：

uploadMedia(mediaType){
   if(mediaType == images){
     uploadTo("myProject/images");
   }else if(mediaType == videos){
     upoloadTo("myProject/videos);  
   }else if(mediaType == pdf){
     uploadTo("myProject/pdf");
  }
}

人们可能使用的另一种选择是 switch-case：

uploadMedia(mediaType){
         switch(mediaType){
         case : images
         uploadTo("myProject/images");
         break;

         case : videos
         uploadTo("myProject/videos");
         break;

         case : pdf
         uploadTo("myProject/pdf");
         break;
    }
}

但是你可以通过使用字典/哈希图/json之类的东西来完全避免条件语句（取决于你使用的任何东西）：

例如：

HashMap<String,String> mediaMap = new HashMap<>();

mediaMap.put("images","myProject/images");
mediaMap.put("videos","myProject/videos");
mediaMap.put("pdf","myProject/pdf");

//mediaType can be images/videos/pdf same as keys of mediaMap
uploadMedia(mediaType){
  uploadTo(mediaMap.get(mediaType));
}

这是一种伪代码，因此可能存在语法错误，但总体而言，这个概念也有助于避免条件。也可以减少代码行数。