首先，有个函数是这么写的。
function exam(){ //从数据库获取数据。 $records = $amodel->where(['type' => 1])->get(); if(empty($records)) return false; $newrecord = $bmodel->map(); foreach($records as $item){ $newrecord->name = $item->name; $newrecord->age = $item->age; try{ $bmodel->insert($newrecord); } catch(\exception $e){ \log::catch($e); } } return true;}

然后，这个函数是这么调用的。
if(true !== exam()){ echo 表更新失败; return false;}

请各位经验丰富的phper看下，这其中有几个地方看着挺别扭的：
函数exam()中,return 的位置是否合适。
假如遇到函数中返回类型不确定，执行完成返回结果，执行失败返回false这种情况下，调用的时候是否适合对结果进行类似 if(false === exam()) 的判断。
感觉在if判断中执行函数，会使得代码的可读性降低，不知是否存在这样的问题。
回复内容： 首先，有个函数是这么写的。
function exam(){ //从数据库获取数据。 $records = $amodel->where(['type' => 1])->get(); if(empty($records)) return false; $newrecord = $bmodel->map(); foreach($records as $item){ $newrecord->name = $item->name; $newrecord->age = $item->age; try{ $bmodel->insert($newrecord); } catch(\exception $e){ \log::catch($e); } } return true;}

然后，这个函数是这么调用的。
if(true !== exam()){ echo 表更新失败; return false;}

请各位经验丰富的phper看下，这其中有几个地方看着挺别扭的：
函数exam()中,return 的位置是否合适。
假如遇到函数中返回类型不确定，执行完成返回结果，执行失败返回false这种情况下，调用的时候是否适合对结果进行类似 if(false === exam()) 的判断。
感觉在if判断中执行函数，会使得代码的可读性降低，不知是否存在这样的问题。
exam()中的return 为什么不在抛出异常的时候就直接return false?
一般返回类型如果是bool类型不建议加第三种类型，你说的那种方法可以判断但是不建议
可读性的问题不是因为在if里，而是你的函数名有问题，如果要返回是bool型建议加个is来表示， 如：
if(false === isexam()){ echo 表更新失败; return false;}
推荐一本书你可以看看：
https://book.douban.com/subje...