Laravel如何获取foreach循环外的$response 解决未定义变量报错

问题根因 #

报错是因为$response变量仅在foreach循环内部的if条件分支里被定义，一旦满足以下任意一种场景，该变量在循环外就未定义，触发报错：

• $notificationData->where('status', 'true')返回的集合为空，根本没有进入foreach循环
• 循环中所有Notifications::where('userEnabledNotificationsId', $value['id'])->exists() == false的判断都不成立，从未执行到$response = Notifications::create($data)这行代码

同时你用到的$tags数组也存在同样的作用域风险，建议一起提前初始化。

调整方案 #

你可以在循环外拿到$response的数据，只需要先初始化变量规避作用域问题即可，另外建议将循环外的判断改为校验$tags数组是否非空，更符合批量插入通知的业务逻辑（$response仅会存储最后一次插入的通知对象，判断所有插入结果的集合$tags更准确）。

调整后完整代码如下：

$notificationData = UserEnabledNotifications::all();
// 提前初始化变量，避免未定义报错
$tags = [];
$response = null;

foreach ($notificationData->where('status', 'true') as $user => $value) {
    // 原exists() == false可以直接用Laravel自带的doesntExist()方法，代码更简洁
    if (Notifications::where('userEnabledNotificationsId', $value['id'])->doesntExist()) {
        $notificationTypeName = NotificationTypes::where('id', $value['notificationTypesId'])
            ->value('notificationTypeName');

        // 查询单条用户数据用first()代替get()，避免结果为空时取下标0触发报错
        $userData = User::where('id', $value['userId'])
            ->first()
            ->toArray();

        $data = [
            'notificationTypesId' => $value['notificationTypesId'],
            'notificationTypeName' => $notificationTypeName,
            'userId' => $value['userId'],
            'email' => $userData['email'],
            'recipientName' => $userData['FullName'],
            'userEnabledNotificationsId' => $value['id'],
        ];

        $response = Notifications::create($data);
        $tags[] = $response;
    }
}

// 改为判断插入结果集合是否非空，逻辑更严谨
if (!empty($tags)) {
    return response()->json([
        'message' => 'success',
        'data' => $tags,
        'statusCode' => 200,
        'status' => 'success'
    ], 200);
}

// 补充无新通知插入时的默认返回，避免接口无响应
return response()->json([
    'message' => 'no new notifications created',
    'data' => [],
    'statusCode' => 200,
    'status' => 'success'
], 200);

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