“如果”与转让是一个很好的做法？

Question

考虑这种情况下：

大型代码：

if($site->getUser($id)){ 
    $user = $site->getUser($id); 
    //more code here 
} 

短代码：

if($user = $site->getUser($id)){ 
    //already have the user 
    //more code here 
} 

在第一种情况下

，我做2次获得BBDD用于获取用户，但在第二只有一个... 第二种情况，是一种很好的做法？

Answer 1

我更喜欢这个。

$user = $site->getUser($id); 
if($user){  
    //more code here 
} 

一般情况下，应避免条件分配。许多编程语言甚至不允许它。太容易犯错误。 它在可读性方面也更好。

Answer 2

重复调用一个潜在的昂贵和/或有副作用的函数在任何情况下都不是一件好事，所以通常应避免使用第一个样本。如果该方法纯粹是一种不执行任何逻辑代码的被动getter方法，则可能出现异常情况;但是由于这个实现可能会改变，所以你应该避免它。

if ($foo = bar()) ... 

针对以上可以说，错别字混淆=和==并不少见，所以这可能会导致错误和/或受到不必要的混乱，未来开发商阅读代码。最少引起争议的方法应该是：

$foo = bar(); 
if ($foo) ... 

但是，有些人更喜欢将其缩短为if-assignment-in;这是一种文体选择。

Answer 3

是的，在我看来。 但是，我习惯于这样的代码。

if($user = ($site->getUser($id))){ 
    //already have the user 
    //more code here 
} 

警告这

if($flag = $usrType == $type)){ 
    //already have the user 
    //more code here 
}