问个PHP代码规范的问题-PHP中文网问答

讲师中心微信公众号

首页

文章

后端开发 web前端数据库开发工具 php框架常见问题人工智能 Java 系统教程电脑教程硬件教程手机教程软件教程游戏教程自媒体新闻

专题

后端开发 web前端数据库开发工具 php框架人工智能 Java 系统教程电脑教程硬件教程手机教程软件教程游戏教程新闻

AI工具

AI聊天问答 Agent智能体 AI文本写作 AI绘画作图 AI设计工具 AI视频创作 AI音频制作 AI办公学习 AI编程开发 AI提示词

学习

大前端后端开发数据库移动端运维开发计算机基础

编程手册

大前端后端开发数据库移动端运维开发计算机基础

下载

js特效网站源码工具下载类库下载网站素材学习资源插件扩展手机游戏

最近更新

问个PHP代码规范的问题

PHPz 2017-04-11 10:40:35

[PHP讨论组]

635

首先，有个函数是这么写的。

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判断中执行函数，会使得代码的可读性降低，不知是否存在这样的问题。

PHPz

学习是最好的投资！

全部回复(2)

ringa_lee2017-04-11 10:42:35 2楼

1.函数exam()中,return 的位置是否合适。

有点疑惑，exam函数返回值就是false 和 true，为什么不直接return exam()
然后你这样的写法可以简化成 if(!eaxm()){return false}

2.假如遇到函数中返回类型不确定，执行完成返回结果，执行失败返回false这种情况下，调用的时候是否适合对结果进行类似 if(false === exam()) 的判断。

适合

3.感觉在if判断中执行函数，会使得代码的可读性降低，不知是否存在这样的问题。

不会

赞 +0

添加回复

大家讲道理2017-04-11 10:42:35 1楼

你的try..catch()...不写return false,
意思是插入失败也会返回true

try{
    $bModel->insert($newRecord);
}
catch(\Exception $e){
    \Log::catch($e);
    return false;  # 没有这个, 插入错误并不回返回false
}

赞 +0

添加回复

专题推荐

更多>

热门话题

热门教程

更多>