20/05/2020 Railsチュートリアル Ruby on Rails 学習メモ

らくだ🐫にもできるRailsチュートリアル｜13.3

※毎回書いたほうがいい気がするので書いておきます※
平文の部分をちょこちょこI18nで日本語化しているのですが
その部分に関しては特に記述していなくてもたまにコードが違ったりしています。
t(‘.hogehoge’)って部分です
どうぞお気になさらず。

13.3 マイクロポストを操作する

モデリングとテンプレートが完成したのでweb経由でマイクロポストを操作できるようにしていく

従来のRailsの開発の慣習と異なる部分

Micropostsリソースへのインターフェイスは、主にプロフィールページとHomeページのコントローラを経由して実行されるので、Micropostsコントローラにはnewやeditのようなアクションは不要ということになります。つまり、createとdestroyがあれば十分です。

よってMicropostのリソースは下記↓のように

・
・
・
  resources :users
  # account_activationsresourceのeditへのルーティングのみを生成
  resources :account_activations, only: [:edit]
  # password_resetsのnew、create、edit、updateのルーティングを生成 
  resources :password_resets,     only: [:new, :create, :edit, :update]
  # micropostsのcreateとdestroyのルーティングを生成
  resources :microposts,          only: [:create, :destroy]

end

・

resources :users

# account_activationsresourceのeditへのルーティングのみを生成

resources :account_activations, only: [:edit]

# password_resetsのnew、create、edit、updateのルーティングを生成

resources :password_resets, only: [:new, :create, :edit, :update]

# micropostsのcreateとdestroyのルーティングを生成

resources :microposts, only: [:create, :destroy]

end

提供されるRESTfulなルート　→　RoRT本文参照

13.3.1 マイクロポストのアクセス制御

Micropostsコントローラ内のアクセス制御から始める
→Micropostsは関連付けられたユーザーを通してアクセスされるので
createアクションやdestroyアクションを利用するユーザーはログインしている必要がある

ログイン済みかどうかを確かめるテストでは、Usersコントローラ用のテストがそのまま役に立ちます

ログイン済でない場合の動作↓

正しいリクエストを各アクションへ発行
マイクロポストの数が変化していないか
正しくリダイレクトされているか

require 'test_helper'

class MicropostsControllerTest < ActionDispatch::IntegrationTest

  def setup
    @micropost = microposts(:orange)
  end

  test "should redirect create when not logged in" do
    # ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない
    assert_no_difference 'Micropost.count' do
      # microposts_pathに　paramsハッシュのデータを持たせてpostのリクエスト
      post microposts_path, params: { micropost: { content: "Lorem ipsum" } }
    end
    # login_urlにリダイレクト
    assert_redirected_to login_url
  end

  test "should redirect destroy when not logged in" do
    # ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない
    assert_no_difference 'Micropost.count' do
      # micropost_path(@micropost)にdeleteのリクエスト
      delete micropost_path(@micropost)
    end
    # login_urlにリダイレクト
    assert_redirected_to login_url
  end
end

require 'test_helper'

class MicropostsControllerTest < ActionDispatch::IntegrationTest

def setup

@micropost = microposts(:orange)

end

test "should redirect create when not logged in" do

# ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない

assert_no_difference 'Micropost.count' do

# microposts_pathに　paramsハッシュのデータを持たせてpostのリクエスト

post microposts_path, params: { micropost: { content: "Lorem ipsum" } }

end

# login_urlにリダイレクト

assert_redirected_to login_url

end

test "should redirect destroy when not logged in" do

# ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない

assert_no_difference 'Micropost.count' do

# micropost_path(@micropost)にdeleteのリクエスト

delete micropost_path(@micropost)

end

# login_urlにリダイレクト

assert_redirected_to login_url

end

実装前のリファクタリング

ログインの有無を確認するためにbeforeフィルターで定義したlogged_in_userメソッド
→Usersコントローラで定義してあるけどMicropostsコントローラでも使いたい！
→各コントローラが継承するApplicationコントローラに移動する

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception
  #Sessionヘルパーモジュールの読み込み
  include SessionsHelper
  
  private
  
    # ログイン済みユーザーかどうか確認
    def logged_in_user
      # logged_in?がfalseの場合
      unless logged_in?
        # SessionsHelperメソッド　store_locationの呼び出し
        store_location
        # flashsでエラーメッセージを表示
        flash[:danger] = t('users.please_log_in')
        # login_urlにリダイレクト
        redirect_to login_url
      end
    end
    
end

class ApplicationController < ActionController::Base

protect_from_forgery with: :exception

#Sessionヘルパーモジュールの読み込み

include SessionsHelper

private

# ログイン済みユーザーかどうか確認

def logged_in_user

# logged_in?がfalseの場合

unless logged_in?

# SessionsHelperメソッド　store_locationの呼び出し

store_location

# flashsでエラーメッセージを表示

flash[:danger] = t('users.please_log_in')

# login_urlにリダイレクト

redirect_to login_url

end

重複しないようにUsersコントローラから該当部分を削除

・
・
・
    # beforeアクション

    # 以下、ハイライト部分を削除
    # # ログイン済みユーザーかどうか確認
    # def logged_in_user
    #   # logged_in?がfalseの場合
    #   unless logged_in?
    #     # SessionsHelperメソッド　store_locationの呼び出し
    #     store_location
    #     # flashsでエラーメッセージを表示
    #     flash[:danger] = t('users.please_log_in')
    #     # login_urlにリダイレクト
    #     redirect_to login_url
    #   end
    # end
    
    # 正しいユーザーかどうか確認
    def correct_user
      @user = User.find(params[:id])
      # root_urlにリダイレクト　current_user?メソッドがfalseの場合
      redirect_to(root_url) unless current_user?(@user)
    end
    
    # 管理者かどうか確認
    def admin_user
      # root_urlにリダイレクト current_user.admin?(ログイン中のユーザーが管理者であるか)がfalseの場合
      redirect_to(root_url) unless current_user.admin?
    end
