RemVee.blog

Remco @ Plugins naar Github

Remco — Wed, 18 Feb 2009 07:45:15 +0100

Eigenlijk moet je dutchify ook niet meer gebruiken omdat je sinds rails 2.2 fantastische i18n features tot je beschikking hebt.

Ik ben niet van plan dutchify aan te passen voor modernere versies van rails maar accepteer uiteraard wel patches. Fork is maar 1 klik en helemaal gratis! :)

Jeroen @ Plugins naar Github

Jeroen — Wed, 18 Feb 2009 07:13:30 +0100

Ik weet niet of dit de beste plek is voor zulk soort dingen maar ik wil je er even op wijzen dat Dutchify volgens mij niet meer werkt met Rails 2.2 en hoger.

Ik vermoed dat het er iets mee te maken heeft dat `default_error_messages’ deprecated is.

Remco @ Plugins naar Github

Remco — Thu, 04 Sep 2008 14:40:05 +0200

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!

p3t0r @ Plugins naar Github

p3t0r — Thu, 04 Sep 2008 13:29:02 +0200

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…

p3t0r @ Plugins naar Github

p3t0r — Thu, 04 Sep 2008 11:27:55 +0200

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.

p3t0r @ Plugins naar Github

p3t0r — Thu, 04 Sep 2008 10:45:49 +0200

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?