Laravel删除品牌时无法同步删图且触发unlink报错如何解决

• 路径拼接缺分隔符：现有代码拼接文件路径时，upload/brands 和数据库存储的logo文件名之间没加 /，拼出来的路径会把目录名和文件名直接粘成一团，根本匹配不到磁盘上的实际文件。
• 文件判断传参错误：现在的is_file($logo) 传的是数据库存的纯文件名（比如202104241126Dell_Logo.png），但is_file()必须拿到文件的完整绝对路径才能正确识别，判断结果完全失效。现在触发的unlink(纯文件名): No such file or directory 报错，就是因为判断逻辑走偏，最终把纯文件名直接传给了unlink函数，自然找不到文件。
• 缺少空值保护：代码没校验品牌记录是否存在、logo字段是否为空，遇到异常数据时容易触发空属性访问的报错。

public function deleteBrand($request){
    // 先查询目标品牌记录，不存在直接返回，避免空值报错
    $brand = $this->brandyModel::where('id', $request->id)->first();
    if (!$brand) {
        return redirect()->back()->with([
            'message' => '待删除的品牌不存在',
            'alert-type' => 'error'
        ]);
    }

    // 空logo、默认占位图跳过删除逻辑，避免误删通用资源
    if (!empty($brand->logo) && $brand->logo !== 'default_logo.png') {
        // 拼接完整绝对路径，补全目录与文件名之间的斜杠
        $logoFullPath = public_path('upload/brands/' . $brand->logo);
        // 只有磁盘上真实存在该文件时才执行删除
        if (is_file($logoFullPath)) {
            unlink($logoFullPath);
        }
    }

    // 执行数据库记录删除
    $brand->delete();
    return redirect()->back()->with([
        'message' => '品牌删除成功',
        'alert-type' => 'success',
    ]);
}

• 统一把logo的完整路径存在一个变量里，不要在存在性判断、删除操作的时候分别拼路径，避免两次拼接逻辑不一致出问题
• 如果数据库里存的logo字段本身已经带了upload/brands/的前缀，就不要重复拼接目录，直接在public_path后面接数据库存的相对路径即可
• 做文件删除操作前，一定要校验路径是否指向预期的存储目录，避免权限过大时误删非目标文件

内容的提问来源于stack exchange，提问作者Я Диша