end

・

# beforeアクション

# 以下、ハイライト部分を削除

# # ログイン済みユーザーかどうか確認

# def logged_in_user

# # logged_in?がfalseの場合

# unless logged_in?

# # SessionsHelperメソッド　store_locationの呼び出し

# store_location

# # flashsでエラーメッセージを表示

# flash[:danger] = t('users.please_log_in')

# # login_urlにリダイレクト

# redirect_to login_url

# end

# 正しいユーザーかどうか確認

def correct_user

@user = User.find(params[:id])

# root_urlにリダイレクト　current_user?メソッドがfalseの場合

redirect_to(root_url) unless current_user?(@user)

end

# 管理者かどうか確認

def admin_user

# root_urlにリダイレクト current_user.admin?(ログイン中のユーザーが管理者であるか)がfalseの場合

redirect_to(root_url) unless current_user.admin?

end

Micropostsコントローラへbeforフィルターを追加

class MicropostsController < ApplicationController
  # 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用
  before_action :logged_in_user, only: [:create, :destroy]
  
  def create
  end

  def destroy
  end
end

class MicropostsController < ApplicationController

# 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用

before_action :logged_in_user, only: [:create, :destroy]

def create

end

def destroy

end

これでテストもGREEN！

演習

なぜUsersコントローラ内にあるlogged_in_userフィルターを残したままにするとマズイのでしょうか? 考えてみてください。

1.
重複したコードはトラブルの元になる！
→片方だけ編集しちゃったりとか

重複したコードを残したまま片方を編集しちゃしまうとどうなるか

class UsersController < ApplicationController

1	class UsersController < ApplicationController

ApplicationControllerを継承したUsersControllerで重複したコードに一部変更があった場合
UsersControllerのコードが優先される→オーバーライド

オーバーライド（英：override）とは

オブジェクト指向におけるオブジェクトの継承の話で出てくる用語のひとつ

であり

親クラスにあるメソッドを子クラスで再定義することによって、子クラス上で親クラスのメソッドを上書きすること

Error 403 (Forbidden)｜「分かりそう」で「分からない」でも「分かった」気になれ...

~~https://wa3.i-3-i.info/word138.html~~

403エラーページです。用語の意味を「ざっくりと」理解するためのIT用語辞典です。

wa3.i-3-i.info

今回は両方のコントローラーで同じ挙動をさせたいのにFインジェクション
重複コードを残したまま、それを忘れてどちらか一方を編集してしまうと呼び出すコントローラにより挙動が変わってしまう
→意図しない挙動になってしまうのでいくない！

13.3.2 マイクロポストを作成する

マイクロポストの作成用フォームはmicropost/newではなくルートパス（ホーム画面）に作る
参考画面は本文参考

最後にホーム画面を実装したときは (図 5.8)、[Sign up now!] ボタンが中央にありました。マイクロポスト作成フォームは、ログインしている特定のユーザーのコンテキストでのみ機能するので、この節の一つの目標は、ユーザーのログイン状態に応じて、ホーム画面の表示を変更することです。

Micropostのcreateアクション

基本的にはUsersコントローラのcreateアクションと似ているけど
User/Micropost関連付けを使って新しいマイクロポストを作る点が違っている
また、外部メソッドのmicropost_paramsでStrong Parametersを使い、マイクロポストのcontent属性だけをWeb経由で変更可能にしてある
実際のコード↓

class MicropostsController < ApplicationController
  # 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用
  before_action :logged_in_user, only: [:create, :destroy]
  
  def create
    # @micropostに　ログイン中のユーザーに紐付いた新しいマイクロポストオブジェクトを返す（引数　micropost_params）
    @micropost = current_user.microposts.build(micropost_params)
    #  @micropostを保存できれば
    if @micropost.save
      # 成功のフラッシュメッセージを表示
      flash[:success] = t('.micropost_created')
      # ルートURLにリダイレクト
      redirect_to root_url
    # 保存できなければ
    else
      # static_pages/homeを描画
      render 'static_pages/home'
    end
  end

  def destroy
  end
  
  private

    def micropost_params
      # micropost属性必須　content属性のみ変更を許可
      params.require(:micropost).permit(:content)
    end
end

class MicropostsController < ApplicationController

# 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用

before_action :logged_in_user, only: [:create, :destroy]

def create

# @micropostに　ログイン中のユーザーに紐付いた新しいマイクロポストオブジェクトを返す（引数　micropost_params）

@micropost = current_user.microposts.build(micropost_params)

# @micropostを保存できれば

if @micropost.save

# 成功のフラッシュメッセージを表示

flash[:success] = t('.micropost_created')

# ルートURLにリダイレクト

redirect_to root_url

# 保存できなければ

else

# static_pages/homeを描画

render 'static_pages/home'

end

def destroy

end

private

def micropost_params

# micropost属性必須　content属性のみ変更を許可

params.require(:micropost).permit(:content)

end

マイクロポスト作成フォーム

専用ページではなくstatic_pages/home.html.erbにフォームを作る
更に、userがログインしているかどうかで表示を変える（ログインしていなければ今まで通り、していたらフォームを表示）
ひとまずこう↓

<!--ログインしていればこっち-->
<% if logged_in? %>
  <div class="row">
    <aside class="col-md-4">
      <section class="user_info">
        <%= render 'shared/user_info' %>
      </section>
      <section class="micropost_form">
        <%= render 'shared/micropost_form' %>
      </section>
    </aside>
  </div>
