Laravel中each()循环内修改外部变量不生效原因及解决方案求助

• PHP 匿名函数通过 use 引入外部变量时，默认采用传值拷贝模式，匿名函数内部对 $titleStatus 的修改仅作用于函数内部的局部副本，不会同步到外部的原始变量，因此遍历结束后外部变量始终保持初始的 false。
• 你在匹配到重复标题时 return false 的逻辑本身可以正确终止 each 遍历，不存在逻辑问题，仅变量传递规则导致赋值不生效。
  补充：你的代码还存在两处可优化的细节：一处是取用户ID时如果传入的用户ID不存在，first()->id 会直接抛出错误；另一处是匿名函数中 $value->value 大概率是笔误，应该对应书籍表的标题字段（比如title）。

方案1：修改变量引入方式为引用传递 #

仅需要在 use 引入 $titleStatus 时加引用符号 &，即可让匿名函数内部直接修改外部原始变量：

public function checkTitle(Request $request){
        $newTitle = $request->input('title');
        $newTitle = $this->cleanTitle($newTitle);
        $id = $request->input('id_user');
        $titleStatus = false;

        // 补充用户存在性校验，避免报错
        $user = DB::table('user')->where('id', $id)->first();
        if (!$user) {
            dd(false);
        }
        $book = DB::table('book')->where('id_user', $user->id)->get();

        collect($book)->each(function ($values) use ($newTitle, &$titleStatus){
                $cleanedExistTitle = $this->cleanTitle($values->title); // 将此处的字段改为你实际的标题字段名
                if($cleanedExistTitle == $newTitle){
                    $titleStatus = true;
                    return false;
                }
            });
        dd($titleStatus);
}

方案2：直接通过查询判断（更推荐，性能更优） #

不需要查询用户的所有书籍再遍历，直接在数据层完成判断，减少内存消耗和IO开销：

public function checkTitle(Request $request){
        $newTitle = $request->input('title');
        $newTitle = $this->cleanTitle($newTitle);
        $id = $request->input('id_user');

        $user = DB::table('user')->where('id', $id)->first();
        if (!$user) {
            dd(false);
        }
        // 直接用Collection的contains方法判断是否存在匹配项
        $titleStatus = DB::table('book')
            ->where('id_user', $user->id)
            ->get()
            ->contains(function ($book) use ($newTitle) {
                return $this->cleanTitle($book->title) === $newTitle;
            });
        
        dd($titleStatus);
}

如果你的cleanTitle逻辑可以转换为SQL写法，还可以进一步优化为直接走数据库查询，性能更高：

// 示例对应cleanTitle为“转小写+去空格”的逻辑，可根据你的实际clean规则调整
$titleStatus = DB::table('book')
    ->where('id_user', $user->id)
    ->whereRaw("REPLACE(LOWER(title), ' ', '') = ?", [strtolower(str_replace(' ', '', $newTitle))])
    ->exists();

内容的提问来源于stack exchange，提问作者Dio Agus Nofrizal