Булевы параметры в методах.Почему бы и нет?

Question

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

def method (flag = true)

reek дает мне предупреждение.Почему reek говорит мне, что это предупреждение?

Answer 1

За пару лет до того, как я написал reek, я написал в блоге об этом запахе кода здесь: http://silkandspinach.net/2004/07/15/avoid-boolean-parameters/

Все проблемы с булевыми параметрами связаны с дублированием и SRP: вызываемый метод имеет две обязанности, и вызывающая сторона знает, какую из них он хочет вызвать.

Answer 2

Старый вопрос, но я подумал, что я хотел бы добавить свои мысли для людей, которые приходят из Google:

Отдельные методы хороши в некоторых случаях, но лучший вариант, на мой взгляд, следующий:

def method(option = {:flag => false})
  options[:flag] ? doFoo : doBar
end

method(:flag => true)

Это также может включать проверку, действительно ли флаг является логическим:

def method(option = {:flag => false})
  flag = options.fetch(:flag) { false }
  unless [true, false].include?(flag)
    raise ArgumentError, ':flag option should be true or false'   
  end
  flag? doFoo : doBar
end

method(:flag => true)

Answer 3

Флаговая переменная считается плохой практикой. Это потому, что это по своей сути нарушает принцип, согласно которому функции должны выполнять одну и только одну вещь. Передавая логический флаг, вы говорите: «Делайте это, если это правда, и это, если это ложно», а это две вещи.