在 ImageUploadHandler 类的 save 方法是不是有性能的优化?

<?php

namespace App\Handlers;

class ImageUploadHandler
{
    // 只允许以下后缀名的图片文件上传
    protected $allowed_ext = ["png", "jpg", "gif", 'jpeg'];

    public function save($file, $folder, $file_prefix)
    {
        .
        .
        .
        $extension = strtolower($file->getClientOriginalExtension()) ?: 'png';

        // 拼接文件名,加前缀是为了增加辨析度,前缀可以是相关数据模型的 ID 
        // 值如:1_1493521050_7BVc9v9ujP.png
        $filename = $file_prefix . '_' . time() . '_' . str_random(10) . '.' . $extension;

        // 如果上传的不是图片将终止操作
        if ( ! in_array($extension, $this->allowed_ext)) {
            return false;
        }
        .
        .
        .
    }
}

如果把判定上传的文件是否是合格格式判定放在前面,先进行判定再做一些文件名的生成是不是会更好一些?

《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《L04 微信小程序从零到发布》
从小程序个人账户申请开始,带你一步步进行开发一个微信小程序,直到提交微信控制台上线发布。
最佳答案

同意,我就是这么做的。

  1. 判定文件后缀
  2. 设置路径
  3. 设置文件名
  4. 保存文件
  5. 压缩文件(如有需要)
  6. 返回参数
5年前 评论
讨论数量: 3

如果纯粹为了性能的优化,是否还可以将 \App\Handlers\ImageUploadHandler::save 中的判断文件后缀名的代码省略,因为这个判断在 \App\Http\Requests\UserRequest::rules 中就已经做了。。

'avatar' => 'mimes:jpeg,bmp,png,gif|dimensions:min_width=200,min_height=200',
5年前 评论

同意,我就是这么做的。

  1. 判定文件后缀
  2. 设置路径
  3. 设置文件名
  4. 保存文件
  5. 压缩文件(如有需要)
  6. 返回参数
5年前 评论

同意,我就是这么做的。

  1. 判定文件后缀
  2. 设置路径
  3. 设置文件名
  4. 保存文件
  5. 压缩文件(如有需要)
  6. 返回参数
5年前 评论

附议,一般建议把检查语句和卫语句放到函数的最前面,提前检查就行

5年前 评论

如果纯粹为了性能的优化,是否还可以将 \App\Handlers\ImageUploadHandler::save 中的判断文件后缀名的代码省略,因为这个判断在 \App\Http\Requests\UserRequest::rules 中就已经做了。。

'avatar' => 'mimes:jpeg,bmp,png,gif|dimensions:min_width=200,min_height=200',
5年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!