在一堆业务逻辑无限 return false 的代码里查 bug,真的很蛋疼。。。

V2EX = way to explore

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

现在注册

已注册用户请登录

这是一个创建于 2798 天前的主题，其中的信息可能已经有所发展或是发生改变。

在一堆业务逻辑无限 return false 的代码里查 bug,真的很蛋疼。。。

就这样～

return

false

蛋疼

bug

18 条回复 • 2017-03-29 11:12:55 +08:00

WenhaoWu

2017-03-28 20:31:42 +08:00

if(dataNotValid(data)){
notValid();
return false;
}

if(dataNotMatch(data)){
notMatch();
return false;
}

handleData(data);
return true;

vs

if(!dataNotValid(data)){
if (!dataNotMatch(data)){
handleData(data);
return true;
}
else {
notMatch();
return false;
}
}
else{
notValid();
return false;
}

mokeyjay

2017-03-28 20:34:54 +08:00 via Android

直接把 return false 替换为抛出错误然后外部捕捉统一返回不是更好吗？

cdwyd

2017-03-28 20:36:43 +08:00

直接 return false 更容易理解。

bk201

2017-03-28 20:37:05 +08:00 via iPhone

@mokeyjay return false 不代表异常啊

xialdj

2017-03-28 20:55:11 +08:00 via iPhone

@WenhaoWu 我喜欢前一种代码清晰条件分拆清楚

laball

2017-03-28 21:23:06 +08:00

这个时候，就是体现代码重构的功底了！

linbiaye

2017-03-28 21:24:29 +08:00

求看看代码张什么样啊

greatbody

2017-03-28 22:18:06 +08:00

建议写一个函数，将所有需要验证的逻辑作为参数数组传给这个函数。
比如：
验证器([
()=>{验证逻辑 1},
()=>{验证逻辑 2},
()=>{验证逻辑 3}])

验证器函数自己逐个调用这些函数，然后遇到一个 false 就直接 return 。

这一朵朵奇葩一样嵌套的 if 判断也是够了。