if(true)是个什么习惯？

[retanoj]

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

// 关闭某个接口

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

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

[gigantic222]

不加逻辑不舒服斯基

[ericls]

以前需要判断 现在不需要了

[blankme]

注释都告诉你了。。。

[Rice]

多半是懒得删除 if 语句结构

[veelog]

我喜欢用 do{}while(0)

[facetest]

软删除

[littleylv]

不是懒。是为了防止以后又要改回去

[syncher]

难道是被产品逼的？

[play78]

业务写多了， 你如果看到这种代码，也见惯不怪了。

boolean flag = check(model); //检查模型部分参数及功能合法性

flag = true; //由于 XXX 需求修改

if(flag == true){
//do something
}else{
//do other
}

[congeec]

我也喜欢这样，不过吃过不少亏
记得旁边加个注释，// FIXME

[DrJoseph]

这叫防御性开发，防止需求的多次更改

[littleylv]

@play78 #8
你这样改问题很大。。。。。。不应该继续执行 check 方法了。

boolean flag = true; //check(model); //检查模型部分参数及功能合法性 // 20170811 由于 XXX 需求修改

我一般会这样改。保持原来的代码在，但注释掉，不要再执行。你懂的万一需求脑抽又要改回去，不能删掉 check 的代码

[LeoNG]

好像在哪见过，/t/375091

[ThatIsFine]

开关要么是配置,要么常量. 直接这么写也就 DEBUG 的时候.发版的时候还有这种就不太专业.

[Mutoo]

我喜欢用注释

//* this block is for test
console.log('this line would be output');
//*/

不用的时候删除掉第一个 /

/*
console.log('this line would not');
//*/

另外还有两个代码块切换的用法

//*
console.log('run this');
/*/
console.log('or this');
//*/

[SourceMan]

防御性开发，他是一个老司机，你要跟他学习

[miniwade514]

很早以前看过的一篇文章
http://www.bitnative.com/2012/10/22/kill-the-zombies-in-your-code/
一直记得他的核心观点：不用就删，别不舍得
“说不准哪天会用到”的东西，十有八九是再也用不到了，或者很久很久才会用到。而这些注释代码对于后续开发是一种干扰，尤其是交给其他人时。
真要保留 A/B 两种方案时，可以尝试分别封装到模块或者函数里，代码干净很多。

[miniwade514]

如果是开发过程中临时 mock，我一般会加上注释
// TODO 调试用，上线前删掉

[raiz]

调试到不用调试了。

[VincentWang]

那些说需求来回改的，你们代码不用版本控制吗？ 老代码在版本里，不要的代码就删了啊...