Question

我有一个应用程序，用户可以获得游戏积分。我的代码中有一个错误，它意味着在过去一周（GameTypeRank表中）按Game_Type对每个用户点进行小计和排名。如果用户具有过去一周的积分，则此工作正常，但是，如果用户过去一周没有积分，则无法正常工作。如果用户没有积分，他们应该在GameTypeRank中更新0分，否则他们会保留他们的最后排名，直到更新（并且排名不正确）。

# user.rb:
def self.update_game_type_weekly_rank
  @game_types = GameType.all
  @game_types.each do |game_type|

  # this query is where the bug is since it is possible a User has no Points  
  @user_with_points = Point.where("game_type_id = ? and created_at >= ?", game_type.id, 1.week.ago).sum(:points, :group => :user_id, :order => 'sum(points) desc')
    rank = point_counter = 0

    @user_with_points.each do |user_id, points|
      @game_type_rank = GameTypeRank.find_or_create_by_user_id_and_game_type_id(user_id, game_type.id)
      if points != point_counter
        point_counter = points
        rank += 1
      end
      @game_type_rank.weekly_rank = rank
      @game_type_rank.weekly_points = points
      @game_type_rank.save
    end
  end
end

# Models

# game_type_rank.rb
# fields - user_id, game_type_id, weekly_points, weekly_rank
belongs_to :game_type
belongs_to :user

# point.rb
# fields - user_id, points, game_type_id
belongs_to :game
belongs_to :game_type
belongs_to :user

我可以创建一个每周运行的方法，并为每周没有任何用户的用户创建Point记录（使用Point.points = 0），但这是一个糟糕的解决方案。

我也可以在方法开头为每个GameTypeRank记录初始化weekly_points和weekly_rank，但这对我来说似乎效率低下（而且我不确定最好的方法）这一点）。

# Sample Data
Points
|user_id|points|game_type_id|created_at
|1      |   10 |           1|2013-05-07
|1      |   10 |           2|2013-05-07
|2      |   20 |           2|2012-12-31 
|1      |   5  |           2|2012-12-31 

before weekly_update - Game_Type_Ranks 
|user_id |game_type_id | weekly_points | weekly_rank|
|1       |2            |5              |2
|2       |2            |20             |1

after weekly_update - Game_Type_Ranks 
|user_id |game_type_id | weekly_points | weekly_rank|
|1       |1            |10             |1
|1       |2            |10             |1
|2       |2            |20             |1

what should happen after weekly_update - Game_Type_Ranks
|user_id |game_type_id | weekly_points | weekly_rank|
|1       |1            |10             |1
|1       |2            |10             |1
|2       |2            |0              |2   <== update for user_id 2

Answer 1

主要问题是，您无法访问具有现有排名但在@user_with_points中针对给定GameType的当前一周未处于活动状态的用户。

为什么不保留原样，但使用updated_at时间戳。在拨打GameTypeRank对象时，只需提取上周更新的内容即可。

# some controller
GameTypeRank.where("id = ? and updated_at >= ?", id, 1.week.ago)

然而，如果这样你就失去了将旧的倾斜用户weekly_points设置为0的能力。

另一种方法是循环遍历所有具有给定GameType的点的用户，无论时间如何，如下所示：

# user.rb:
def self.update_game_type_weekly_rank
  @game_types = GameType.all
  @game_types.each do |game_type|

    @user_with_points = Point.where("game_type_id = ?", game_type.id).sum(:points, :group => :user_id, :order => 'sum(points) desc')
      rank = point_counter = 0
      @user_with_points.each do |user_id, points|
        @game_type_rank = GameTypeRank.find_or_create_by_user_id_and_game_type_id(user_id, game_type.id)
        if points != point_counter
          point_counter = points
          rank += 1
        end
        @game_type_rank.weekly_rank = rank
        @game_type_rank.weekly_points = points
        @game_type_rank.save
      end
    end

  end
end

在此之后，您的桌子应该是您想要的方式。现在这可能是您可能意识到的代价。也许你可以从不活动中得到某种切断。比如说6个月。

# users.rb
....
@user_with_points = Point.where("game_type_id = ? and created_at >= ?", game_type.id, 6.months.ago).sum(:points, :group => :user_id, :order => 'sum(points) desc')
....

# some controller
GameTypeRank.where("id = ? and created_at >= ?", id, 6.months.ago)

这两种方法都存在一些效率低下的问题，但它可以完成工作。另外，您可能正在使用cron作业和/或后台作业运行此方法，因此对用户的影响应为零。

Rails：与Rails查询相关的逻辑错误

1 个答案: