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

[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) {...

[Leviathann]

废话文学家

[Aliberter]

明明 3 行就写清楚的事，愣是啰嗦了十几行，给我整无语了
var isOperate_BangDing = "bangDing".equalsIgnoreCase(operateName);
var isOperate_ShuaXin = "shuaXin".equalsIgnoreCase(operateName);
if (isOperate_BangDing || isOperate_ShuaXin)

[Aliberter]

@Leviathann 精辟

[bugFactory]

为什么要设两个变量，直接 if ("bangDing".equalsIgnoreCase(operateName) || "shuaXin".equalsIgnoreCase(operateName)) {
}
不行么？

[lyy16384]

@Aliberter #2 原代码两个变量不会同时为 true ，跟你这个不完全等价，不了解需求哪个是对的不知道

[cmdOptionKana]

英语不太好，但就这点代码体现不出水平高低，他只是选择了非常 verbose 的表达方式而已。好处是看到这句

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

的时候很舒服，比挤在一起看起来舒服多了。

[JKeita]

变量名风格也看得难受

[spicecch]

谁写的，刁他啊

[happinessnch]

这段代码不管多烂，也体现不出什么。
提出这个问题的人要想一下，为什么要纠结这个事情。

[cpstar]

一行变七行，这 KPI ，杠杠的

[liangkang1436]

你实话实说，你们公司，是不是把代码量考虑进绩效里面了

[aneostart173]

@bugFactory 你那样每次都要算一次 equalsIgnoreCase

[Cu635]

看工作量的考评方式。
如果工作量考核是用的代码行数或者代码大小（毕竟一个字符要占用一个字节嘛），那么这个代码就是优秀的。

[cpstar]

正经应该：
let operators = ['bangDing','shuaXin'];
if (operators.includes(operateName)) {
当然了，这里边没办法解决大小写的问题，可以再写一个大小写转换的 func

[BreadKiller]

我只吐槽一下 英文和拼音最好不要混着写

[theqiang]

不看逻辑光看这变量名，驼峰➕下划线➕拼音能把人看傻

[ersic]

没啥问题啊，写的挺清楚的。
单看这几行好像挺啰嗦，但我觉得后面 isOperate_ShuaXin 跟 isOperate_ShuaXin 还会用到的吧。

[V2LIYANG]

对于后来的维护者会非常友好

[tbxark]

可能一开始不是这样写的后来需求改来改去就开始摆烂打补丁了

[mlhadoop]

遇到这种问题问自己
1. 是老板不
2. 能正常跑不
3. 能发帖摸鱼不
4. 关机