Rails реорганизовать код

Question

У меня был код ниже:

@brand = Brand.find_by_id(params[:id]) 
    if @brand.nil? 
    flash[:error] = Flash.record_not_found 
    redirect_to admin_brands_path 
    end 

И еще одно изменение ниже:

@brand = Brand.find_by_id(params[:id]) 
return(flash[:error] = Flash.record_not_found and redirect_to admin_brands_path) if @brand.nil? 

Какой код вы считаете более эффективным и объяснить? , и когда у вас есть другое предложение, вы можете поделиться тоже.

Заранее спасибо.

Answer 1

Я чувствую верхний один код лучше, как легко понять, и очень чистый, но вы можете написать это следующим образом тоже

unless @brand = Brand.find_by_id(params[:id]) 
    flash[:error] = Flash.record_not_found 
    redirect_to admin_brands_path 
end 

Answer 2

Первый вариант, безусловно, намного лучше - нет никаких сомнений , Он читается, не слишком много логики внутри, и он делает только то, что должны делать контроллеры. Наличие наименьших строк кода действительно не является хорошей метрикой.

Что касается рефакторинга, я оставил бы его так, как есть. Возможно, измените #find_by_id на #find, но это все.

Answer 3

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

def action 
    @brand = Brand.find(params[:id]) 
rescue ActiveRecord::RecordNotFound 
    redirect_to admin_brands_path, flash: {error: Flash.record_not_found} 
end