Laravel Model、これを行うきちんとした正しい方法

Question

私はブロックされたユーザーと呼ばれるこのテーブルを持っています、それはそのようにレイアウトされています。

id, user_id, blocked_user_id, reason, created_at, updated_at 

ここで私は2人が会話できるメッセージコントローラを持っています。私はそのようにしたメッセージを送信する前に、ユーザーがブロックされているかどうかを確認したい。

$blkchk = $thread->participants()->withTrashed()->get(); 
    foreach ($blkchk as $usr) { 
     $usrs[] = $usr->user_id; 
    } 
    $block = BlockedUsers::where('user_id',$usrs[0])->where('blocked_user_id', $usrs[1])->first(); 
    $block2 = BlockedUsers::where('user_id',$usrs[1])->where('blocked_user_id', $usrs[0])->first(); 
    if (!empty($block) || !empty($block2)) { 
     return response()->view('errors.403', ['error' => 'One of the users in this conversation is blocked.'], 404); 
    } 

これはかなり面倒ですが、blkchkは両方のユーザーIDを会話に追加しています。それらを配列として設定し、それぞれを個別にターゲットにします。完璧に動作しますが、私はこれがverry messyであると信じています。

これを行う方法については、正確で最善の方法がどのようになるのだろうと思っています。

Answer 1

あなたは同様にwhereIn()メソッドを使用することができ、特定のエントリがblocked_usersテーブルに存在するかどうかをチェックしたいを考える：

個人的に

$blockedUsers = BlockedUsers::whereRaw(
        'user_id = ? AND blocked_user_id = ?', [$usrs[0], $usrs[1]] 
       ) 
       ->orWhereRaw(
        'user_id = ? AND blocked_user_id = ?', [$usrs[1], $usrs[0]] 
       ); 

if ($blockedUsers->count()) { 
    return 'Some of the users are blocked'; 
} 

を、私はあなたのデータベース設計はそれについて移動する最良の方法だとは思いませんしかし、これはあなたが必要とするもののために働くでしょう。