公正评价，这代码什么水平

[Aliberter]

var isOperate_BangDing = false;
var isOperate_ShuaXin = false;
if ("bangDing".equalsIgnoreCase(operateName))
{
isOperate_BangDing = true;
}
else if ("shuaXin".equalsIgnoreCase(operateName))
{
isOperate_ShuaXin = true;
}

if (isOperate_BangDing == true
|| isOperate_ShuaXin == true) {...

[Aliberter]

@bugFactory 一是可读性，二是下边的方法还要用这个变量

[cht]

linter 都过不了…

[Aliberter]

@lyy16384 是完全等价的，因为不可能存在 operateName 既等于 bangDing 又等于 shuaXin

[Aliberter]

@cmdOptionKana sonar 的代码规范里不允许这么写，这么写会报警

[blackboom]

精梳！这段代码越看越废话，一般人根本写不出来。

[lakehylia]

== true 是什么鬼

[Aliberter]

@JKeita 对 且不说这些吧 姑且可以容忍

[Aliberter]

@spicecch 我的领导写的 可怕不

[Aliberter]

@happinessnch 因为这是他写的代码，他现在让我去改，我看着难受，想吐槽

[Aliberter]

@cpstar
@liangkang1436
@Cu635
这是我领导写的，所以不存在为了涨绩效啥的，他之前根本不会 java ，属于是现学现卖的，之前写 c#的

[Aliberter]

@V2LIYANG 我就是后来的维护者，我已近吐了

[Aliberter]

@tbxark 不是 是他一气呵成的

[Aliberter]

@mlhadoop 哈哈 老哥 精辟

[cmdOptionKana]

@Aliberter 领导写的，就很好理解了，几乎可以肯定他就是像写伪代码一样，既不考虑语言特性，也不综合考虑上下文，就是想到一句写一句，这样可以写得很舒服，大脑低电量模式半梦游状态就可以写了，舒舒服服写完找属下改成风格良好的代码就行，反正他自己舒服。

[anonydmer]

有没有可能他是觉得比楼主 2 楼的方法在运行时会可能少一次计算量？

[pengtdyd]

垃圾，建议进厂打螺丝！

[glaucus]

一看就不是 IDEA 写的

[glaucus]

哦才看见是前端，当我没说

[AllenHua]

告诉他下划线 underline 和小驼峰 low camel 两种风格最好不要混用，还有最好能用英文单词就不要用中文全拼……

[lyy16384]

@Aliberter #23 对 我没看清比的是同一个变量，既然领导让你改，你直接重构就行了