Laravel如何排除无关联描述的分类以生成下拉选项组？

看起来你已经离解决问题很近了，不过有两个关键点需要调整：一个是方法名的拼写错误，另一个是更高效的过滤时机选择。让我一步步帮你修正：

问题分析 #

你之前的代码里用了doesnthave，但Laravel的正确方法名是驼峰式的doesntHave。另外，就算修正了拼写，在each闭包里用continue也不对——Laravel集合的each方法中，需要用return来跳过当前元素，而不是continue（那是原生foreach的语法）。

不过更推荐的做法是在数据库查询阶段就过滤掉没有关联描述的分类，这样不仅代码更简洁，还能避免查询不必要的数据，提升性能。

最优解决方案：查询阶段过滤 #

修改你的descriptions方法，在查询时加入has('activeDescriptions')条件，直接排除掉没有关联活跃描述的分类：

public static function descriptions() {
    return static::active()
        ->has('activeDescriptions') // 只保留有至少一个activeDescriptions的分类
        ->with('activeDescriptions')
        ->orderBy('name', 'asc')
        ->get()
        ->mapWithKeys(function ($category) {
            // 先对描述按名称排序，再提取name和id的键值对
            $sortedDescriptions = $category->activeDescriptions
                ->sortBy('name')
                ->pluck('name', 'id')
                ->toArray();
            
            return [$category->name => $sortedDescriptions];
        });
}

代码解释 #

• has('activeDescriptions')：这是Laravel的关联查询条件，会自动过滤出拥有至少一个activeDescriptions关联模型的分类。
• mapWithKeys：比each更简洁的集合方法，直接构建你需要的键值对数组，不需要额外声明$descriptions变量。
• 先sortBy再pluck：确保描述是按名称排序后再生成键值对，避免排序失效（因为pluck返回的是数组，数组的排序逻辑和集合不同）。

如果你坚持在集合层面过滤 #

如果因为某些原因需要在查询后再过滤，修正你的each逻辑：

public static function descriptions() {
    $descriptions = [];
    
    static::active()
        ->with('activeDescriptions')
        ->orderBy('name', 'asc')
        ->get()
        ->each(function ($category) use (&$descriptions) {
            // 修正方法名拼写，并用return跳过当前元素
            if ($category->doesntHave('activeDescriptions')) {
                return;
            }
            
            $descriptions[$category->name] = $category->activeDescriptions
                ->sortBy('name')
                ->pluck('name', 'id')
                ->toArray();
        });
    
    return $descriptions;
}

这里的关键是把doesnthave改成doesntHave，并且用return代替continue来跳过无关联的分类。

为什么之前的方法无效？ #

1. 方法名拼写错误：Laravel的方法是驼峰命名的doesntHave，不是全小写的doesnthave，这会导致调用不存在的方法，条件判断失效。
2. continue不适用：在Laravel集合的each闭包中，continue不会生效，需要用return来跳过当前迭代的元素。

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