コードリファクタリングについて

Question

form_withの行が異なるだけで、それ以下の2行は同じです。
自分ではこういう形でしか記載できませんでしたが、もう少しシンプルに記載する方法はあるのでしょうか？

  <% if params[:id].blank? %>
    <%= form_with(url: schedules_path(day: params[:day]), method: :post) do |form| %>
      <%= render "form", schedule: @schedule %>
      <%= form.submit %>
    <% end %>
  <% else %>
    <%= form_with(url: schedule_path(@schedules.first, day: params[:day]), method: :patch) do |form| %>
      <%= render "form", schedule: @schedule %>
      <%= form.submit %>
    <% end %>
  <% end %>

---

▼▼　追記　▼▼

要件としては、indexページでスケジュールの新規登録・編集ができるようにしたいと考えています。
そのためidがあれば新規登録、idがなければ編集画面が表示される形となっています。

参照：Railsで更新も編集も削除も、全部indexページでやりたい

Comments

[m.ts10806]

もう少し要件を文章で説明してもらった方がいいかもしれません。
コードだけだと既に実現されたものですし、リファクタリングの本質はただコードだけを見て書き直すことではなく、書き直したとしても要件を満たさないといけませんから。
見た目だけ近くてそれっぽいコードにすることはできるかもしれませんが、それが要件満たすかどうかはまた別の話なので。

[no1knows]

ありがとうございます！追記いたしましたが、こちらで十分でしょうか？

[m.ts10806]

回答になっているか分かりませんが、現状で私が思うことを回答にしました。
ご参考まで。

Answer 1

シンプルになるかどうかは分からないのですが、
私としてはなるべくVIEWにロジックを書きたくないので、
コントローラ側で切り分けてそれぞれ別のVIEWを呼び出します。
画面や機能が違うなら尚更そのようにしたいかなと。
何か起きたときの問題切り分けもしやすくなります。

Comments

[no1knows]

ありがとうございます！
具体的には、下記の最初の投稿のようなイメージでしょうか？
https://discourse.clean-rails.org/t/topic/134

[m.ts10806]

はい。提示の通り、ケースによりrenderでVIEWに投げます。

[no1knows]

ありがとうございます！
今後おきるであろう問題の切り分けのためにもこのような実装方針で進めてみたいと思います！