Laravel为事件集合批量生成对应路由 仅返回5条问题求解

核心问题是return语句的位置放置错误，导致遍历路由模板的foreach循环仅执行第一次迭代就中断。

• 你当前的代码将return $eventMenuItems;写在了foreach ($eventRoutes as $route)循环内部。
• 循环第一次执行时，会处理第一条路由模板，完成字符串替换并push到集合后，立刻触发return，直接跳出了each方法的回调函数，剩余6条路由模板根本不会被处理。
• 按照这个逻辑，5个Event实例每个都只生成1条路由，最终总数就是5条，和你实际运行的结果完全吻合。

// 存在错误的代码片段
$events->each(function ($event) use ($eventRoutes, &$eventMenuItems) {
    foreach ($eventRoutes as $route) {
        $newRoutes = str_replace('{tournamentKey}', $event->tournamentkey, $route);
        $eventMenuItems->push($newRoutes);
        // 错误：return写在循环内部，第一次迭代执行完就直接退出整个回调
        return $eventMenuItems;
    }
});

将return语句移到foreach循环外部，保证当前Event对应的7条路由全部完成替换、push到集合后，再退出回调：

$eventRoutes = [
    'events/{tournamentKey}/home',
    'events/{tournamentKey}/tournament-info',
    'events/{tournamentKey}/teams',
    'events/{tournamentKey}/schedule-and-results',
    'events/{tournamentKey}/standings',
    'events/{tournamentKey}/stats/general',
    'events/{tournamentKey}/editions'
];
$eventMenuItems = collect();
$events = Event::whereFederationId($federationId)->get();

$events->each(function ($event) use ($eventRoutes, &$eventMenuItems) {
    foreach ($eventRoutes as $route) {
        $newRoutes = str_replace('{tournamentKey}', $event->tournamentkey, $route);
        $eventMenuItems->push($newRoutes);
    }
    // 修正：return移到循环外部，等待当前event的所有路由处理完成后再退出
    return $eventMenuItems;
});

不需要手动维护空集合、嵌套写遍历逻辑，用Laravel集合的flatMap方法可以实现更简洁的实现：

$eventMenuItems = Event::whereFederationId($federationId)
    ->get()
    ->flatMap(function ($event) use ($eventRoutes) {
        // 对每个Event，批量替换所有路由模板，最终自动扁平化为一维集合
        return array_map(
            fn($route) => str_replace('{tournamentKey}', $event->tournamentkey, $route),
            $eventRoutes
        );
    });

这种写法执行后会直接返回包含35条路由的集合，逻辑更直观，也避免了手动控制循环位置容易出错的问题。

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