Laravel Artisan命令封装eaDir过滤方法失效问题排查

核心问题是PHP数组默认采用值传递机制：

• 你封装的eaDir()方法接收的$infile参数，是传入的类属性数组的独立拷贝，方法内部所有过滤、遍历修改操作，都只会作用在这个副本上，完全不会影响外部$src、$clients、$slider三个类属性的原始值。
• 当你把foreach过滤逻辑直接写在handle()方法内时，操作的是类属性本身，不存在传参产生的副本隔离问题，过滤结果会直接保留在原数组上，因此可以正常生效。
• 未被过滤的@eaDir路径进入图片处理流程后，程序尝试向Synology系统保留的@eaDir目录写入缩略图，该目录默认不开放普通进程写入权限，就会触发Intervention\Image\Exception\NotWritableException报错。

推荐两种无副作用、逻辑清晰的封装方式，可根据自己的编码习惯选择：

方案1：返回过滤后的新数组（推荐，可读性最好） #

把过滤方法设计为纯函数，接收原数组、返回过滤完成的新数组，调用时手动赋值回原属性，不存在隐式修改逻辑，后续维护成本最低：

// 方法名建议改为见名知意的filterEaDir，不要用无意义的eaDir命名
protected function filterEaDir(array $fileList): array
{
    // 替换为你实际的@eaDir过滤规则，拼接目录分隔符避免误判文件名带@eaDir的正常文件
    return array_filter($fileList, function (string $path): bool {
        return !str_contains($path, DIRECTORY_SEPARATOR . '@eaDir' . DIRECTORY_SEPARATOR);
    });
}

public function handle()
{
    // 1. 归集所有目标图片路径到类属性的逻辑
    // ...
    
    // 2. 执行过滤
    $this->src = $this->filterEaDir($this->src);
    $this->clients = $this->filterEaDir($this->clients);
    $this->slider = $this->filterEaDir($this->slider);
    
    // 3. 执行缩略图生成逻辑
    // ...
}

如果想进一步减少重复代码，可以把需要过滤的属性统一遍历处理：

foreach (['src', 'clients', 'slider'] as $property) {
    $this->$property = $this->filterEaDir($this->$property);
}

方案2：使用引用传递直接修改原数组 #

在方法定义时给参数加&标记，声明参数按引用传递，方法内部的修改会直接作用在原数组上，不需要手动赋值返回：

// 注意参数前的&符号
protected function filterEaDir(array &$fileList): void
{
    $fileList = array_filter($fileList, function (string $path): bool {
        return !str_contains($path, DIRECTORY_SEPARATOR . '@eaDir' . DIRECTORY_SEPARATOR);
    });
}

public function handle()
{
    // 1. 归集路径逻辑
    // ...
    
    // 2. 过滤，不需要接收返回值
    $this->filterEaDir($this->src);
    $this->filterEaDir($this->clients);
    $this->filterEaDir($this->slider);
    
    // 3. 生成缩略图逻辑
    // ...
}

注意：引用传递会产生隐式修改，排查问题时不容易定位变量修改点，非必要场景优先选择方案1。

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