Есть ли более компактный способ написать этот код ROR?

Question

Есть ли более компактный способ написать следующий код. Я хотел бы избавиться от строки, которая присваивает пустую строку, когда flash [: add_run_error] равен нулю.

unless run.save 
    run.errors.each do |attr, msg| 
    flash[:add_run_error] += '<br/>' if flash[:add_run_error] 
    flash[:add_run_error] = '' unless flash[:add_run_error] 
    flash[:add_run_error] += "Invalid #{attr}. Follow examples below." 
    end 
end 

Answer 1

Вы можете просто присоединиться к части своих ошибок вместе с attr.

flash[:add_run_error] = run.errors.map{|attr, msg| "Invalid #{attr}. Follow examples below."}.join('<br/>') 

Answer 2

Я хотел бы сделать это следующим образом:

unless run.save 
    add_run_errors = [] 
    run.errors.each do |attr, msg| 
    add_run_errors << "Invalid #{attr}. Follow examples below." 
    end 
    flash[:add_run_error] = add_run_errors.join '<br />' 
end 

Но это не первый <br /> - вы можете добавить его просто:

flash[:add_run_error] = '<br /'> + (add_run_errors.join '<br />')