<!--ログインしていなければこっち-->
<% else %>
  <div class="center jumbotron">
    <h1><%= t('.Welcome_to_the_Sample_App') %></h1>
  
    <h2>
      <%= t('.welcome_message_html') %>
    </h2>
  
    <%= link_to "Sign up now!", signup_path, class: "btn btn-lg btn-primary" %>
  </div>
  
  <%= link_to image_tag("rails.png", alt: "Rails logo"),
              'http://rubyonrails.org/' %>
<% end %>

<% if logged_in? %>

<%= render 'shared/user_info' %>

</section>

<%= render 'shared/micropost_form' %>

</section>

</aside>

</div>

<% else %>

<h2>

<%= t('.welcome_message_html') %>

</h2>

<%= link_to "Sign up now!", signup_path, class: "btn btn-lg btn-primary" %>

</div>

<%= link_to image_tag("rails.png", alt: "Rails logo"),

'http://rubyonrails.org/' %>

<% end %>

イマイチなので演習でリファクタリングするのだそうです

また、パーシャルもいくつか追加

<!--サイドバーのユーザー情報部分-->
<%= link_to gravatar_for(current_user, size: 50), current_user %>
<h1><%= current_user.name %></h1>
<span><%= link_to "view my profile", current_user %></span>
<span><%= pluralize(current_user.microposts.count, "micropost") %></span>

<%= link_to gravatar_for(current_user, size: 50), current_user %>

<!--マイクロポスト投稿フォーム-->
<%= form_for(@micropost) do |f| %>
  <%= render 'shared/error_messages', object: f.object %>
  <div class="field">
    <%= f.text_area :content, placeholder: "Compose new micropost..." %>
  </div>
  <%= f.submit "Post", class: "btn btn-primary" %>
<% end %>

<%= form_for(@micropost) do |f| %>

<%= render 'shared/error_messages', object: f.object %>

<%= f.text_area :content, placeholder: "Compose new micropost..." %>

</div>

<%= f.submit "Post", class: "btn btn-primary" %>

<% end %>

マイクロポストの投稿フォームを動かす

homeアクションにマイクロポストのインスタンス変数を追加

class StaticPagesController < ApplicationController
  
  def home
    # @micropostに　ユーザーがログインしていればログイン中のユーザーに紐付いたMicropostオブジェクトを返す
    @micropost = current_user.microposts.build if logged_in?
  end

・
・
・

class StaticPagesController < ApplicationController

def home

# @micropostに　ユーザーがログインしていればログイン中のユーザーに紐付いたMicropostオブジェクトを返す

@micropost = current_user.microposts.build if logged_in?

end

・

これによりユーザーがログインしている時のみ@micropost変数が定義される

エラーメッセージのパーシャルを定義

_micropost_form.html.erb内のこのコード↓が動くようにするため

<%= render 'shared/error_messages', object: f.object %>

1	<%= render 'shared/error_messages', object: f.object %>

現在のエラーパーシャルはこう

<% if @user.errors.any? %>
  <div id="error_explanation">
    <div class="alert alert-danger">
       <%= t('.errors count', errors_count: @user.errors.count) %>
    </div>
    <ul>
    <% @user.errors.full_messages.each do |msg| %>
      <li><%= msg %></li>
    <% end %>
    </ul>
  </div>
<% end %>

<% if @user.errors.any? %>

<%= t('.errors count', errors_count: @user.errors.count) %>

</div>

<ul>

<% @user.errors.full_messages.each do |msg| %>

<% end %>

</ul>

</div>

<% end %>

このままではviewのこの↓コードが動かない

<%= render 'shared/error_messages', object: f.object %>

1	<%= render 'shared/error_messages', object: f.object %>

元々のエラーメッセージパーシャルでは@users変数を直接参照している
→今回は@micropost変数を参照したい！
→@users変数でも@micropostでも参照できたい！
→フォーム変数fをf.objectにして関連付けられたオブジェクトにアクセスできるようにする！

パーシャルにオブジェクトを渡すために、値がオブジェクトで、キーがパーシャルでの変数名と同じハッシュを利用します。これで、リスト 13.39の2行目のコードが完成します。言い換えると、object: f.objectはerror_messagesパーシャルの中でobjectという変数名を作成してくれるので、この変数を使ってエラーメッセージを更新すればよいということです

<% if object.errors.any? %>
  <div id="error_explanation">
    <div class="alert alert-danger">
      The form contains <%= pluralize(object.errors.count, "error") %>.
    </div>
    <ul>
    <% object.errors.full_messages.each do |msg| %>
      <li><%= msg %></li>
    <% end %>
    </ul>
  </div>
<% end %>

<% if object.errors.any? %>

The form contains <%= pluralize(object.errors.count, "error") %>.

</div>

<ul>

<% object.errors.full_messages.each do |msg| %>

<% end %>

</ul>

</div>

<% end %>

error_messagesパーシャルを更新してもテストはRED
→error_messagesパーシャルは他のviewからも呼び出されているため
関連するviewを更新するとGREENになる！
※users/new.html.erbとusers/edit.html.erbのフォームはパーシャルでまとめているので
　本文と違ってusers/_form.html.erbを更新する

<!--yieldメソッドで個別に設定したurlを渡す-->
<%= form_for(@user, url: yield(:url)) do |f| %>
    <%= render 'shared/error_messages', object: f.object %>

  <%= f.label :name %>
  <%= f.text_field :name, class: 'form-control' %>

<%= form_for(@user, url: yield(:url)) do |f| %>

