web-dev-qa-db-fra.com

Ruby embellissement du code, diviser les longues instructions sur plusieurs lignes

Comment pouvons-nous écrire la déclaration suivante pour améliorer la lisibilité?

Promotion.joins(:category).where(["lft>=? and rgt<=?", c.lft, c.rgt]).joins(:shops).where(:promotions_per_shops => { :shop_id => shops_id }).count('id', :distinct => true)

Ce qui suit ne compile pas

Promotion.joins(:category)
         .where(["lft>=? and rgt<=?", c.lft, c.rgt])
         .joins(:shops)
         .where(:promotions_per_shops => { :shop_id => shops_id })
         .count('id', :distinct => true)

syntax error, unexpected '.', expecting kEND
                     .where(["lft>=? and rgt<=?", c.lft, c.rgt])
rubycode-formatting
41
EricLarch

Fais-le comme ça:

Promotion.joins(:category).
         where(["lft>=? and rgt<=?", c.lft, c.rgt]).
         joins(:shops).
         where(:promotions_per_shops => { :shop_id => shops_id }).
         count('id', :distinct => true)
51
Geo

Également possible de faire

Promotion.joins(:category) \
         .where(["lft>=? and rgt<=?", c.lft, c.rgt]) \
         .joins(:shops) \
         .where(:promotions_per_shops => { :shop_id => shops_id }) \
         .count('id', :distinct => true)
56
EricLarch

Il devrait être compilé en 1.9. Dans les versions précédentes, il n'était pas valide en effet.

14
Mchl