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

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

[Bronya]

@Aliberter #30 哈哈哈，我说我看好一会在想这是啥语言，var 关键字还有 C#换行的风格，小驼峰下划线同时用，结果是 java🤣

[libook]

变量名 is 开头表明是布尔型，这个感觉是加分项；
驼峰+大小写取决于项目整体命名规则；
用英文还是英文拼音混合看团队约定。

else if 后面没有 else 处理异常情况，或者没有双 false 的处理情况，比如 operateName 既不是 bangDing 又不是 shuaXin 是否需要处理。

因为 operateName 只能有一个确定的值，虽然不一定 bangDing 或 shuaXin ，所以给两个变量赋值的部分应该是可以简化的。

下面 if 判断 true 的情况，可以直接判断变量，不需要再写“== true”。

要不要把两个变量直接省略，然后把两个 equalsIgnoreCase 直接放到最下面 if 条件里，取决于下文是否还会用到这两个 equalsIgnoreCase 结果，当然 if 条件太长可读性也未必好。

[laucenmi]

又不是不能用 /狗头

[cheese]

@glaucus #38 这是 java ，js 没有 equalsIgnoreCase 方法

[huntagain2008]

#2 小白反正看了几分钟硬是没看懂代码的意思，但是看了楼主的代码 2 秒就明白了意思。

[Leviathann]

@anonydmer 那也应该是
var isOperate_ShuaXin = isOperate_BangDing ? false : "shuaXin".equalsIgnoreCase(operateName)

[anonydmer]

非要不考虑上下文只考虑代码行数么？

if (Stream.of("shuaXin", "bangDing").anyMatch(operateName::equalsIgnoreCase)) {}

哦，还要加个 operateName 为空的 guard 语句，两行顶天了。

[bestwaytowait]

kpi master

[fredli]

水平很差，看过的代码太少，模仿也不会

[acehowxx]

只看这 1 几行，我觉得还好吧。最后不用==true 判断，再有就是应该定义个常量进行比较而不该用魔法数。至于变量命名，英文不好没办法啦。

[mara1]

@Aliberter , C#不背这个锅，事实上在 VS 里写 C#，各种 resharper 之类的提示，跟着提示改，好多小毛病都能改掉。

[Suddoo]

不清楚需求，别乱动屎山，
改完，测试提一个二次引入的 bug ，那就尴尬了

[Leonard]

为什么英文和拼音混用，驼峰和下划线混用

[binux]

你们都没写过 switch...case 吗？

[Aliberter]

@mara1 啊 对不起 没有黑 c#的意思 上边只是陈述下事情

[Aliberter]

@Suddoo 在屎山上开发也挺难受的

[Aliberter]

@Leonard 哈哈 挺逗的吧 我也没这么写过

[chenmoGnar]

把变量定义提前摆出来不是很好的习惯吗，楼主的改进版也仅仅是把定义和计算放在了一起而已；相反我觉得把变量类型声明成 boolean 看着更舒服

[sarices]

主要是代码风格有问题

[kooze]

可能按代码量考核绩效吧