Laravel框架条件内循环时Element元素重复显示问题排查

问题描述 #

即便已经设置了判断条件，且使用的是多对多关联关系，前端依然出现下载链接重复显示的问题，相关代码如下：

@foreach (\App\ClientDocument::where('id_client', $clients->id)->get() as $item)    
    @if (\App\Document::where('id',$item->id_document)->get())
    <a class="img-fluid " href="{{asset('images/'.$item->file)}}" download="{{asset('images/'.$item->file)}}" alt="" name="download"  > Telecharger </a>
    @endif  
@endforeach

问题根因 #

• 现有if判断完全失效：\App\Document::where('id',$item->id_document)->get()返回的是Laravel集合对象，即使没有匹配的文档，空集合的布尔校验结果也为true，无法过滤无效关联
• 直接遍历中间表ClientDocument，如果中间表存在重复的id_client+id_document关联记录，就会重复渲染相同的下载链接
• 未使用Laravel原生多对多关联特性，手写查询逻辑冗余且容易出错

解决方案 #

快速修复现有代码 #

将if判断中的get()替换为exists()，仅当文档真实存在时才渲染链接，同时对中间表查询结果按id_document去重，避免重复输出：

@foreach (\App\ClientDocument::where('id_client', $clients->id)->get()->unique('id_document') as $item)    
    @if (\App\Document::where('id',$item->id_document)->exists())
    <a class="img-fluid " href="{{asset('images/'.$item->file)}}" download="{{asset('images/'.$item->file)}}" alt="" name="download"  > Telecharger </a>
    @endif  
@endforeach

推荐规范写法（基于多对多关联） #

首先在Client模型中定义与Document的多对多关联：

// App/Models/Client.php （Laravel 5及以下版本路径为 App/Client.php）
public function documents()
{
    return $this->belongsToMany(Document::class, 'client_documents', 'id_client', 'id_document');
}

如果下载用到的file字段存储在中间表client_documents中，关联定义需要补充withPivot指定要读取的中间表字段：

public function documents()
{
    return $this->belongsToMany(Document::class, 'client_documents', 'id_client', 'id_document')
                ->withPivot('file');
}

之后Blade模板中直接遍历关联数据即可，天然避免重复，无需额外判断：

{{-- file字段存储在document表用这个写法 --}}
@foreach ($clients->documents as $document)
    <a class="img-fluid " href="{{asset('images/'.$document->file)}}" download="{{asset('images/'.$document->file)}}" alt="" name="download"  > Telecharger </a>
@endforeach

{{-- file字段存储在中间表用这个写法 --}}
@foreach ($clients->documents as $document)
    <a class="img-fluid " href="{{asset('images/'.$document->pivot->file)}}" download="{{asset('images/'.$document->pivot->file)}}" alt="" name="download"  > Telecharger </a>
@endforeach

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