写这个循环的更好方法是什么？

Question

关于如何为这个轨道控制器建立一个更好的循环的任何想法？

有时我在逻辑循环中有点迷路，所以我诉诸于尼安德特人的代码。

def index 

@step1_status = current_user.steps.pluck(:step1).first 
@step2_status = current_user.steps.pluck(:step2).first 
@step3_status = current_user.steps.pluck(:step3).first 
@step4_status = current_user.steps.pluck(:step4).first 
@step5_status = current_user.steps.pluck(:step5).first 
@step6_status = current_user.steps.pluck(:step6).first 
@step7_status = current_user.steps.pluck(:step7).first 
@step8_status = current_user.steps.pluck(:step8).first 
@step9_status = current_user.steps.pluck(:step9).first 

if @step9_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4', '5', '6', '7', '8', '9']) 
elsif @step8_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4', '5', '6', '7', '8']) 
elsif @step7_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4', '5', '6', '7']) 
elsif @step6_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4', '5', '6']) 
elsif @step5_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4', '5']) 
elsif @step4_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3', '4']) 
elsif @step3_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2', '3']) 
elsif @step2_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => ['1', '2']) 
elsif @step1_status == true 
    @task = Task.limit(1).order('sort_id ASC').where.not(:sort_id => '1') 
else 
    @task = Task.limit(1).order('sort_id ASC') 
end 

end 

感谢您的帮助！

Answer 1

我认为这应该是一样的。

首先尝试

@step_status = [] 
9.times do |n| 
    @step_status << current_user.steps.pluck("step#{n+1}").first 
end 

highest_step = @step_status.rindex(true) 
if highest_step 
    @task = Task.order('sort_id ASC').where.not(sort_id: (1..(highest_step+1))).first 
else 
    @task = Task.order('sort_id ASC').first 
end 

有点更加优化，更少的代码

highest_step = (1..9).to_a.rindex{|n| current_user.steps.pluck("step#{n+1}").first} + 1 

@task = Task.order('sort_id ASC') 
@task = @task.where.not(sort_id: (1..(highest_step))) if highest_step 
@task = @task.first 

只有2 DB调用

pluck_array = (1..9).map{|n| "step#{n}"} 
highest_step = current_user.steps.pluck(pluck_array).first.rindex(true) + 1 
#the + 1 accounts for the 1 being at index 0 through 9 at index 8 

@task = Task.order('sort_id ASC') 
@task = @task.where.not(sort_id: (1..(highest_step))) if highest_step 
@task = @task.first 

有点更容易阅读（虽然有点长）

step_array = (1..9).to_a 
pluck_array = step_array.map{|n| "step#{n}"} 
highest_index = current_user.steps.pluck(pluck_array).first.rindex(true) 
highest_step = step_array[highest_index] 

@task = Task.order('sort_id ASC') 
@task = @task.where.not(sort_id: (1..(highest_step))) if highest_step 
@task = @task.first