리팩토링 ActiveRecord 쿼리

Question

무작위로 선택된 Matchup 오브젝트를 가져 오는 ActiveRecord 쿼리가 있는데,이 플레이어는 현재 대전 상대가 "Bye Week"인 팀에 속하지 않습니다.

모델의 정의 :

매치업은 플레이어 오브젝트의 ID 두 개의 필드, player_1 및 player_2을 가지고 있습니다.

각 플레이어 belongs_to 팀. 여러 개의 필드가 있습니다 (예 : opponent_week_1, opponent_week_2 ... opponent_week_17.

current_week는 오늘 날짜를 기준으로 1에서 17 사이의 정수를 반환합니다 (현재 NFL 시즌의 주를 결정 함).

는 지금은 이런 식으로 일을하고있다,하지만 난 조건 충족, 무작위로, 루프를 제거하고 개체를 선택하려면 그냥 쿼리를 사용하고 싶습니다 :

def getRandomMatchup 
    current_week = view_context.current_week 

    matchup = Matchup.order("RANDOM()").first 

    opponent1 = Team.find(Player.find(m.player_1).team.send("opp_week_#{current_week}")).name  
    opponent2 = Team.find(Player.find(m.player_2).team.send("opp_week_#{current_week}")).name 

    while opponent1 == "Bye Week" || opponent2 == "Bye Week" 
    matchup = Matchup.order("RANDOM()").first 

    opponent1 = Team.find(Player.find(matchup.player_1).team.send("opp_week_#{current_week}")).name 
    opponent2 = Team.find(Player.find(matchup.player_2).team.send("opp_week_#{current_week}")).name 
    end 

    return matchup 
end 

내 생각되었다 같은 것을 할,하지만 난 그것을 정확하게 실행하는 방법을 잘 모르겠어요하기 :

m = Matchup.where.not(Team.find(Player.find(m.player_1).team.send("opp_week_#{current_week}")).name: "Bye Week", 
    Team.find(Player.find(m.player_2).team.send("opp_week_#{current_week}")).name: "Bye Week").order("RANDOM()").first 

을 내가 루비 2 레일을 사용하고 4

Answer 1

그래서 당신은 두 개의 랜덤 재생 사이 매치업을 생성해야 이번 주 상대 팀이 '안녕히 주'가 아니니?

방법에 대한

Player.joins(:teams).where.not('team.opponent_week_#{current_week}' => 'Bye Week').limit(2) 

이제 난수의 돌봐 잎 두 선수가 같은 팀에 속해 있다는 가능성을 열지 않습니다.

는

편집 :

Matchup.joins(:players).where('players.id = ? OR players.id = ?', player_1, player_2).joins(:teams).where.not('team.opponent_week_#{current_week}' => 'Bye Week')