首先,有个函数是这么写的。
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...
已抢65164个
抢已抢28740个
抢已抢15654个
抢已抢7896个
抢已抢9881个
抢已抢20032个
抢