<%= render 'shared/error_messages', object: f.object %>

<%= f.label :name %>

<%= f.text_field :name, class: 'form-control' %>

<% provide(:title, t('.reset_password_title')) %>
<h1><%= t('.reset_password_title') %></h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= form_for(@user, url: password_reset_path(params[:id])) do |f| %>
      <%= render 'shared/error_messages', object: f.object %>

      <%= hidden_field_tag :email, @user.email %>
・
・
・

<% provide(:title, t('.reset_password_title')) %>

<%= form_for(@user, url: password_reset_path(params[:id])) do |f| %>

<%= render 'shared/error_messages', object: f.object %>

<%= hidden_field_tag :email, @user.email %>

・

GREENにならないね？って思ったら日本語化のアレでナニてったので直してGREEN！

演習

Homeページをリファクタリングして、if-else文の分岐のそれぞれに対してパーシャルを作ってみましょう。

1.
それぞれのファイルを作って該当部分を移動
（I18n対応はja.yml作り直すの面倒だったので階層で指定）

<div class="row">
  <aside class="col-md-4">
    <section class="user_info">
      <%= render 'shared/user_info' %>
    </section>
    <section class="micropost_form">
      <%= render 'shared/micropost_form' %>
    </section>
  </aside>
</div>

<%= render 'shared/user_info' %>

</section>

<%= render 'shared/micropost_form' %>

</section>

</aside>

</div>

<div class="center jumbotron">
  <h1><%= t('static_pages.home.Welcome_to_the_Sample_App') %></h1>

  <h2>
    <%= t('static_pages.home.welcome_message_html') %>
  </h2>

  <%= link_to "Sign up now!", signup_path, class: "btn btn-lg btn-primary" %>
 </div>

<%= link_to image_tag("rails.png", alt: "Rails logo"),
              'http://rubyonrails.org/' %>

<h2>

<%= t('static_pages.home.welcome_message_html') %>

</h2>

<%= link_to "Sign up now!", signup_path, class: "btn btn-lg btn-primary" %>

</div>

<%= link_to image_tag("rails.png", alt: "Rails logo"),

'http://rubyonrails.org/' %>

それぞれを呼び出すコードを追加

<!--ログインしていればこっち-->
<% if logged_in? %>
  <%= render 'static_pages/user_logged_in' %>
<!--ログインしていなければこっち-->
<% else %>
  <%= render 'static_pages/user_logged_out' %>
<% end %>

<% if logged_in? %>

<%= render 'static_pages/user_logged_in' %>

<% else %>

<%= render 'static_pages/user_logged_out' %>

<% end %>

13.3.3 フィードの原型

homeのページにマイクロポストが表示されるようにする→マイクロポストのフィードに実装
RoRT本文参照
全てのユーザーがフィードを持つので、feedメソッドはUserモデルに作る
→まずはログインユーザーのマイクロポストをすべて取得
→後々フォローしているユーザーのマイクロポストを含めたfeedを実装するので今回はwhereメソッドを使う

・
・
・
  # 試作feedの定義
  # 完全な実装は次章の「ユーザーをフォローする」を参照
  def feed
    Micropost.where("user_id = ?", id)
  end

    private
・
・
・

・

# 試作feedの定義

# 完全な実装は次章の「ユーザーをフォローする」を参照

def feed

Micropost.where("user_id = ?", id)

end

private

・

上記ハイライト行はセキュリティ上重要な役割を果たしている！

SQLクエリに代入する前にidがエスケープされるため、SQLインジェクション (SQL Injection) と呼ばれる深刻なセキュリティホールを避けることができます。この場合のid属性は単なる整数 (すなわちself.idはユーザーのid) であるため危険はありませんが、SQL文に変数を代入する場合は常にエスケープする習慣をぜひ身につけてください。

SQLインジェクションとは何ぞ

穴埋めになっているSQL文の穴埋め部分に、作った人が意図しない内容を入れることによって、おかしな動きをさせること。もしくは、それができるようになっている状態

Error 403 (Forbidden)｜「分かりそう」で「分からない」でも「分かった」気になれ...

~~https://wa3.i-3-i.info/word12800.html~~

403エラーページです。用語の意味を「ざっくりと」理解するためのIT用語辞典です。

wa3.i-3-i.info

Micropost.where(“user_id = ?”, id)とはなんぞ

プレースホルダー（?　の事）を使った書き方
第一引数の user_id =　？　の？に　第2引数の　id が置き換わる

whereメソッドを使って欲しいデータの取得をしよう！

https://pikawaka.com/rails/where#プレースホルダの記述の仕方

railsのwhereメソッド使い方を実際の例を用いてわかりすく解説します！どういうときに使うのか？どのように使うのか？この記事を読んで検索機能を実装してみましょう。

Pikawaka - ピカ1わかりやすいプログラミング用語サイト

SQLインジェクションを避けることが出来る！
また、上記のコードは現状コレ↓と同等だけど後々（14章）の為にこの↑形に

def feed
  microposts
end

def feed

microposts

end

フィード機能を実装していく

ログインユーザーのフィード用にインスタンス変数@feed_itemsを追加

class StaticPagesController < ApplicationController
  
  def home
    # もしログインしていたら
    if logged_in?
      # @micropostに　current_userに紐付いたMicropostオブジェクトを代入
      @micropost  = current_user.microposts.build
      # @feed_itemsにcurrent_userに紐付いたフィードのpaginate(page: params[:page])を代入
      @feed_items = current_user.feed.paginate(page: params[:page])
    end
  end
・
・
・

class StaticPagesController < ApplicationController

def home

# もしログインしていたら

if logged_in?

