rails7 メンターさんからのフィードバック

指摘箇所２つ

1. ルーティングでresourcesを使う

2. パーシャルを使う

ルーティングで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を追加するだけ

パーシャルを使う

２つの意図がありました。

• 他の箇所でも同じコードを使い回したい時に使う

• 可読性を上げる

通常1つのhtmlファイルに「header」,「main」「footer」が含まれる。ただ開発しているとmainだけ修正したいといった考えになることが多いです。そこでそれぞれパーシャルを分けることでmainの修正の際にheaderやfooterといったコードを見る必要がないので修正箇所を特定しやすいといったメリットがある