[Rails]collectionをrenderするときパーシャルのなかにパーシャルがある場合の最適な実装

Question

前提・実現したいこと

index.slim

= render partial: 'model', collection: models

_model.slim

= model.id

= render 'model_infomartion', model: model

このような場合、collectionの恩恵を受けることができないと思うのですが、良い手段はないでしょうか。
よろしくおねがいします。

Comments

[winterboum]

「collectionの恩恵を受けることができない 」 以前に、_model.slim で collection が参照されていないですね。これでは恩恵もなにも。。。。
どういう恩恵を受けたいのでしょう

[adagio]

> 以前に、_model.slim で collection が参照されていないですね。これでは恩恵もなにも。。。。

- models.each do |model|
= render 'model', model: model

とすると、renderが呼ばれるたびに`find_template`が呼ばれ、パフォーマンス的によろしくないという前提で、
以下のようにすると_model.slimを探すための`find_template`は一回で済みます。これが私が意図していた恩恵です。説明不足でした。
```
= render partial: 'model', collection: models
```

ここまではあたりまえのことですが、_model.slimの中でrenderを呼び出すため、結局models.size分renderが呼び出され、find_templateもmodels.size分呼ばれます。
これを避けるためのアイデアがないかを質問しています。
よろしくおねがいします。

Answer 1

_model_infomartionの中身がどれほどのものかわかりませんが、
パーシャルに分けずに_modelに含めてしまうというのが一番現実的なのかと思います。

量が多いとか他の場所でも使うとかだと現状の使い方が最適であり、
ある程度のパフォーマンス低下(？)について許容は必要になるかと思います。

collectionの恩恵を受けることができないと思うのですが、良い手段はないでしょうか。

_model.slimの中でrenderを呼び出すため、結局models.size分renderが呼び出され、find_templateもmodels.size分呼ばれます。

質問中と追記部分のこの内容についてですが、もしcollectionを使わない場合は、models.size * 2回分renderが呼ばれることになると思うので、collectionを使う恩恵に関してはそれを使った時点で受けていると思うのですがどうでしょうか？

Answer 2

なるほど。
templateを探す負荷が気になるほど models が大きいのでしょうか？

で、一般的に「最適」というのはわかりませんが、この例で考えると

model.slim は（ほぼ)これで全てですか？もしそうならばこれをindexに取り込んでも可読性は落ちないと思います。
それで templateを探す回数が半分になります。

= render partial: 'model', collection: models
とするならば　partial model にて

- collection.each do |model|
  = model.id
  = render 'model_infomartion', model: model

とすることになるかと思いますが、これですと partial models になっちゃいますね。意味合い的にというか、ん〜〜〜　感覚的にというか、なんか感じ悪くないですか？

Comments

[rhiroe]

そのサンプルコードだとNameErrorが起きてしまいますね...。
NameError (undefined local variable or method `collection' for main:Object)
が発生すると思います...。

[winterboum]

あれ？　
= render partial: 'model', collection: models
で collection が渡るかと思ったのですが

[rhiroe]

= render partial: 'model', local: { collection: models }
と書けばそうなりますけど、この場合の collection はオプションにあたります。

[winterboum]

あ、そうですね。