草野球の出欠確認Webアプリを作ろう！　part.7

これから作っていく簡単なWebアプリの作成メモ（自分の備忘）です。
自分用なのであまり凝りすぎないように書いていきたい。

[<<前回の記事](https://qiita.com/tomodachi_uec/items/b1b72b30713e97443230)

##今回やったこと

###前回作ったスケジュールの編集機能の修正点について

前回記事にて、一覧画面からテーブルの一行をクリックすることで、そのスケジュールの編集画面に遷移できるようにする変更をした。しかし、この時は眠くて画面遷移の動作確認を「一覧画面」⇒「編集画面」（⇒既設linkにて「一覧画面」）と、一度しか実施しなかった。
今日、改めて触ってみると、ページリロード後の「一覧画面」⇒「編集画面」⇒「一覧画面」は移動できるが、その後もう一度同じ動きで（或いはテーブルにある別のスケジュール行で）編集画面に遷移しようとしても、何も起こらなかった（！？）。

![無題.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/534558/c7b46f67-5bc8-2a73-8db0-5775598e9f83.png)

上記一覧画面からスケジュールをクリックして...

![無題.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/534558/98b25f38-e139-a72b-6604-2b7cf61f8759.png)

赤枠のリンクをクリックすることで一覧画面に戻って...

![無題.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/534558/c7b46f67-5bc8-2a73-8db0-5775598e9f83.png)

戻ってきた一覧画面からもう一度同じ遷移をしようとしても、何も起きない！？

なぜうまくいかないのか、だいぶ考えてしまいました。。。
そして出した結論が、
「もしかして、JQuery側でレンダリングしてる処理が最初（一覧画面の一番はじめの表示時）しか動いてない？？」
です。
ではなぜ、そのような現象が起こってしまうのか。
いろいろとWeb記事を見回って、やっとこれだというものを発見（以下記事です）。
[【Rails】turbolinksやAjaxの影響でjQueryが正常に動作しないとき](https://qiita.com/nsy_13/items/51d8a156d71964a135f4)

turbolinksがページ遷移時の動作を軽くしてくれていますが、これによってJQueryのイベント発火のタイミングが少し考慮しなければいけなくなり、その部分にまるで気が付いていなかったので、今回のようなダメな実装例が出来上がってしまったということです。
反省しつつ、前回書いたJQueryの処理を上記記事の通りに修正。

```js:app/javascript/packs/schedules.js
jQuery(document).on('turbolinks:load', function() {
  $(".clickable-row").css("cursor","pointer").on('click', function() {
    location.href = $(this).data("href");
  });
});
```

turbolinksを使う際には、このあたりの考慮をしないといけなかったんですね。
無知故致し方なし...。

###スケジュールの編集に対する画面遷移の変更

修正をしていて思ったこと。
「一覧からクリックしていきなり編集画面になるの、なんか変では？」

最初に作った時にはまるで気が付かなかったけど、一般的には以下のようになっていることが多い。
一覧画面⇔詳細表示画面⇔編集画面

ようやくそのことに気が付いたので、これも修正する。
まずは一覧画面からテーブルの行をクリックしたときの遷移先を詳細表示画面にする。

```erb:app/views/schedules/index.html.erb
（略）
<tbody>
  <% @schedules.each do |lst| %>
    <tr class="clickable-row" data-href="<%= schedule_path lst.id %>">
（略）
```

さらに、詳細表示画面から編集画面へのリンクを用意する。

```erb:app/views/schedules/show.html.erb
（略）
<div class="row_line">
  <%= link_to '編集', edit_schedule_path(@schedules.id), class: 'btn primary-btn' %>
</div>
```

![無題.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/534558/2dbb647f-cb5e-9439-d985-10932db3a49b.png)

動作も確認できた。




###スケジュールの削除

スケジュールを編集できても、削除できないと不便（というか最低限出来ないと困る）。
なので、スケジュールを削除できるようにします。
以下の記事は参考に読んだ。
[[Rails]deleteメソッドとdestroyメソッドの違いについて！！[初心者]](https://qiita.com/jackie0922youhei/items/588cd6f62cb1ff9e9809)

まずはコントローラーの手入れから。

```rb:app/controllers/schedules_controller.rb
（略）
def destroy
  @schedules = Schedule.find(params[:id])
  destroyed_title = @schedules.title
  @schedules.destroy
  redirect_to schedules_url, notice: "予定『#{destroyed_title}』を削除しました。"
end
（略）
```

※これまで、しれっと`destroy`アクションのことを`delete`って書いてしまっていたことに気が付いて、これも修正（Userコントローラーのほうもアクション名を修正した）。

次に、詳細表示画面に削除ボタンを配置する。

```erb:app/views/schedules/show.html.erb
（略）
<div class="row_line">
  <%= link_to '編集', edit_schedule_path(@schedules.id), class: 'btn primary-btn' %>
  <%= link_to '削除', schedule_path(@schedules.id), method: :delete, class: 'btn btn-danger' %>
</div>
```

できた。

![無題.png](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/534558/8bc82bfa-796e-69a8-89a2-9ff383df71ed.png)

動作も確認できたのでOK。

###今後やるべきこと

まだまだ最低限使えるものへの道のりは長い...。

- [ ] ユーザーでのログインの仕組みづくり
  - [ ] ユーザーの新規登録
  - [ ] ユーザーの編集
  - [ ] ユーザーの詳細表示
  - [ ] ユーザーの削除
- [ ] 見た目の改善
  - [ ] Bootstrapの有効化
- [ ] レスポンシブデザインへの対応
- [ ] データ削除時に「本当に削除しますか」の確認の実装
- [ ] 各種バリデーションの実装

今回はここまで。
記述ミスもまだまだあるので、見つけ次第修正したい。
あと、最低限必要なものが揃ったら、リファクタリングも時間を割いておきたい（願望）。


[次回の記事>>](https://qiita.com/tomodachi_uec/items/759418e0c871c7076c5f)