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

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

[chenmoGnar]

@TWorldIsNButThis 是不是简单的逻辑要做到声明即初始化；因为我看 jdk 工具包里面都在方法的开始把后面用到的变量都声明了先，后面才初始化。平时我们自己写业务代码也不会单独起一行声明一个变量，也是跟初始化一起。

[zdt3476]

别的先不说，我真的不理解把 bool 值和 true/false 比较的人到底在想啥

[leexy]

又不是不能运行😏

[danieladu]

switch(operateName.ToLower()){
case "binding":
case "refresh":
dosomething();
break;
...
}

[Ben4]

你来我办公室一趟

[lujiaosama]

看着难受, 整那么多 ifelse 做什么, 三目运算符, includes 用上三行以内搞定. 用变量名注解含义还是要的, 但是这个变量名中英文混用,驼峰下划线混用, 真的挖槽. 这种代码读起来像是实习生的面条代码, 又臭又长, 逻辑稍微多一点就要被淹没在 ifelse 的汪洋里.

[hailiang88]

这就是自己爽就行，不在乎别人的代码，写起来一点心智负担都没有

[DOLLOR]

当 value 为 bool 型的时候，if(value == true)这种写法相比 if(value)有什么好处吗？

[hfl1995]

`when{
"bangDing".equalsIgnoreCase(operateName),
"shuaXin".equalsIgnoreCase(operateName)->{

}

}
`

[hfl1995]

`when{`
`"bangDing".equalsIgnoreCase(operateName),`
`"shuaXin".equalsIgnoreCase(operateName)->{`

​ `}`

`}`

[DreamSaddle]

可能是按行数计费时期留下来的代码。

[UserNameisNull]

可能是美团的？要统计代码行数？

[adoal]

团队有 coding style 就拿出来怼他。没有……说个捷豹。

[lurenw]

这感觉就是一种"流水账"式的写法，先 XXX 然后 XXX ，在分支不多的情况下，这种写法对后来人的维护，我觉得没啥坏处，甚至不要动脑子，比起动不动在一个 if 判断里面加入 N 个 condition 的精简代码，我更喜欢读这种。

只不过最后这个 if (isOperate_BangDing == true|| isOperate_ShuaXin == true) 有点无脑和随意了。然后这个人的命名也很随意。

[wangtian2020]

我会直接改成中文枚举值

if (onperationName == '绑定'||onperationName == '刷新') {

}

还在用 var 是什么屑前端，兼容 ie ？

[Leviathann]

@DOLLOR 如果是 java 的话，Boolean 可能为 null ，boolean 则不会，有时候懒得区分就直接写明 == true 了

[efaun]

这么写会降低运行效率吗, 如果不会, 就是正常水平

[hackfly]

@gps949 赞同，强语言才考虑这些，胶水语言很少看到这些考虑。不过就一段代码看不出什么来。就算是整合别人的，在整体考虑别人思路会花费较多时间

[sutra]

俩拼音，不及格。

[cnrting]

能赚钱的代码就是好代码🐶