if(true)是个什么习惯？

[retanoj]

经常在代码里看到这种用法

// 关闭某个接口

if(true) { return APIResultVo.error(xxxx); }

这个 if( true ) 是什么编程习惯？有什么说法么？

[Perry]

好奇怎么通过 code review 的 还什么防御性开发 牛逼

[Monad]

@littleylv #12 然而发现 Check 是个有副作用的函数 卒

[nicevar]

@VincentWang 版本控制如果改动多的话时间一长几十个版本去找浪费时间，而且代码逻辑复杂的情况下更麻烦，上面说的，写 fixme+注释比较妥当

[hiboshi]

我们也经常这样 ，有时候需要 临时关闭 某个功能，我的经验是 老代码写成这样 往往都是有原因的。

[shard]

骗编译器？
比如
```
if(true) return xxx;
// other codes
...
```

[Lucups]

@VincentWang +1

很多新手都有这个问题，代码舍不得删，各种大段注释。我说从版本库里找回来不就行了，他们说太麻烦。。。
其实就是懒。

话说，那么多注释的代码看着不累么。。。

[stephen9357]

@littleylv 这样做是过不了代码覆盖率检查的。

[fan123199]

如果不加 if，return 下面如果有代码，有些 ide 会报错

[bk201]

代码少还好，多了看得是比较烦人的。

[loveCoding]

防御性开发最多做个为空判断 ,其他代码都删掉吧 ,基本上用不到.

[ma125125t]

上面说 code review、版本控制的，纯粹是你们的需求改的不够多罢了。
当然，我承认这样写不好。

[jason2017]

还什么防御性开发，代码不规范就是不规范，懒就是懒。
所谓的为了防止需求变动，你就那 2 行代码改起来都觉得费事吗？
而且对于经常变动的业务，也不应该写这种硬编码的代码，更应该考虑把相关逻辑转移到配置文件或者数据库中，这才是"防御性开发"。

[mcfog]

@miniwade514 为什么要立这样的 flag ……

if (ENV_IS_NOT_PRODUCTION) {
//do sth. dirty
}

[am241]

@Mutoo 碰巧我也是…

[pynix]

当注释用吧，，，

[esmdxx1]

楼主你用你得先夺为主得偏见，首先就确定事情得结构，事实上得情况 ，复杂很多，所以原因也太多，比如，自己定义得条件编译，或者本来需要两段代码，

[gdtv]

我承认我这样写不规范，但需求更不规范，如果需求能保证始终如一后续不改变，我保证能写出规范的代码。

[flowci]

不要滥用概念，，，，防御性编程是一种编程习惯，是指预见在什么地方可能会出现问题，然后创建一个环境来测试错误，当预见的问题出现的时候通知你，并执行一个你指定的损害控制动作，如停止程序执行，将用户重指向到一个备份的服务器，或者开启一个你可以用来诊断问题的调试信息。。。。

[dorentus]

@fan123199 不过即使加了，比较新的编译器也还是会报错或者警告……

[iyaozhen]

说实话没有见到过这样的代码，写过的、review 过的代码也有十几万行了。这样些是会被直接打回的