

V2 的老哥们倾向哪种写法?


V2 的老哥们倾向哪种写法?
|
JoooooooooDec 4, 2021
1
if else 最朴素的写法呀, 清晰永远都是第一要务.
后面改这块逻辑的人很可能还是你自己. |
|
autoxbcDec 4, 2021
2
() => {
switch(true) { case exp1 : case exp2 : case exp3 : return false ; } return true ; }; 这是我觉得最工整的写法 |
|
BuildingDec 4, 2021 via iPhone
3
guard … else {
return } guard … else { return } …… return true |
|
binuxDec 4, 2021 via Android
4
你为什么要写或,写与条件不是更清晰吗?
|
|
QingXuJiaZhiDec 4, 2021
5
我选第二种,并且会加注释。第二种也更容易加注释。
|
|
noe132Dec 4, 2021
6
|
|
v2gbaDec 4, 2021
7
我个人的话:把第一个的 if 改成真值 > 第二个 > 第一个
|
|
8
@cmdOptionKana 感觉不是更方便注释,而是条件下有新增逻辑需求的时候不用拆条件。
|
|
QingXuJiaZhiDec 4, 2021
9
@wednesdayco 注释也很重要啊,第一种,以及 #6 @noe132 给出的写法,注释要怎么写,很难描述。
如果发现用人类语言很难描述,通常就可能有问题,而第二种很好描述 “如果审批不 OK ,就不用看别的因素,必然是个无效价格”……如此这般,每一个判断都非常清晰。 |
|
10
return !(item.approvalStatus!==
STATUS.OK || item.isEffective Il (moment(orderTime) <= moment(item.endTime) && moment(orderTime) ›= moment(item.startTime))) |
|
zxCoderDec 4, 2021
11
我会选第二种,但是为什么不加花括号!!!!!!!!!!!!!
|
|
ch2Dec 4, 2021
12
第二种可读性更好
|
|
bwangelDec 4, 2021
13
第二种啊,第一种条件多了,容易写成箭头型代码。
五六层缩进,不好看 |
|
cpstarDec 4, 2021
14
def rtn = true;
if rtn=... else if rtn=... else if rtn=... else rtn=... return rtn 哈哈 |
|
InnovatinoDec 4, 2021 via iPhone
15
我觉得第二种更好。
但是我觉得把需求写在方法上面,把注释写在条件上面是更好的实践。 |
|
kaikiDec 4, 2021
16
我经常 return (xxx) ? true : false;
想着反正以后不会再去维护这块了 |
|
newtype0092Dec 4, 2021
17
@noe132 这种为形式而形式了吧,本来一个条件就可以中断了,没必要把所有条件都算完。
非要追求这样的格式的话也应该分别包到闭包里。 |
|
newtype0092Dec 4, 2021
18
第二种是合理的,条件多的时候按类型分开,可以确定 return 的提前 return 出去,这样后面条件越来越简单越来越清晰。
第一种一个条件连一大串,看得眼睛花了都。 if body 要加大括号 要加大括号 要加大括号 |
|
WilliamYangDec 5, 2021
21
可以看下《代码整洁之道》
|
|
binuxDec 5, 2021 via Android
22
return item.approvalStatus==
STATUS.OK && !item.isEffective && ! moment(orderTime).isBetween(item.startTime, item.endTime) |
|
hm20062006okDec 5, 2021 via iPhone
23
写两个叹号是啥意思?
|
|
chairuosenDec 5, 2021
24
写代码给人看是第一要素,不建议 10 楼写法
|
|
25
|
|
noparking188Dec 5, 2021
26
|
|
rosuDec 5, 2021 via iPhone
27
@noparking188 理论上都可以收拢到 item 对象里的,约束和复用程度更好~
|
|
huijieweiDec 5, 2021
28
一句话
尽早返回 |
|
munan56Dec 5, 2021
29
第二种啊
|