Rails ルーティングとアクションの効率的な書きかた

Question

以下のようなルーティングとコントローラーの書き方をしている場合、

より少ない行数で書く方法などはないでしょうか？

教えて頂けると幸いです。

get "policy1" => "top#policy1"
get "policy2" => "top#policy2"
get "policy3" => "top#policy3"
get "policy4" => "top#policy4"
get "policy5" => "top#policy5"
             .
             .
             .
             .
get "policy20" => "top#policy20"

  def policy1
  end
  def policy2
  end
  def policy3
  end
  def policy4
  end
  def policy5
  end
             .
             .
             .
             .
  def policy20
  end

Comments

[m.ts10806]

メソッドは全て別でないといけませんか？

[mbaby__8]

メソッドの中には何も書かないので、1つでも表示できるのなら大丈夫だと思います。
policy1.html.erb～policy20.html.erbを表示するために書いている感じです。

[gouf]

フォーマットは html.erb を採用していますが、実際は静的ページだったりしますか? (ERB 不要ですか?)

Answer 1

あまり深く考えずに、Ruby のコードを書いてみてもいいかもしれません

config/routes.rb:

Rails.application.routes.draw do
  (1..20).each do |i|
    get "policy#{i}" => "top#policy#{i}"
  end
end

app/controllers/top_controller.rb:

class TopController < ApplicationController
  (1..20).each do |i|
    define_method("policy#{i}") {} # NOTE: メソッド定義のみで何もしない
  end
end

なにか参考になれば幸いです

Answer 2

メソッドの中には何も書かないので、1つでも表示できるのなら大丈夫だと思います。
policy1.html.erb～policy20.html.erbを表示するために書いている感じです。

下記の２つのどちらかで実現できそうな気がしますがいかがでしょうか？

１．Publicに置く
参考：https://www.javadrive.jp/rails/ini/index8.html

２． high_voltage gemを利用する（←使ったことはないですが・・・）
参考：https://machida.github.io/articles/20160718-rails-high-voltage/

Comments

[mbaby__8]

high_voltage初めて見ました。
静的なページたくさん作らないとならないので使えそうです。
ありがとうございます!

Answer 3

手元にあったのがチュートリアル用に立てたRails5.1なので適用できるか分かりませんが、下記のように試してみました。

routes.rb

get 'policy:cd'=>"top#policy"

top_controller.rb

    def policy
        render "policy".concat(params[:cd])
    end

これでブラウザから
/policy1とアクセスするとtop/policy1.html.erbが実行され
/policy2とアクセスするとtop/policy2.html.erbが実行されるようになりました。

私自身もRailsについては最近始めたばかりで、あくまで「policy/:cdができるならpolicy:cdでもいけるんじゃないか」という、これまで別の言語でWebアプリケーションを開発してきた経験からの仮説で試してみました。

これがいけるとなると、結構動的なURL,Viewを実行させられそうですね。
※動的すぎると全体像が見えなくなるので程ほどに・・・

Comments

[no1knows]

このやり方は初めてみました！（独学での経験しかないですが・・・）
ちなみに:cdは何の略語で、どういった意味があるのでしょうか？

[m.ts10806]

任意に勝手につけたパラメータ名なので特に何の略とかはないですが、大体は「code」の略ではないでしょうか。

あとご存じかもしれませんが、 :{名称} は「シンボル」ですね。
https://railstutorial.jp/chapters/rails_flavored_ruby?version=5.1#sec-hashes_and_symbols

[no1knows]

なるほど！勉強になります。いつもありがとうございます！

[no1knows]

すいません。1つ確認です。
このやり方の場合、renderファイルがないときは、Template is missingとかのエラーになってしまうのでしょうか？

[m.ts10806]

あくまで「できそう」で試した結果うまいことRailsが合致してくれただけだと思ってます。
実質Ruby触り始めて1か月程度なので、きっともっと良いやり方があるはずで。。。。

>このやり方の場合、renderファイルがないときは、Template is missingとかのエラーになってしまうのでしょうか？

手元に2まで作っていたので試しに　/policy3 でアクセスしたらおっしゃる通りTemplate is missingになりました。
エラーページに飛ばすように、エラーハンドリングも必要そうですね。

[no1knows]

あぁ、いいやり方だと思ったのですが・・・そうなっちゃうんですね。
検証いただきありがとうございます！

[m.ts10806]

> あぁ、いいやり方だと思ったのですが・・・そうなっちゃうんですね。

renderで直接テンプレート指定しているので仕方ない部分はあります。
ただ、エラーハンドリングなしのままでアプリケーション作ることはないと思うし、
「既知ではないルーティング」としてコントローラ側で処理させるのはある話だと思います。

低評価理由が気になりますが最近コメントつかない不純のなものばかりなので、無視しときます。

[no1knows]

実装方法の1つなので、低評価はおかしいですね。。。

[mbaby__8]

出来ました!
自分+で結合してエラーになっていたんですが、concatならエラーにならないんですね。
ありがとうございます。

[m.ts10806]

文字列結合の仕方は1つではないので、単に何かしら間違えてたのではないかと。
たまたま調べてすぐ使えそうだったのがconcatなので採用しただけです。