質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
86.12%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

解決済

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

no1knows
no1knows

総合スコア3256

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

1回答

0リアクション

1クリップ

193閲覧

投稿2022/09/25 12:37

編集2022/09/26 00:47

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

Ruby

<% 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ページでやりたい

以下のような質問にはリアクションをつけましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

リアクションが多い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

気になる質問をクリップする

クリップした質問は、後からいつでもマイページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

m.ts10806

2022/09/25 21:09

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

2022/09/26 00:48

ありがとうございます!追記いたしましたが、こちらで十分でしょうか?
m.ts10806

2022/09/26 00:57

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

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
86.12%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

同じタグがついた質問を見る

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。