你的核心问题是没有基于原图片路径数组的索引去替换对应位置的值，而是直接追加新路径。我们可以通过先获取原路径数组，再根据上传文件的索引替换对应元素，最后重新拼接成字符串的方式解决这个问题。

步骤1：重构控制器的update方法 #

原控制器代码存在重复查询数据库、直接追加路径的问题，我们调整逻辑如下：

public function update(Request $request, $id) {
    // 1. 获取目标记录（找不到则抛出404）
    $popup = Popup::findOrFail($id);
    // 2. 把原逗号分隔的路径拆分为数组
    $paths = explode(",", $popup->image_path);
    
    // 3. 遍历每个上传的文件，按索引匹配替换
    foreach ($request->filep as $index => $file) {
        // 仅当有上传文件且原数组存在对应索引时执行替换
        if (!empty($file) && isset($paths[$index])) {
            // 生成唯一文件名
            $image_name = time() . $file->getClientOriginalName();
            $destinationPath = public_path('/popups');
            // 移动新文件到目标目录
            $file->move($destinationPath, $image_name);
            
            // 替换数组中对应索引的旧路径
            $paths[$index] = $image_name;
            
            // 可选：删除旧图片文件（清理存储空间，按需启用）
            $oldImagePath = public_path('/popups/' . $paths[$index]);
            if (file_exists($oldImagePath)) {
                unlink($oldImagePath);
            }
        }
    }
    
    // 4. 将更新后的数组重新拼接为逗号分隔的字符串
    $updatedImagePath = implode(",", $paths);
    // 5. 一次性更新数据库，避免多次查询
    $popup->update(['image_path' => $updatedImagePath]);
    
    // 返回成功响应（可根据需求调整）
    return redirect()->back()->with('success', '图片更新完成');
}

步骤2：确认视图代码的正确性 #

你的视图代码已经正确给每个文件输入设置了对应索引的name属性（name="filep[{{ $y }}]"），这能保证控制器通过索引精准匹配原数组的位置。如果用户不想替换某张图片，只需不上传新文件，控制器会自动保留原路径，无需额外修改。

关键改动说明 #

• 一次性获取原记录并拆分路径数组，避免重复查询数据库，提升效率
• 按上传文件的索引直接替换原数组对应位置的元素，实现精准更新
• 最后统一拼接数组为字符串，一次性提交数据库更新
• 可选添加旧图片删除逻辑，避免无用文件占用存储空间

这样修改后，新上传的图片就会替换对应位置的原有路径，而不是追加到字段末尾了。

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