优化问题

app/models/challenge.rb

@@ -89,34 +89,16 @@ class Challenge < ApplicationRecord
   # end
 
   ## 用户关卡状态 0: 不能开启实训; 1:直接开启; 2表示已完成
-  def user_tpi_status shixun, user_id
-    # todo: 以前没加索引导致相同关卡,同一用户有多个games
-    # 允许跳关则直接开启
-    game = games.where(user_id: user_id).take
-    if game.blank?
-      position == 1 ? 1 : 0
-    else
-      if game.status == 3
-        shixun.task_pass ? 1 : 0
-      elsif game.status == 2
-        2
-      else
-        1
-      end
-    end
-  end
-
-  # ## 用户关卡状态 0: 不能开启实训; 1:直接开启; 2表示已完成
-  # def user_tpi_status shixun
+  # def user_tpi_status shixun, user_id
   #   # todo: 以前没加索引导致相同关卡,同一用户有多个games
   #   # 允许跳关则直接开启
-  #   identifier = self.identifier
-  #   if identifier.blank?
-  #     self.position == 1 ? 1 : 0
+  #   game = games.where(user_id: user_id).take
+  #   if game.blank?
+  #     position == 1 ? 1 : 0
   #   else
-  #     if status == 3
+  #     if game.status == 3
   #       shixun.task_pass ? 1 : 0
-  #     elsif status == 2
+  #     elsif game.status == 2
   #       2
   #     else
   #       1
@@ -124,6 +106,24 @@ class Challenge < ApplicationRecord
   #   end
   # end
 
+  # ## 用户关卡状态 0: 不能开启实训; 1:直接开启; 2表示已完成
+  def user_tpi_status shixun
+    # todo: 以前没加索引导致相同关卡,同一用户有多个games
+    # 允许跳关则直接开启
+    identifier = self.identifier
+    if identifier.blank?
+      self.position == 1 ? 1 : 0
+    else
+      if status == 3
+        shixun.task_pass ? 1 : 0
+      elsif status == 2
+        2
+      else
+        1
+      end
+    end
+  end
+
   def tags_show
     if self.challenge_tags.nil?
       "--"

app/views/challenges/index.json.jbuilder

@@ -30,6 +30,7 @@ if @challenges.present?
       json.down_url index_down_shixun_challenge_path(challenge, :shixun_identifier =>  @shixun.identifier) if @shixun.challenges_count != challenge.position
     end
     #json.passed challenge.has_passed?(@user.id)
-    json.status challenge.user_tpi_status(@shixun, @user.id)
+    #json.status challenge.user_tpi_status(@shixun, @user.id)
+    json.status challenge.user_tpi_status(@shixun)
   end
 end