CakePHP结合Security Hash实现单图上传的代码问题及优化咨询

• 文件名生成逻辑错误：仅基于原始文件名做SHA1哈希，只要两个文件的原始名称相同，无论内容是否一致都会生成相同的存储文件名，会直接覆盖已有文件，且多页面共用相同文件名的图片时，本质上共用同一份磁盘文件，删除/更新其中一个页面就会删除公共文件，导致其他页面图片失效。
• 编辑逻辑存在严重漏洞：未判断用户是否上传了新图片，只要提交编辑请求就会尝试读取image_file参数，若用户未上传新图片，调用getClientFilename()会直接抛出致命错误；且未判断新文件是否保存成功、页面数据是否入库成功，就提前删除了旧图片，一旦后续操作失败会导致旧图片永久丢失。
• 缺少必要异常处理：文件移动、删除操作都没有做错误捕获和失败处理，出现磁盘权限不足、文件不存在等问题时会直接抛出PHP原生错误，没有友好的错误提示和回滚逻辑。
• 重复上传无校验：相同文件重复上传时会重复写入磁盘，浪费存储空间，也容易引发文件覆盖异常。

你可以根据业务需求选择两种优化方向：

方案1：无需支持多页面共用图片（实现最简单） #

每个页面的图片完全独立，生成存储文件名时加入唯一标识符，确保就算是完全相同的文件，不同页面上传也会生成不同的文件名，不会出现共用文件的情况，删除时互不影响。

方案2：需要支持多页面共用图片（节省存储空间） #

新增一张图片资源表page_images，字段包含id、file_hash（文件内容的哈希值）、filename（存储文件名）、reference_count（引用计数），上传文件时先校验文件内容哈希是否已存在：

• 已存在则直接将对应记录的引用计数+1，无需重复存文件
• 不存在则写入磁盘并新增记录，引用计数初始化为1
  编辑/删除页面时，先将旧图片的引用计数-1，只有引用计数降到0时才真正删除磁盘文件和表记录。

public function add()
{
    $page = $this->Pages->newEmptyEntity();

    if ($this->request->is('post')) {
        $page = $this->Pages->patchEntity($page, $this->request->getData());
        $image = $this->request->getData('image_file');

        // 仅当有上传文件且实体无错误时处理图片
        if (!$page->getErrors() && $image && $image->getError() === UPLOAD_ERR_OK) {
            // 文件名加入唯一标识，避免重复
            $nameSeed = $image->getClientFilename() . uniqid() . time();
            $filename = \Cake\Utility\Security::hash($nameSeed, 'sha1') . '.' . pathinfo($image->getClientFilename(), PATHINFO_EXTENSION);
            $targetPath = WWW_ROOT . 'img' . DS . 'pages' . DS . $filename;
            
            try {
                $image->moveTo($targetPath);
                $page->image = $filename;
            } catch (\Exception $e) {
                $page->setError('image_file', '图片上传失败，请检查目录权限');
            }
        }

        if ($this->Pages->save($page)) {
            $this->Flash->success(__('Your page has been created.'));
            return $this->redirect(['action' => 'index']);
        }
        $this->Flash->error(__('Unable to add your page.'));
    }
    $this->set('page', $page);
}

/**
 * Edit method
 *
 * @param string|null $id Page id.
 * @return \Cake\Http\Response|null|void Redirects on successful edit, renders view otherwise.
 * @throws \Cake\Datasource\Exception\RecordNotFoundException When record not found.
 */
public function edit($id)
{
    $page = $this->Pages->get($id);
    $oldImage = $page->image;

    if ($this->request->is(['post', 'put'])) {
        $page = $this->Pages->patchEntity($page, $this->request->getData());
        $image = $this->request->getData('image_file');
        $newFilename = null;

        // 仅当有新上传文件且实体无错误时处理图片
        if (!$page->getErrors() && $image && $image->getError() === UPLOAD_ERR_OK) {
            $nameSeed = $image->getClientFilename() . uniqid() . time();
            $filename = \Cake\Utility\Security::hash($nameSeed, 'sha1') . '.' . pathinfo($image->getClientFilename(), PATHINFO_EXTENSION);
            $targetPath = WWW_ROOT . 'img' . DS . 'pages' . DS . $filename;
            
            try {
                $image->moveTo($targetPath);
                $newFilename = $filename;
                $page->image = $filename;
            } catch (\Exception $e) {
                $page->setError('image_file', '图片上传失败，请检查目录权限');
            }
        }

        if ($this->Pages->save($page)) {
            // 保存成功后再删除旧图片
            if ($newFilename && $oldImage && file_exists(WWW_ROOT . 'img' . DS . 'pages' . DS . $oldImage)) {
                @unlink(WWW_ROOT . 'img' . DS . 'pages' . DS . $oldImage);
            }
            $this->Flash->success(__('Your page has been updated.'));
            return $this->redirect(['action' => 'index']);
        }
        // 保存失败的话删除刚刚上传的新图片
        if ($newFilename && file_exists(WWW_ROOT . 'img' . DS . 'pages' . DS . $newFilename)) {
            @unlink(WWW_ROOT . 'img' . DS . 'pages' . DS . $newFilename);
        }
        $this->Flash->error(__('Unable to update your page.'));
    }
    $this->set('page', $page);
}

/**
 * Delete method
 *
 * @param string|null $id Page id.
 * @return \Cake\Http\Response|null|void Redirects to index.
 * @throws \Cake\Datasource\Exception\RecordNotFoundException When record not found.
 */
public function delete($id)
{
    $page = $this->Pages->get($id);
    $image = $page->image;

    if ($this->Pages->delete($page)) {
        // 删除数据成功后再删图片
        if ($image && file_exists(WWW_ROOT . 'img' . DS . 'pages' . DS . $image)) {
            @unlink(WWW_ROOT . 'img' . DS . 'pages' . DS . $image);
        }
        $this->Flash->success(__('The {0} page has been deleted.', $page->title));
        return $this->redirect(['action' => 'index']);
    }
}

上述代码额外补充了文件后缀保留、上传错误判断、失败回滚逻辑，避免出现数据丢失问题。

内容的提问来源于stack exchange，提问作者user11791297