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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

For Existing Member Sign In

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

This topic created in 1449 days ago, the information mentioned may be changed or developed.

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

Supplement 1 · May 11, 2022

声明下是 java ，var 的用法只是因为用的是 jdk11 ，支持这样写了，但我不爱用

151 replies • 2022-05-12 10:08:07 +08:00

1 2

❮

❯

101

nonwill

May 11, 2022

这还好吧，引荐去看几个高手大佬们的 fork 出来的改进项目，加简直是 shi++++山

https://github.com/goldendict/goldendict/issues/1340#issuecomment-1123296947

102

gogogo1203

May 11, 2022

1. 有些语言 var xx boolean 默认就是 false, 再指明是 false 就是废话
2. 除非能 if else 就结算，不然一律使用 switch
3. 统一处理 input toLowerCase, 然后再 switch, 然后直接处理各个逻辑。

槽点实在是太多了

103

Leviathann

May 11, 2022

@wangtian2020 这个代码逻辑其实是反的，相当于把最后一个 else 写到了最上面，如果是经常考虑完备性的人读这个代码会感觉很不适

104

dufu1991

May 11, 2022

@Aliberter 可能这就是他能当领导的原因。

105

oppoic

May 11, 2022

var a = list.Contains("a") ? true : false;

过了很久，IDE 智能了许多，提示我这块可以优化，我才发现。

106

cslive

May 11, 2022

看到 var 和变量名以为是 javascript ，后面再看 equalsIgnoreCase 才知道是 java ，差点没认出来

107

ClorisYe

May 11, 2022

这么短的代码看不出别的，就知道很容易看清楚它想表达什么意思。我认为容易理解的就是好代码。

108

bthulu

May 11, 2022

@zdt3476 bool 跟 true 比较是有用的, 有可能你的这个对象里没有这个 bool 属性, 如果直接 if(a.b)就会出错, 而 if(a.b == true)却能整厂运行

109

pkwenda

May 11, 2022

我还以为是 js 重新了 String 呢

String.prototype.equalsIgnoreCase = function(str){ return true }

110

akira

May 11, 2022

挺好的啊，一目了然不用做任何猜测。

能运行，并且清晰的代码就是好代码。。。

111

pkwenda

May 11, 2022

@nonwill #101 我没看明白你喷的点是什么，你也没喷代码写的烂啊，项目进度慢？ 😂

112

nonwill

May 11, 2022

@pkwenda “喷”过了，不管用
https://github.com/goldendict/goldendict/pull/1447
https://www.v2ex.com/t/842896

估计几个大佬就是为了练习英语吧

113

goobai

May 11, 2022 via Android

代码跟人，一个能跑就行！再怎么好看的代码，也就那点工资

114

jones2000

May 11, 2022

代码水平主要看是否满洲当前业务需求，如果不满足，在漂亮的代码都是白给。其次就是可以让其他人可以看懂代码逻辑。看懂这个标准就不好说了，依据其他人的教育程度，文化水平，代码功底，业务了解程度等等.....

满洲当前业务需求这很重要，如果需求 10 并发量，你写了 1W 个并发量，在老板看来都一样，业务能上线了。题外话，如果你真写了 1W 并发的，那你们老板 2 期项目怎么升级收费呢？

115

encro

May 11, 2022

还不错，
用到了防御性编程和变量命名表意，
不亏是领导，
应该看过代码大全之类的。

差的是对语言不熟悉。

116

Kamiyu0087

May 11, 2022

我觉得能一眼就看明白的代码也不算太差吧😂

117

UIXX

May 11, 2022

对于已经语义化了的变量，我有一种评判代码逻辑组织是否合理的方式，即

用自然语言去试读代码是否会让人感到冗余。

从这个角度来看，我是同意 LZ 的说法，比如 if isOpened == true (假如已打开为真）就不如 if isOpened （假如已打开）。

118

SG4YK

May 11, 2022 via iPhone

😅

119

justin2018

May 11, 2022

变量命名看着好累一下英文一下拼音~~

代码写得挺好的~

逻辑一下就看清楚了知道是干嘛的

120

timothyye

May 11, 2022

如果工资是以代码行数来计算的话，这是高人一等的水平

121

v2eb

May 11, 2022 via Android

就是感觉啰嗦，哈哈哈

122

l00t

May 11, 2022

随意了点，啰嗦了点，其他问题不大。至少结构是清晰的。就这么几行也看不出别的，不知道需求的情况下有些没法说好不好。

123

gogogo1203

May 11, 2022

大家聊了那么多都不知道为什么

operateName =toLowerCase(operateName)

switch(operateName ) {
case "bangding":
....
break;
case "shuaxin":
...
break;
default:
...
}

124

gogogo1203

May 11, 2022

我搞不明白了,为什么还会有人说原代码容易读.

125

oldmanong

May 11, 2022 via iPhone

代码写得绕，简单逻辑写复杂，但能保证性能，也故意埋点儿坑。于是乎一有问题都找他，上司看着工作饱和，老板觉得不可替代。你看他 2 ，他看你也 2

126

documentzhangx66

May 11, 2022

1.定义变量，并初始化赋值，是有意义的。你们没写过 C/CPP ，没调试过复杂程序，自然不知道为啥要这么干。
var isOperate_BangDing = false;
var isOperate_ShuaXin = false;

2. if else 这里，我不觉得有啥问题，你要改成别的结构也可以，但我觉得 if else 更能体现思路。
if ("bangDing".equalsIgnoreCase(operateName))
{
isOperate_BangDing = true;
}
else if ("shuaXin".equalsIgnoreCase(operateName))
{
isOperate_ShuaXin = true;
}

3.这里就不得不说了，不能这样子做，一条 if 语句，应该尽量少的包含操作，这样才利于 debug 。
if (isOperate_BangDing == true
|| isOperate_ShuaXin == true) {...

我建议这里加一条临时变量，并且为了可读性，把括号也加上：
var tempResult = ( (isOperate_BangDing == true) || (isOperate_ShuaXin == true) );
if( tempResult ) ....