CTO：再写if-else，逮着罚款1000！

怼烎@ 2022-12-30 13:52 167阅读 0赞

“

本文并不肯定或者否定哪一种写法，仅仅为大家提供一些其他的编码思路或者一些值得借鉴的点子。

![format_png][]

*图片来自 Pexels*

![format_png 1][]

*设计更好的软件，替换 If-Else 的 5 种方法，从入门到高级示例*

If-Else 通常是一个糟糕的选择，它导致设计复杂，代码可读性差，并且可能导致重构困难。

但是，If-Else 已成为事实上的代码分支解决方案，这确实是有道理的。这是向所有有抱负的开发人员讲授的第一件事。

不幸的是，许多开发人员从来没有前进到更合适的分支策略。有些人的口头禅是：If-Else 是一把锤子，一切都是钉子。

我将向大家展示一些技巧和模式，这些技巧和模式将终结这种可怕的做法。每个示例的难度都会增加。

完全不必要的 Else 块

这也许是那些初级开发人员最负罪的之一。下面的示例很好地说明了当你被认为 If-Else 很棒时会发生什么：

![format_png 2][]

*Simple if-else*

只需删除 else\` 块即可简化此过程，如下图：

![format_png 3][]

*Removed else*

看起来更专业吧？你会发现，实际上根本不需要其他块。像在这种情况下一样，你想要在满足特定条件的情况下执行某些操作并立即返回。

价值分配

如果你要根据提供的某些输入为变量分配新值，请停止 If-Else 废话，一种更具可读性的方法。

![format_png 4][]

*Value assignment with if-else*

尽管很简单，但它却很糟糕。首先，If-Else 很容易在这里被开关取代。但是，我们可以通过完全删除 else 来进一步简化此代码。

![format_png 5][]

*If statements with fast return*

如果不使用 else，则我们将剩下干净的可读代码。请注意，我也将样式更改为快速返回而不是单返回语句。如果已经找到正确的值，继续测试一个值根本没有意义。

前提条件检查

通常，我发现，如果方法提供了无效的值，则继续执行是没有意义的。假设我们从以前就有了 DefineGender 方法，要求提供的输入值必须始终为 0 或 1。

![format_png 6][]

*Method without value checks*

在没有价值验证的情况下执行该方法没有任何意义。因此，在允许方法继续执行之前，我们需要检查一些先决条件。

应用保护子句防御性编码技术，你将检查方法的输入值，然后继续执行方法。

![format_png 7][]

*Check preconditions with guard clauses*

至此，我们确保仅在值落在预期范围内时才执行主逻辑。现在，IF 也已被三元代替，因为不再需要在结尾处默认返回"未知"。

将 If-Else 转换为字典，完全避免 If-Else

假设您需要执行一些操作，这些操作将根据某些条件进行选择，我们知道以后必须添加更多操作。

![format_png 8][]

也许有人倾向于使用久经考验的 If-Else。如果添加新操作，则只需简单地添加其他内容即可。很简单 但是，就维护而言，这种方法不是一个好的设计。

知道我们以后需要添加新的操作后，我们可以将 If-Else 重构为字典。

![format_png 9][]

可读性已大大提高，并且可以更轻松地推断出该代码。注意，仅出于说明目的将字典放置在方法内部。您可能希望从其他地方提供它。

扩展应用程序，完全避免使用 If-Else

这是一个稍微高级的示例。通过用对象替换它们，知道何时甚至完全消除 If。

通常，您会发现自己不得不扩展应用程序的某些部分。作为初级开发人员，您可能会倾向于通过添加额外的 If-Else（即 else-if）语句来做到这一点。

举这个说明性的例子。在这里，我们需要将 Order 实例显示为字符串。首先，我们只有两种字符串表示形式：JSON 和纯文本。

在此阶段使用 If-Else 并不是什么大问题，如果我们可以轻松替换其他，只要如前所述即可。

![format_png 10][]

知道我们需要扩展应用程序的这一部分，这种方法绝对是不可接受的。

上面的代码不仅违反了"打开/关闭"原则，而且阅读得不好，还会引起可维护性方面的麻烦。

正确的方法是遵循 SOLID 原则的方法，我们通过实施动态类型发现过程（在本例中为策略模式）来做到这一点。

重构这个混乱的过程的过程如下：

*  **使用公共接口将每个分支提取到单独的策略类中。**
 *  **动态查找实现通用接口的所有类。**
 *  **根据输入决定执行哪种策略。**

替换上面示例的代码如下所示。是的，这是更多代码的方式。它要求您了解类型发现的工作原理。但是动态扩展应用程序是一个高级主题。

我只显示将替换 If-Else 示例的确切部分。如果要查看所有涉及的对象，请查看此要点。

![format_png 11][]

让我们快速浏览一下代码。方法签名保持不变，因为调用者不需要了解我们的重构。

首先，获取实现通用接口 IOrderOutputStrategy 的程序集中的所有类型。然后，我们建立一个字典，格式化程序的 displayName 的名称为 key，类型为 value。

然后从字典中选择格式化程序类型，然后尝试实例化策略对象。最后，调用策略对象的 ConvertOrderToString。

END

--------------------

推荐阅读：

*  [漫话：如何给女朋友解释什么是Mock？][Mock]  
    
 *  [万字详解本地缓存之王 Caffeine][Caffeine]  
    
 *  [女朋友做了个梦......][......]  
    
 *  [数据库读写分离这个坑，让刚入职的我一脸懵逼！][Link 1]

**![format_png 12][]喜欢我可以给我设为星标哦![format_png 12][]**

![format_png 13][]

![format_png 14][]

**好文章，我****“在看”**

![format_png 15][]

[format_png]: /images/20221120/2421dbee64944de98a62467ebdb7fbed.png
[format_png 1]: /images/20221120/15855d6cead946b3bba1d188032c35e8.png
[format_png 2]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVrRDFlRGdnRkM5bDM3bk1NQUdCNDE4U1ZWS0FaT1FReDhpY1VkdlVacmJuWURVVWp2Q09jSmljUS82NDA?x-oss-process=image/format,png
[format_png 3]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVkQVpIUllJOVlLa2FsYVdzaGZCSUdJV2NLTnZpYmVLSHNHeE1CMlNuekNwSGliWmlhV2VxUmljdDF3LzY0MA?x-oss-process=image/format,png
[format_png 4]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVjZWFVazdBeVMzWVNCN0NGWjNFdm9zQU5EVjVoYlJWUk44MXU1ZW14OG5icUlWY1RLUW5aTlEvNjQw?x-oss-process=image/format,png
[format_png 5]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVPOWF2bzREaWFDS0FpYlNwTUhpYUQxb0FPRE1qM2JrNjdYTHNXRllCSG5maWNMNW9EOE01aWI5ZDR0dy82NDA?x-oss-process=image/format,png
[format_png 6]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVJSlVMSXduZXBscjVrQUF0VTQ3WkxMOXlnM0hoU3YzSWNrcnp0aWJ5QnFNaWJUUEdrU1Y2eHNkQS82NDA?x-oss-process=image/format,png
[format_png 7]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVCQVJMdGhTWGt1WWNrcDF4UHVpYjlSNm8zdGNFbkdzOU5LdHBHUzgxSzJLRUdOeVRRaWJXRWhwQS82NDA?x-oss-process=image/format,png
[format_png 8]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGUxUnpjZnBtZ0psNmp3M21YN24yaWE1b3k0Y1owSHIwWmY2WWlhSDcycFBleUZjb3k2dW1hUTZtQS82NDA?x-oss-process=image/format,png
[format_png 9]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVIZ1o2UFd3R3ZWUWRuVkZjOWE0Q0MwOFFXMnVnRDRYNEkzN2NuQTJiem43R1dlZ211ZklvZmcvNjQw?x-oss-process=image/format,png
[format_png 10]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVjTHRON0tiTkhJaWJJUWx2UUdDMGtpYk9Ta01JQXlza2w1SklONmliMGZDVk1VcDJlcGdNbG44ZncvNjQw?x-oss-process=image/format,png
[format_png 11]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9vVEtIYzZGOHRzaWF1UGxpY1dmWWxGb1ZpYm9pY09rZzc0NGVVQmpIbUF4bFhKNDE4aWFkRFNMZkRnR0JwWVNzU1BpYlFmN2xjV0o5dDMwQ1g0aWFnSXpKNXRCQmcvNjQw?x-oss-process=image/format,png
[Mock]: http://mp.weixin.qq.com/s?__biz=Mzg3MjA4MTExMw%3D%3D&chksm=cef60ac0f98183d6a6319635a77e05e8c8b966eea443ba179f56eb184ce3654d579d1b57d5cc&idx=1&mid=2247498102&scene=21&sn=e08075239280edec95137f4fb40ff3d6#wechat_redirect
[Caffeine]: http://mp.weixin.qq.com/s?__biz=Mzg3MjA4MTExMw%3D%3D&chksm=cef60a22f981833462807661c6ae937bfbf9949965e577103e1a78fcf669ce22162254746fc5&idx=2&mid=2247498132&scene=21&sn=6982e20048f37a379efd11ead0e0502f#wechat_redirect
[......]: http://mp.weixin.qq.com/s?__biz=Mzg3MjA4MTExMw%3D%3D&chksm=cef60af7f98183e189e5db3916e4f865a5fe5eecb5696c06eaf50bdde0ea83cdd6a366a560e0&idx=1&mid=2247498049&scene=21&sn=5a168aa5f61b442508f99d500e1b958a#wechat_redirect
[Link 1]: http://mp.weixin.qq.com/s?__biz=Mzg3MjA4MTExMw%3D%3D&chksm=cef60a61f98183776d88418677d3e933fc65d26f7e92cb7cad132561edfc05bd5cfcdb1ed4b9&idx=2&mid=2247498199&scene=21&sn=3083e59355aacdb1683f038e40764913#wechat_redirect
[format_png 12]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X3BuZy9DMXVETURxam4xaWJIYkNVMnVTdHpnaWNraWNQVng4ZFR2WHNKeDBiU0JpY01mZnlWMGwyQTlaaEZMSjZ5TmY0c3lkRnBmQ3c4aFZwc3Y5QTNKTlkweVlpYmZBLzY0MA?x-oss-process=image/format,png
[format_png 13]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9DMXVETURxam4xaWJIYkNVMnVTdHpnaWNraWNQVng4ZFR2WEQzeEJ3OGdqTEQzRzFkd0NQWGZia3BVcllhV0hsekJETUdHcFZ2dWVpYWFBWXRBRUVrY1FpY0h3LzY0MA?x-oss-process=image/format,png
[format_png 14]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9DMXVETURxam4xOExQUWZaa0JUelI0MkxpYXBCaGhpYVV4SEJiclFubGt4elpxWkdmZ2lhcTRHN2dIYWR2UWliaFZxR3BlWU1OY2M5S2tkVkxpYktpYTJYSkhrZy82NDA?x-oss-process=image/format,png
[format_png 15]: https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2dpZi83UVJUdmtLMnFDNnNMbEFQY1BoZURMeWN1allyT3ZzWnZpYVNPRGJSYlF6UXZXOFZMMXlpYUdBOEZUUGJwT2JGdUppYlNkaWIzVDU1c0pPYTNRZlZDeWlhUGNBLzY0MA?x-oss-process=image/format,png