Rubocop -> Используйте пункт охраны - PullRequest
0 голосов
/ 17 мая 2019

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

Error

Style / GuardClause: используйте предложение guard вместо переноса кода внутри условного выражения. if @ save_search.save

код

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

1 Ответ

2 голосов
/ 17 мая 2019

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

return unless @saved_search.save

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

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

...