指摘箇所2つ
ルーティングでresourcesを使う
パーシャルを使う
ルーティングでresourcesを使う
修正前(私のコード)
Rails.application.routes.draw do root to: 'product#new' get 'product/show/:id', to: 'product#show', as: 'product_show' end
修正後(↓のように修正すること)
Rails.application.routes.draw do resources :products,only: [:new, :show] end
なぜこのようにするのか?
⇨新しいルーティングを追加する時に修正後の方が楽
例えばindexアクションにつながるルーティングを作成したい場合
resources :products,only: [:new, :show,:index]
indexを追加するだけ
パーシャルを使う
2つの意図がありました。
他の箇所でも同じコードを使い回したい時に使う
可読性を上げる
通常1つのhtmlファイルに「header」,「main」「footer」が含まれる。ただ開発しているとmainだけ修正したいといった考えになることが多いです。そこでそれぞれパーシャルを分けることでmainの修正の際にheaderやfooterといったコードを見る必要がないので修正箇所を特定しやすいといったメリットがある