Qiita Teams that are logged in
You are not logged in to any team

Log in to Qiita Team
Community
OrganizationAdvent CalendarQiitadon (β)
Service
Qiita JobsQiita ZineQiita Blog
5
Help us understand the problem. What is going on with this article?
@AknEp

Draper(Decorator)の中でpartialを使うパターンが結構よかった話

More than 3 years have passed since last update.

概要

RailsのHelperはグローバルな名前空間を汚染するのでなるべくDraper(もしくはActiveDecorator)を使ってDecoratorパターンを使おうって話はあると思うんですが、
ここで、ちょっとしたHTMLのモジュールを表示するメソッドをDraperの中で書こうと思うと結構だるいわけですよ。

content_tag の闇

module ItemDecorator
  def facebook_btn
    h.content_tag(:div, class:'btn fb-btn') do
      # ごちゃごちゃいろいろ書く羽目になる
    end
  end
end

partial を render しちゃうと楽

で、普段はslimとかで書いてて、slimのコードをcontent_tagとか使ったコードに移植してたらミスも起きやすいし、甚だ非効率だと思ったので、こういうパターンを採用してみたら、思った以上に良かったので喧伝。

module ItemDecorator
  def facebook_btn
    h.render '/partials/items/facebook_btn', {item: self}
  end
end
/partials/items/_facebook_btn.html.slim
.btn.fb-btn
  / いつものslim記法で書けるぞ!

これだけで随分利便性あがるなーと思ったのでした。

「これなら単にpartialをrenderすればよくないか?」への反論

これだったら、単にrenderするコードをviewの中に埋めれば一緒じゃないか、と思われるかも知れないので先に反論しつつメリットがあるのだと主張しておきます。

まず、単にrenderするコードにすると、引数に指定すべきものが多岐に渡る場合、どの引数が必要か分かりにくいという問題があり、Decoratorのメソッドを挟むことでそれが解決されます。ようは以下の例。

module ItemDecorator
  def buy_btn(price, disabled=false)
    h.render '/partials/items/buy_btn', {item: self, price: price, disabled: disabled}
  end
end

通常のメソッドを挟んでいることで、IDEによる補完の恩恵も受けやすいです。

また、テストコードが書きやすいのもメリットです。 単に partialだけにすると、少しテストが書きにくいです(本質的には先に指摘したメソッドが切り出されているとイイネってのと同じことなのかもしれない)

5
Help us understand the problem. What is going on with this article?
Why not register and get more from Qiita?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
AknEp
旧アカです

Comments

No comments
Sign up for free and join this conversation.
Sign Up
If you already have a Qiita account Login
5
Help us understand the problem. What is going on with this article?