# @micropostに　current_userに紐付いたMicropostオブジェクトを代入

@micropost = current_user.microposts.build

# @feed_itemsにcurrent_userに紐付いたフィードのpaginate(page: params[:page])を代入

@feed_items = current_user.feed.paginate(page: params[:page])

end

・

# 以前のコードが
@micropost = current_user.microposts.build if logged_in?

↓

# こうなっている
if logged_in?
  @micropost  = current_user.microposts.build
  @feed_items = current_user.feed.paginate(page: params[:page])
 end

# 以前のコードが

@micropost = current_user.microposts.build if logged_in?

↓

# こうなっている

if logged_in?

@micropost = current_user.microposts.build

@feed_items = current_user.feed.paginate(page: params[:page])

end

これはRubyの慣習で、1行の時は後置if文、2行以上の時は前置if文を使う為

Homeページにフィード用のパーシャルを追加

<% if @feed_items.any? %>
  <ol class="microposts">
    <%= render @feed_items %>
  </ol>
  <%= will_paginate @feed_items %>
<% end %>

<% if @feed_items.any? %>

<%= render @feed_items %>

</ol>

<%= will_paginate @feed_items %>

<% end %>

この時

<%= render @feed_items %>

1	<%= render @feed_items %>

@feed_itemsにはcurrent_userに紐付いたfeedのpaginate(page: params[:page])が代入されており

# @feed_itemsにcurrent_userに紐付いたfeedのpaginate(page: params[:page])を代入
@feed_items = current_user.feed.paginate(page: params[:page])

1 2	# @feed_itemsにcurrent_userに紐付いたfeedのpaginate(page: params[:page])を代入 @feed_items = current_user.feed.paginate(page: params[:page])

feedってなんだっけって言うと

def feed
  # Micropostテーブルからuser_idがidのユーザーをすべて取得
  Micropost.where("user_id = ?", id)
end

def feed

# Micropostテーブルからuser_idがidのユーザーをすべて取得

Micropost.where("user_id = ?", id)

end

つまり@feed_itemsの各要素はMicropostクラスを持っている
よってRailsはMicropostのパーシャル（_micropost.html.erb）を呼び出すことができる。

このように、Railsは対応する名前のパーシャルを、渡されたリソースのディレクトリ内から探しにいくことができます。

→Homeページにステータスフィードを追加

<div class="row">
  <aside class="col-md-4">
    <section class="user_info">
      <%= render 'shared/user_info' %>
    </section>
    <section class="micropost_form">
      <%= render 'shared/micropost_form' %>
    </section>
  </aside>
    <div class="col-md-8">
      <h3>Micropost Feed</h3>
      <%= render 'shared/feed' %>
    </div>
</div>

<%= render 'shared/user_info' %>

</section>

<%= render 'shared/micropost_form' %>

</section>

</aside>

<h3>Micropost Feed</h3>

<%= render 'shared/feed' %>

</div>

※本文はapp/views/static_pages/home.html.erbになってるけど
13.3.2の演習でパーシャルに分けているので
「ログイン済の際に表示されるほう」のパーシャルにコードを追加する

現状の問題

現時点では、新しいマイクロポストの作成は図 13.15で示したように期待どおりに動作しています。ただしささいなことではありますが、マイクロポストの投稿が失敗すると、 Homeページは@feed_itemsインスタンス変数を期待しているため、現状では壊れてしまいます。

→「NoMethodError in Microposts#create」になる
回避用のコードを追加する

・
・
・
  def create
    @micropost = current_user.microposts.build(micropost_params)
    if @micropost.save
      flash[:success] = "Micropost created!"
      redirect_to root_url
    else
      @feed_items = []
      render 'static_pages/home'
    end
  end
・
・
・

・

def create

@micropost = current_user.microposts.build(micropost_params)

if @micropost.save

flash[:success] = "Micropost created!"

redirect_to root_url

else

@feed_items = []

render 'static_pages/home'

end

・

