Laravel使用unlink删除文件时报“是目录”错误的原因及方案咨询

算力阿峰

2026-11-3

报错根本原因

你碰到的unlink is a directory提示，完全是两个逻辑缺口叠加出来的问题：

从PHP本身的函数特性来说，unlink() 只支持删除普通文件，完全没有删除目录的能力，只要传入的路径指向文件夹，就必然抛出这个warning级错误。
你当前的前置判断有漏洞：只拿file_exists($path)做拦截，而这个函数对文件、目录都会返回true，根本不会区分路径到底是图片还是文件夹——只要拼出来的路径存在（不管是要删的图片，还是某个空目录），就会直接调用unlink()。
路径之所以会指向目录，基本都是历史脏值导致的：比如之前某次代码逻辑出错，把$event->img的值存成了上传目录本身（比如存了events/而不是具体的events/xxx.jpg文件名），或者上传流程异常没正确写入图片路径，最终拼出来的$path就指向了storage下的某个文件夹，直接触发报错。

为什么加@跑通是伪解决方案

很多人觉得加@不弹错就是解决问题了，实际上这属于典型的不良实践：

@符只是无差别压制PHP抛出的所有warning，根本没有解决"路径是目录、unlink删不了"的核心问题——这时候unlink根本没执行成功，旧的图片资源甚至目录都没被删掉，时间长了storage目录下会堆一堆无用文件，白占磁盘空间。
@的压制不做区分，哪怕是权限不足、文件被进程占用这类真正需要关注的错误，也会被一起屏蔽，后续真出现资源没删掉的问题，根本没有日志可以排查。

零错误抑制符的修复方案

完全不需要加@，只要补全校验逻辑，甚至用Laravel自带的文件系统能力，就能从根源上避免报错：

方案1：手动补全判断逻辑，在原有判断基础上加is_file()校验，确认路径指向的是普通文件再执行删除，遇到目录类脏值直接记日志，不硬删也不中断流程：

$file = $request->file('image');
if($file->getSize() < 2048000){
    // 取模型数据库里存的原始旧图片路径，避免提前赋值导致删错新文件
    $oldImgPath = storage_path('app/public/' . $event->getOriginal('img'));
    // 同时满足两个条件才删：路径存在 + 是普通文件
    if (file_exists($oldImgPath) && is_file($oldImgPath)) {
        unlink($oldImgPath);
    } elseif (file_exists($oldImgPath)) {
        // 碰到脏值（路径是目录）时记日志，不抛错
        \Log::warning("删除事件旧图片时检测到目录路径，跳过删除", [
            'event_id' => $event->id,
            'path' => $oldImgPath
        ]);
    }
    $filename = Str::uuid() . "." . $file->getClientOriginalExtension();
    $event->img = $file->storeAs('events', $filename, 'public');
}
$event->save();

方案2：更贴合Laravel开发习惯的简化写法，直接用框架封装的文件系统facade，不用手动拼路径、手动调unlink——Storage::disk('public')->delete() 方法本身就会自动校验路径类型，是普通文件才会删除，遇到目录、不存在的路径会直接跳过，完全不会抛这类warning：
```
// 替换上面的手动删除逻辑，代码更简洁，也不会出现路径拼接错误
if (!empty($event->getOriginal('img'))) {
    \Storage::disk('public')->delete($event->getOriginal('img'));
}
```

小提示：用$event->getOriginal('img')读取模型的原始值，比直接读$event->img更安全，能避免因为提前修改模型属性，导致误删新上传图片的问题。

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