RemVee.blog

Cool, kan ik lekker makkelijk meegluren.

Ik vroeg me af waarom je in die NullableBooleans die extra if in het begin hebt staan. Dit als die weg laat en nil aan de derde when toevoegt werkt het ook.. met minder code:

def self.value_to_boolean(value)
  case value.to_s.downcase
    when 'true', 't', '1'
      true
    when 'false', 'f', '0'
      false
    when 'nil', ''
      nil
    else
      false
  end
end

Of is dat performancewise niet zo slim?

Hmmm nu ik er nog eens over nadenk is dit eigenlijk nog korter:

def self.value_to_boolean(value)
  case value.to_s.downcase
    when 'true', 't', '1'
      true
    when 'nil', ''
      nil
    else
      false
  end
end

aangezien je default returnvalue toch false is. Bovendien ben je dan twee branches kwijt waardoor performance misschien nog beter is dan met die passthrough if.

ik kon het niet laten:

def self.value_to_boolean(value)
  [nil, ''].include?(value) ? nil : ['true', 't', '1'].include?(value.to_s.downcase) ? true : false
end

ik geef toe dat de leesbaarheid niet optimaal meer is…

Leesbaarheid (mijn versie) vind ik belangrijker dan branches of feestjes met de ternary operator. Overigens is het gebruik van ternary operators in Ruby vaak lelijk:

hungry? ? :food : :drinks

Argh.. te veel vraagtekens en dubbele punten!