diff --git a/app/controllers/exercise_questions_controller.rb b/app/controllers/exercise_questions_controller.rb
index 96951ac48..490791918 100644
--- a/app/controllers/exercise_questions_controller.rb
+++ b/app/controllers/exercise_questions_controller.rb
@@ -384,24 +384,22 @@ class ExerciseQuestionsController < ApplicationController
       begin
         opr = params[:opr]
         current_q_p = @exercise_question.question_number.to_i  #问题的当前位置
-        # last_q_p = @exercise.exercise_questions.find_by_custom("question_number",(current_q_p - 1)).first  # 当前问题的前一个问题
-        # next_q_p = @exercise.exercise_questions.find_by_custom("question_number",(current_q_p + 1)).first  # 当前问题的后一个问题
-        last_q_p = @exercise.exercise_questions.last_exercise(current_q_p)  # 当前问题的前一个问题
-        next_q_p = @exercise.exercise_questions.next_exercise(current_q_p)  # 当前问题的后一个问题
         if @exercise.exercise_status.to_i == 1
           if opr.present?
             if opr.to_s == "up"
+              last_q_p = @exercise.exercise_questions.find_by(question_number: (current_q_p - 1))  # 当前问题的前一个问题
               if last_q_p.present?
                 @exercise_question.update_attribute('question_number', (current_q_p - 1))
-                last_q_p.update_attribute('question_number', (@exercise_question.question_number.to_i + 1)) # 重新获取当前问题的位置
+                last_q_p.update_attribute('question_number', current_q_p) # 重新获取当前问题的位置
                 normal_status(0, "问题上移成功！")
               else
                 normal_status(-1, "移动失败，已经是第一个问题了！")
               end
             elsif opr.to_s == "down"
+              next_q_p = @exercise.exercise_questions.find_by(question_number: (current_q_p + 1))  # 当前问题的前一个问题
               if next_q_p.present?
                 @exercise_question.update_attribute('question_number', (current_q_p + 1))
-                next_q_p.update_attribute('question_number', (@exercise_question.question_number.to_i - 1))
+                next_q_p.update_attribute('question_number', current_q_p)
                 normal_status(0, "问题下移成功！")
               else
                 normal_status(-1, "移动失败，已经是最后一个问题了！")
diff --git a/app/controllers/poll_questions_controller.rb b/app/controllers/poll_questions_controller.rb
index 74c73968c..47cd236a9 100644
--- a/app/controllers/poll_questions_controller.rb
+++ b/app/controllers/poll_questions_controller.rb
@@ -204,7 +204,6 @@ class PollQuestionsController < ApplicationController
               if last_q_p.present?
                 @poll_question.update_attribute("question_number", (current_q_p - 1))
                 last_q_p.update_attribute("question_number", current_q_p) # 重新获取当前问题的位置
-
                 normal_status(0, "问题上移成功！")
               else
                 normal_status(-1, "移动失败，已经是第一个问题了！")
diff --git a/app/models/exercise_question.rb b/app/models/exercise_question.rb
index ae58a7592..5f456e21f 100644
--- a/app/models/exercise_question.rb
+++ b/app/models/exercise_question.rb
@@ -15,8 +15,8 @@ class ExerciseQuestion < ApplicationRecord
   scope :find_by_custom, lambda {|k,v| where("#{k} = ?",v)}  #根据传入的参数查找问题
   scope :left_question_choose, lambda {|k,v| where("#{k} > ?",v)}  #根据传入的参数查找问题
   scope :find_objective_questions, -> {where("question_type != ?",4)}  #查找全部客观题
-  scope :next_exercise, lambda {|k| where("question_number > ?",k).first}
-  scope :last_exercise, lambda {|k| where("question_number < ?",k).last}
+  # scope :next_exercise, lambda {|k| where("question_number > ?",k).first}
+  # scope :last_exercise, lambda {|k| where("question_number < ?",k).last}
 
   def question_type_name
     case self.question_type
diff --git a/app/models/poll_question.rb b/app/models/poll_question.rb
index 958b69acd..1fc8e0c16 100644
--- a/app/models/poll_question.rb
+++ b/app/models/poll_question.rb
@@ -8,8 +8,6 @@ class PollQuestion < ApplicationRecord
   scope :ques_count, lambda {|k| where("question_type = ?",k)}
   scope :ques_necessary, -> {where("is_necessary = ?",1)}
   scope :insert_question, lambda {|k| where("question_number > ?",k)}
-  scope :next_poll, lambda {|k| where("question_number > ?",k).first}
-  scope :last_poll, lambda {|k| where("question_number < ?",k).last}
 
   def question_type_name
     case self.question_type