この状態でマイクロポストの投稿を失敗させるとfeedの表示がおかしくなる→[]を代入したため@feed_itemsがemptyの状態
(上記の件どこでなおすとか書いてないけど、14章で完成させる機能っぽいから14章でかなぁ🐫）

演習

新しく実装したマイクロポストの投稿フォームを使って、実際にマイクロポストを投稿してみましょう。Railsサーバーのログ内にあるINSERT文では、どういった内容をデータベースに送っているでしょうか? 確認してみてください。

コンソールを開き、user変数にデータベース上の最初のユーザーを代入してみましょう。その後、Micropost.where(“user_id = ?”, user.id)とuser.microposts、そしてuser.feedをそれぞれ実行してみて、実行結果がすべて同じであることを確認してみてください。ヒント: ==で比較すると結果が同じかどうか簡単に判別できます。

  SQL (0.9ms)  INSERT INTO "microposts" ("content", "user_id", "created_at", "updated_at") VALUES (?, ?, ?, ?) 
　 [["content", "てすとのとうこう"], ["user_id", 1], ["created_at", "2020-05-16 17:36:16.920347"], ["updated_at", "2020-05-16 17:36:16.920347"]]

1 2	SQL (0.9ms) INSERT INTO "microposts" ("content", "user_id", "created_at", "updated_at") VALUES (?, ?, ?, ?) 　 [["content", "てすとのとうこう"], ["user_id", 1], ["created_at", "2020-05-16 17:36:16.920347"], ["updated_at", "2020-05-16 17:36:16.920347"]]

macropostsテーブルの指定のカラムに対応した値を配列の形で送信している

2.
動作確認のみにて省略

13.3.4 マイクロポストを削除する

マイクロポストを削除する機能を実装する！！！
→ユーザー削除と同様に「delete（削除）」リンクによって作動させる
ユーザーの削除は管理者ユーザーのみが実行できるという制限だったけど
マイクロポストの削除は投稿した本人のみが削除できるようにする

viewにリンクの追加とdestroyアクションの実装

<!--CSSのidにマイクロポストのidを割り振っている　一般的に良いとされる慣習との事-->
<li id="micropost-<%= micropost.id %>">
  <!--gravatarからユーザの画像を呼び出して表示＆ユーザー詳細へリンク-->
  <%= link_to gravatar_for(micropost.user, size: 50), micropost.user %>
  <span class="user"><%= link_to micropost.user.name, micropost.user %></span>
  <span class="content"><%= micropost.content %></span>
  <span class="timestamp">idv
    <!--〇〇分前に投稿的な文字列を出してくれるヘルパーメソッド-->
    <!--Posted <%= time_ago_in_words(micropost.created_at) %> ago.-->
    <!--上記を取り入れて日本語化-->
    <%= t('.posted', time_ago: time_ago_in_words(micropost.created_at) ) %>
    <% if current_user?(micropost.user) %>
      <%= link_to t('.delete'), micropost, method: :delete,
                                       data: { confirm: t('.sure') } %>
    <% end %>
  </span>
</li>

<%= link_to gravatar_for(micropost.user, size: 50), micropost.user %>

<span class="timestamp">idv

<%= t('.posted', time_ago: time_ago_in_words(micropost.created_at) ) %>

<% if current_user?(micropost.user) %>

<%= link_to t('.delete'), micropost, method: :delete,

data: { confirm: t('.sure') } %>

<% end %>

</span>

</li>

Micropostsコントローラにdestroyアクションを定義していく

関連付けを使ってマイクロポストを見つけるようにしている点です。これにより、あるユーザーが他のユーザーのマイクロポストを削除しようとすると、自動的に失敗するようになります。

→correct_userフィルターを定義し、その中で削除したいマイクロポストがログインしているユーザーのものかを確認する

class MicropostsController < ApplicationController
  # 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用
  before_action :logged_in_user, only: [:create, :destroy]
  # 直前にcorrect_userメソッドを実行　destroyアクションにのみ適用
  before_action :correct_user,   only: :destroy
・
・
・
  def destroy
    @micropost.destroy
    flash[:success] = t('.micropost_deleted')
    # リダイレクト　（request.referrerで返される）一つ前のURL　もしくはroot_url
    redirect_to request.referrer || root_url
  end
  
  private

    def micropost_params
      # micropost属性必須　content属性のみ変更を許可
      params.require(:micropost).permit(:content)
    end
    
    def correct_user
      # @microposutに代入　current_userのmicropostsからparams[:id]を持つmicropostを取得
      @micropost = current_user.microposts.find_by(id: params[:id])
      # root_urlにredirect　もし@micropostがnilなら
      redirect_to root_url if @micropost.nil?
    end
end

class MicropostsController < ApplicationController

# 直前にlogged_in_userメソッド（ApplicationController）を実行　:create, :destroyアクションにのみ適用

before_action :logged_in_user, only: [:create, :destroy]

# 直前にcorrect_userメソッドを実行　destroyアクションにのみ適用

before_action :correct_user, only: :destroy

・

def destroy

@micropost.destroy

flash[:success] = t('.micropost_deleted')

# リダイレクト　（request.referrerで返される）一つ前のURL　もしくはroot_url

redirect_to request.referrer || root_url

end

private

def micropost_params

# micropost属性必須　content属性のみ変更を許可

params.require(:micropost).permit(:content)

end

def correct_user

# @microposutに代入　current_userのmicropostsからparams[:id]を持つmicropostを取得

@micropost = current_user.microposts.find_by(id: params[:id])

# root_urlにredirect　もし@micropostがnilなら

redirect_to root_url if @micropost.nil?

end

request.referrerメソッドとは何ぞ

このメソッドはフレンドリーフォワーディングのrequest.url変数 (10.2.3) と似ていて、一つ前のURLを返します (今回の場合、Homeページになります)13 。このため、マイクロポストがHomeページから削除された場合でもプロフィールページから削除された場合でも、request.referrerを使うことでDELETEリクエストが発行されたページに戻すことができるので、非常に便利です。

redirect_to request.referrer || root_url

1	redirect_to request.referrer \|\| root_url

また、元に戻すURLが見つからなかった場合は、||演算子を使ってroot_urlにリダイレクトされるようになっている

演習

マイクロポストを作成し、その後、作成したマイクロポストを削除してみましょう。次に、Railsサーバーのログを見てみて、DELETE文の内容を確認してみてください。

redirect_to request.referrer || root_urlの行をredirect_back(fallback_location: root_url)と置き換えてもうまく動くことを、ブラウザを使って確認してみましょう (このメソッドはRails 5から新たに導入されました)。

1.
idがエスケープされている

   (0.1ms)  begin transaction
  SQL (0.2ms)  DELETE FROM "microposts" WHERE "microposts"."id" = ?  [["id", 304]]
   (4.6ms)  commit transaction

(0.1ms) begin transaction

SQL (0.2ms) DELETE FROM "microposts" WHERE "microposts"."id" = ? [["id", 304]]

(4.6ms) commit transaction

2.
→動く

redirect_back(fallback_location: root_url)

1	redirect_back(fallback_location: root_url)

redirect_backメソッドで直前に実行したアクションへリダイレクト
引数のfallback_locationオプションで例外エラーが発生した際のリダイレクト先にroot_urlを指定している

13.3.5 フィード画面のマイクロポストをテストする

Micropostsコントローラの認可をチェックするテストと
まとめの統合テストを書いていく

テストの準備

マイクロポスト用のFixtureにそれそれ別のユーザーに紐付けられたマイクロポストを追加する
（今回は1つしか使わないけど後で他のマイクロポストも利用するとの事）

・
・
・
#  別のユーザーに所属しているマイクロポストを追加
ants:
  content: "Oh, is that what you want? Because that's how you get ants!"
  created_at: <%= 2.years.ago %>
  user: archer

zone:
  content: "Danger zone!"
  created_at: <%= 3.days.ago %>
  user: archer

tone:
  content: "I'm sorry. Your words made sense, but your sarcastic tone did not."
  created_at: <%= 10.minutes.ago %>
  user: lana

van:
  content: "Dude, this van's, like, rolling probable cause."
  created_at: <%= 4.hours.ago %>
  user: lana

・

# 別のユーザーに所属しているマイクロポストを追加

ants:

content: "Oh, is that what you want? Because that's how you get ants!"

created_at: <%= 2.years.ago %>

user: archer

zone:

content: "Danger zone!"

created_at: <%= 3.days.ago %>

user: archer

tone:

content: "I'm sorry. Your words made sense, but your sarcastic tone did not."

created_at: <%= 10.minutes.ago %>

user: lana

van:

content: "Dude, this van's, like, rolling probable cause."

created_at: <%= 4.hours.ago %>

user: lana

Micropostsコントローラの認可をチェックするテスト

自分以外のユーザーのマイクロポストを削除しようとすると適切にリダイレクトされるテスト

・
・
・
  test "should redirect destroy for wrong micropost" do
    # michaelでログイン
    log_in_as(users(:michael))
    # microposutに代入　（fixturesのマイクロポスト）antsを代入
    micropost = microposts(:ants)
    # ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない
    assert_no_difference 'Micropost.count' do
      # micropost_path(micropost)にdeleteのリクエスト
      delete micropost_path(micropost)
    end
    # root_urlにリダイレクト
    assert_redirected_to root_url
  end
end

・

test "should redirect destroy for wrong micropost" do

# michaelでログイン

log_in_as(users(:michael))

# microposutに代入　（fixturesのマイクロポスト）antsを代入

micropost = microposts(:ants)

# ブロックで渡されたものを呼び出す前後でMicropost.countに違いがない

assert_no_difference 'Micropost.count' do

# micropost_path(micropost)にdeleteのリクエスト

delete micropost_path(micropost)

end

# root_urlにリダイレクト

assert_redirected_to root_url

end

次に統合テスト

ログイン
マイクロポストのページネーションの確認
無効なマイクロポストを投稿
マイクロポストの削除
他のユーザーのマイクロポストには [delete] リンクが表示されない

といった手順で書いていく
→テストを生成

$ rails generate integration_test microposts_interface
Running via Spring preloader in process 10139
      invoke  test_unit
      create    test/integration/microposts_interface_test.rb

$ rails generate integration_test microposts_interface

Running via Spring preloader in process 10139

invoke test_unit

create test/integration/microposts_interface_test.rb

テストを実装

require 'test_helper'

class MicropostsInterfaceTest < ActionDispatch::IntegrationTest

  def setup
    # @userにmichaelを代入
    @user = users(:michael)
  end

  test "micropost interface" do
    # @userでログイン
    log_in_as(@user)
    # root_pathにgetのリクエスト
    get root_path
    # 特定のHTMLタグが存在する→ class = "pagination"を持つdiv
    assert_select 'div.pagination'
    # 無効な送信
    # ブロックで渡されたものを呼び出す前後でMicropost.countに違いがな
    assert_no_difference 'Micropost.count' do
      # microposts_pathにpostのリクエスト　→　micropost: { content: "" }（無効なデータ）
      post microposts_path, params: { micropost: { content: "" } }
    end
    # 特定のHTMLタグが存在する→　id = "error_explanation"を持つdiv
    assert_select 'div#error_explanation'
    # 有効な送信
    # contentに代入　→　"This micropost really ties the room together"
    content = "This micropost really ties the room together"
    # ブロックで渡されたものを呼び出す前後でMicropost.countが+1
    assert_difference 'Micropost.count', 1 do
      # microposts_pathにpostのリクエスト　→　micropost: { content: content }（有効なデータ）
      post microposts_path, params: { micropost: { content: content } }
    end
    # root_urlにリダイレクト
    assert_redirected_to root_url
    #　指定されたリダイレクト先に移動
    follow_redirect!
    # 表示されたページのHTML本文すべての中にcontentが含まれている
    assert_match content, response.body
    # 投稿を削除する
    # 特定のHTMLタグが存在する→　text: '削除'を持つa
    assert_select 'a', text: '削除'
    # first_micropostに代入　@user.micropostsの1ページ目の1番目のマイクロポスト
    first_micropost = @user.microposts.paginate(page: 1).first
    # ブロックで渡されたものを呼び出す前後でMicropost.countが-1
    assert_difference 'Micropost.count', -1 do
      # micropost_path(first_micropost)にdeleteのリクエスト
      delete micropost_path(first_micropost)
    end
    # 違うユーザーのプロフィールにアクセス (削除リンクがないことを確認)
    # user_path(users(:archer))にgetのリクエスト
    get user_path(users(:archer))
    # 特定のHTMLタグが存在する→　text: '削除'を持つaが0個
    assert_select 'a', text: '削除', count: 0
  end
end

require 'test_helper'

class MicropostsInterfaceTest < ActionDispatch::IntegrationTest

def setup

# @userにmichaelを代入

@user = users(:michael)

end

test "micropost interface" do

# @userでログイン

log_in_as(@user)

# root_pathにgetのリクエスト

get root_path

# 特定のHTMLタグが存在する→ class = "pagination"を持つdiv

assert_select 'div.pagination'

# 無効な送信

# ブロックで渡されたものを呼び出す前後でMicropost.countに違いがな

assert_no_difference 'Micropost.count' do

# microposts_pathにpostのリクエスト　→　micropost: { content: "" }（無効なデータ）

post microposts_path, params: { micropost: { content: "" } }

end

# 特定のHTMLタグが存在する→　id = "error_explanation"を持つdiv

assert_select 'div#error_explanation'

# 有効な送信

# contentに代入　→　"This micropost really ties the room together"

content = "This micropost really ties the room together"

# ブロックで渡されたものを呼び出す前後でMicropost.countが+1

assert_difference 'Micropost.count', 1 do

# microposts_pathにpostのリクエスト　→　micropost: { content: content }（有効なデータ）

post microposts_path, params: { micropost: { content: content } }

end

# root_urlにリダイレクト

assert_redirected_to root_url

#　指定されたリダイレクト先に移動

follow_redirect!

# 表示されたページのHTML本文すべての中にcontentが含まれている

assert_match content, response.body

# 投稿を削除する

# 特定のHTMLタグが存在する→　text: '削除'を持つa

assert_select 'a', text: '削除'

# first_micropostに代入　@user.micropostsの1ページ目の1番目のマイクロポスト

first_micropost = @user.microposts.paginate(page: 1).first

# ブロックで渡されたものを呼び出す前後でMicropost.countが-1

assert_difference 'Micropost.count', -1 do

# micropost_path(first_micropost)にdeleteのリクエスト

delete micropost_path(first_micropost)

end

# 違うユーザーのプロフィールにアクセス (削除リンクがないことを確認)

# user_path(users(:archer))にgetのリクエスト

get user_path(users(:archer))

# 特定のHTMLタグが存在する→　text: '削除'を持つaが0個

assert_select 'a', text: '削除', count: 0

end

機能は実装済なのでテストはGREEN！

演習

リスト 13.55で示した４つのコメント (「無効な送信」など) のそれぞれに対して、テストが正しく動いているか確認してみましょう。具体的には、対応するアプリケーション側のコードをコメントアウトし、テストが redになることを確認し、元に戻すと greenになることを確認してみましょう。

サイドバーにあるマイクロポストの合計投稿数をテストしてみましょう。このとき、単数形 (micropost) と複数形 (microposts) が正しく表示されているかどうかもテストしてください。ヒント: リスト 13.57を参考にしてみてください。

動作確認のみにて省略
I18nで言語をjaにしている影響で単数形と複数形は関係ない感じになっちゃってるけどこう↓

・
・
・
    test "micropost sidebar count" do
    # @userでログイン
    log_in_as(@user)
    # root_pathにgetのリクエスト
    get root_path
    # 表示されたページのHTML本文すべての中に　#{@user.microposts.count}個の投稿があります　が含まれている
    assert_match "#{@user.microposts.count}個の投稿があります", response.body
    # other_userに代入 → users(:malory)　（まだマイクロポストを投稿していないユーザー）
    other_user = users(:malory)
    # other_user出pログイン
    log_in_as(other_user)
    # root_pathにgetのリクエスト
    get root_path
    # 表示されたページのHTML本文すべての中に　0個の投稿があります　が含まれている
    assert_match "0個の投稿があります", response.body
    # other_userに紐づいたmicropostを作成（content属性に値"A micropost"をセット）
    other_user.microposts.create!(content: "A micropost")
    # root_pathにgetのリクエスト
    get root_path
    # 表示されたページのHTML本文すべての中に　1個の投稿があります　が含まれている
    # 日本語にしていなければFILL_INは"1 micropost"（micropostsではなく単数形にする）
    assert_match "1個の投稿があります", response.body
  end
end

・

test "micropost sidebar count" do

# @userでログイン

log_in_as(@user)

# root_pathにgetのリクエスト

get root_path

# 表示されたページのHTML本文すべての中に　#{@user.microposts.count}個の投稿があります　が含まれている

assert_match "#{@user.microposts.count}個の投稿があります", response.body

# other_userに代入 → users(:malory)　（まだマイクロポストを投稿していないユーザー）

other_user = users(:malory)

# other_user出pログイン

log_in_as(other_user)

# root_pathにgetのリクエスト

get root_path

# 表示されたページのHTML本文すべての中に　0個の投稿があります　が含まれている

assert_match "0個の投稿があります", response.body

# other_userに紐づいたmicropostを作成（content属性に値"A micropost"をセット）

other_user.microposts.create!(content: "A micropost")

# root_pathにgetのリクエスト

get root_path

# 表示されたページのHTML本文すべての中に　1個の投稿があります　が含まれている

# 日本語にしていなければFILL_INは"1 micropost"（micropostsではなく単数形にする）

assert_match "1個の投稿があります", response.body

end

まとめとか感想

長かったねええええええ！
本文に書いたけど、フィードの挙動がちゃんとしてないけど追々するのかな？
どこでするかわかったら追記するようにしたいと思います（曖昧

らくだ🐫にもできるRailsチュートリアルとは

「ド」が付く素人のらくだ🐫が勉強するRailsチュートリアルの学習記録です。
自分用に記録していますが、お役に立つことがあれば幸いです。

調べたとはいえらくだ🐫なりの解釈や説明が含まれます。間違っている部分もあるかと思います。そんな所は教えて頂けますと幸いなのですが、このブログにはコメント機能がありません💧お手数おかけしますがTwitterなどでご連絡いただければ幸いです