【发布时间】:2011-08-24 01:53:41
【问题描述】:
当使用代码点火器时,我创建了很多看起来像这样的函数;
function info($id){
$r = $this->db->select('id', 'name', 'age')->from('users')->where('id', $id)->get();
return ($r->num_rows() == 0 ? false : $r->result());
}
现在当我使用这个函数时,我也使用这个函数来检查用户是否存在并将其分配给一个变量。
所以它会被用作类似的东西
if( ($user = $this->user->info($_GET['id'])) === false )
die('User not found');
//now we can continue and $user contains the user info
我的问题是,有什么理由这是一个坏主意吗?
据我所知,这与
相同$user = $this->user->info($_GET['id']);
if( $user === false )
die('User not found');
但在我看来实际上更容易理解。
【问题讨论】:
标签: php codeigniter coding-style if-statement