else 和 return，你喜歡哪種？

[fliar]

今天被要求把這段改成else
if(file_exists($fn))
{
//printf("it is a file\n");
$re = $serv->sendfile($fd, $fn);
//printf("Send %s\n", $re ? "Success" : "Failed");
return;
}
$serv->send($fd, $result);
改成
if(file_exists($fn))
{
//printf("it is a file\n");
$re = $serv->sendfile($fd, $fn);
//printf("Send %s\n", $re ? "Success" : "Failed");
}
else
{
$serv->send($fd, $result);
}
請幫忙分析下哪種好，為什麼：）

[xmbaozi]

@angelface 赞同

[lidashuang]

没有return的语言只能用if else了

[cxshun]

以前喜欢if else，现在喜欢return，没用的代码早点结束，看着舒服多了，而且嵌套层次也可以少点。

[ellipse42]

raise

[Septembers]

@Phariel
单出口 大多数情况下会提高代码复杂度
尽可能降低代码复杂度更为重要

完成逻辑尽快退出
执行生命周期过长
不利于GC优化

[Septembers]

修正 单出口 我是指function内只有一个return

[blue7wings]

我还是比较倾向于第一个的,但是如果有多个if,我更喜欢用goto语句:
if(file_exists($fn)){
//printf("it is a file\n");
$re = $serv->sendfile($fd, $fn);
//printf("Send %s\n", $re ? "Success" : "Failed");
goto end;
}
if (condition) {
//something error
goto end;
}
$serv->send($fd, $result);

end:
return false;
有点try-catch的意思了...

[iyangyuan]

虽然单出口是原则，但我还是觉得有时候用return 可以使代码很简洁易读，读者不必读完整个代码段。尤其是在做参数合法性的时候。