Rubocop - ›Используйте охранную оговорку

Как лучше всего исправить форматирование, чтобы избежать ошибки rubocop? Или есть способ изменить рубокоп? Что лучше?

Ошибка

Style / GuardClause: используйте защитное предложение вместо того, чтобы заключать код в условное выражение. если @ saved_search.save

Код

   if @saved_search.save
      respond_to do |format|
        format.html { redirect_to saved_searches_path }
        format.js {}
      end
    end

ruby-on-rails ruby-on-rails-5 rubocop

user2012677 17.05.2019 источник

arrow_upward
2
arrow_downward

Если у вас есть только это поведение (без учета "else"), вы можете сделать в этом формате:

return unless @saved_search.save

respond_to do |format|
  format.html { redirect_to saved_searches_path }
  format.js {}
end

Но я предлагаю вам добавить предложение else, чтобы позаботиться о том, чтобы сохранение вернулось false.

Douglas Lise 17.05.2019