》关于代码中按条件返回布尔值怎样写更好这件事《

[wednesdayco]

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

[Jooooooooo]

if else 最朴素的写法呀, 清晰永远都是第一要务.

后面改这块逻辑的人很可能还是你自己.

[autoxbc]

() => {
　 switch(true)
　{
　　 case exp1 :
　　 case exp2 :
　　 case exp3 :
　　　 return false ;
　}

　 return true ;
};

这是我觉得最工整的写法

[Building]

guard … else {
return
}

guard … else {
return
}

……

return true

[binux]

你为什么要写或，写与条件不是更清晰吗？

[cmdOptionKana]

我选第二种，并且会加注释。第二种也更容易加注释。

[noe132]

[MrGba2z]

我个人的话：把第一个的 if 改成真值 > 第二个 > 第一个

[wednesdayco]

@cmdOptionKana 感觉不是更方便注释，而是条件下有新增逻辑需求的时候不用拆条件。

[cmdOptionKana]

@wednesdayco 注释也很重要啊，第一种，以及 #6 @noe132 给出的写法，注释要怎么写，很难描述。

如果发现用人类语言很难描述，通常就可能有问题，而第二种很好描述 “如果审批不 OK ，就不用看别的因素，必然是个无效价格”……如此这般，每一个判断都非常清晰。

[ZE3kr]

return !(item.approvalStatus!==
STATUS.OK ||
item.isEffective Il
(moment(orderTime)
<= moment(item.endTime) && moment(orderTime) ›= moment(item.startTime)))

[zxCoder]

我会选第二种，但是为什么不加花括号！！！！！！！！！！！！！

[ch2]

第二种可读性更好

[bwangel]

第二种啊，第一种条件多了，容易写成箭头型代码。

五六层缩进，不好看

[cpstar]

def rtn = true;
if rtn=...
else if rtn=...
else if rtn=...
else rtn=...
return rtn

哈哈

[Innovatino]

我觉得第二种更好。
但是我觉得把需求写在方法上面，把注释写在条件上面是更好的实践。

[kaiki]

我经常 return (xxx) ? true : false;
想着反正以后不会再去维护这块了

[newtype0092]

@noe132 这种为形式而形式了吧，本来一个条件就可以中断了，没必要把所有条件都算完。
非要追求这样的格式的话也应该分别包到闭包里。

[newtype0092]

第二种是合理的，条件多的时候按类型分开，可以确定 return 的提前 return 出去，这样后面条件越来越简单越来越清晰。
第一种一个条件连一大串，看得眼睛花了都。


if body 要加大括号 要加大括号 要加大括号

[ZE3kr]

@kaiki 你这直接 return xxx 就行了

[kaiki]

@ZE3kr 差不多，只是我喜欢用:?缩写，能直接返回布尔值也没必要专门搞一道