如何重构PHP访客性别统计代码以提升执行效率？

你的现有代码会循环每个活动ID，每次循环里分别发起3次查询/集合筛选来统计不同性别的访客数量，当活动数量较多时，重复的查询/筛选会导致性能明显下降——本质是做了3*N次重复操作，完全可以通过一次性分组统计来大幅优化。

优化方案（针对Eloquent查询构建器） #

如果$visitors是Eloquent查询构建器实例（还没执行数据库查询），直接用数据库的分组统计能力，只需要1次数据库请求：

// 一次性查询所有目标活动的性别分组统计
$stats = $visitors->whereIn('event_id', $events_id)
    ->selectRaw('event_id, sex, count(*) as total')
    ->groupBy('event_id', 'sex')
    ->get()
    ->keyBy(fn($item) => "{$item->event_id}_{$item->sex}");

// 初始化结果数组，遍历活动ID填充统计值
$men = [];
$women = [];
$kids = [];

foreach ($events_id as $id) {
    $men[] = $stats->get("{$id}_male")?->total ?? 0;
    $women[] = $stats->get("{$id}_female")?->total ?? 0;
    $kids[] = $stats->get("{$id}_kids")?->total ?? 0;
}

优化方案（针对内存中的集合） #

如果$visitors已经是内存中的集合（比如已经执行过get()），用集合的分组方法减少遍历次数：

// 先过滤目标活动的访客，再按「活动ID_性别」分组统计数量
$groupedStats = $visitors->whereIn('event_id', $events_id)
    ->groupBy(fn($item) => "{$item->event_id}_{$item->sex}")
    ->map(fn($items) => count($items));

// 填充结果数组
$men = [];
$women = [];
$kids = [];

foreach ($events_id as $id) {
    $men[] = $groupedStats->get("{$id}_male") ?? 0;
    $women[] = $groupedStats->get("{$id}_female") ?? 0;
    $kids[] = $groupedStats->get("{$id}_kids") ?? 0;
}

优化逻辑说明 #

不管是数据库层面还是集合层面，核心都是把3*N次的重复查询/筛选，变成1次分组统计，再通过键值映射快速获取每个活动的对应性别数量，性能提升幅度会随着活动数量的增加而愈发明显。

内容的提问来源于stack exchange，提问作者Abdoullah al-hajj