rubyのTemplate::Error(userが未定義？)を解消したい

Question

前提・実現したいこと

Bookers2という閲覧、投稿サイトにフォロー・フォロワー機能を追加したいのですが、
NameError in Users#show　が発生してしましました。

定義を確認してみたのですが間違っている部分が見つけられなかったので、教えていただきたいです。
まだ勉強を始めたばかりの初心者ですので、余裕があれば解説していただけると助かります。

また、初投稿なので質問できていない部分があれば教えてください。
よろしくお願いいたします。

発生している問題・エラーメッセージ

エラーメッセージ

NameError in Users#show

ndefined local variable or method `users' for #<#<Class:0x00007f5df5903e90>:0x00007f5df5901d70>
Did you mean?  user
               @user

というエラーメッセージが出ています。




### 該当のソースコード
```ruby
エラーが発生していたファイル
```ruby
/app/views/users/_info.html.erb（ファイル）

<table class='table'>
  <thead>
    <tr>
      <th>image</th>
      <th>name</th>
      <th colspan="3"></th>
    </tr>
  </thead>
  <tbody>
    <% users.each do |user| %>
      <th>
        <td><%= attachment_image_tag(user, :profile_image, :fill, 100, 100, fallback: "no_image.jpg") %></td>
	      <td><%= user.name %></td>
        <td>フォロー数: <%= user.followings.count %></td>
        <td>フォロワー数: <%= user.followers.count %></td>
        <td>
          <% if current_user != user %>
            <% if current_user.following?(user) %>
              <%= link_to "フォロー外す", user_relationships_path(user.id), method: :delete %>
            <% else %>
              <%= link_to "フォローする", user_relationships_path(user.id), method: :post %>
            <% end %>
          <% end %>
        </td>
        <td><%= link_to 'Show', user_path(user), class: "user_#{user.id}" %></td>
      </tr>
    <% end %>
  </tbody>
</table>

  

usersのcontrollerファイル

users_controller(ファイル)

class UsersController < ApplicationController
  before_action :authenticate_user!, except: [:home, :about]
  before_action :ensure_correct_user, only: [:edit, :update]

  def show
    @user = User.find(params[:id])
    @books = @user.books
    @book = Book.new
  end

  def index
    @users = User.all
    @book = Book.new
  end

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

  def update
    if @user.update(user_params)
      redirect_to user_path(@user), notice: "You have updated user successfully."
    else
      render "edit"
    end
  end

  private
  def user_params
    params.require(:user).permit(:name, :introduction, :profile_image)
  end

  def ensure_correct_user
    @user = User.find(params[:id])
    unless @user == current_user
      redirect_to user_path(current_user)
    end
  end
end

routesのファイル

Rails.application.routes.draw do
  get 'followings' => 'relationships#followings', as: 'followings'
  get 'followers' => 'relationships#followers', as: 'followers'
  devise_for :users


  root to: 'homes#top'
  get '/home/about' => 'homes#about'

  resources :books, only: [:index, :show, :edit, :create, :destroy, :update]
  resources :users, only: [:index, :show, :edit, :update] do
    resource :relationships, only: [:create, :destroy]
    get 'followings' => 'relationships#followings', as: 'followings'
    get 'followers' => 'relationships#followers', as: 'followers'
  end

end

今回、フォロー・フォロワー機能を追加するにあたり、作成したファイル

models/relationship.rb（ファイル）

class Relationship < ApplicationRecord
  belongs_to :follower, class_name: "User"
  belongs_to :followed, class_name: "User"
end

今回作成したファイル

controllers/relationships_controller.rb（ファイル）

class RelationshipsController < ApplicationController
  def create
    current_user.follow(params[:user_id])
    redirect_to request.referer
  end

  def destroy
    current_user.unfollow(params[:user_id])
    redirect_to request.referer
  end

  def followings
    user = User.find(params[:user_id])
    @users = user.followings
  end

  def followers
    user = User.find(params[:user_id])
    @users = user.followers
  end
end

今回、修正・追加したファイル

user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :books
  has_many :book_comments, dependent: :destroy

  has_many :reverse_of_relationships, class_name: "Relationship", foreign_key: "followed_id", dependent: :destroy
  has_many :relationships, class_name: "Relationship", foreign_key: "follower_id", dependent: :destroy
  has_many :followers, through: :reverse_of_relationships, source: :follower
  has_many :followings, through: :relationships, source: :followed

  attachment :profile_image, destroy: false

  validates :name, uniqueness: true, length: {maximum: 20, minimum: 2}
  validates :introduction, length: {maximum: 50}

  def follow(user_id)
    relationships.create(followed_id: user_id)
  end

  def unfollow(user_id)
    relationships.find_by(followed_id: user_id).destroy
  end

  def following?(user)
    followings.include?(user)
  end

end

今回ほぼ触ってませんが、エラー画面に出ていたので念のため。

view/users/show.html

<div class='container px-5 px-sm-0'>
  <div class='row'>
    <div class='col-md-3'>
      <h2>User info</h2>
      <%= render 'info', user: @user %>
      <h2 class="mt-3">New book</h2>
      <%= render 'books/form', book: @book %>
    </div>
    <div class='col-md-8 offset-md-1'>
      <h2>Books</h2>
      <%= render 'books/index',books: @books %>
    </div>
  </div>
</div>

コード

試したこと

Did you mean? user
@user
という文字があったため@userにもしてみましたが変わりませんでした。
また、定義を一通り確認してみましたが、間違っている部分は見つかりませんでした。
each文も間違っている箇所はみつけられませんでした。

補足情報（FW/ツールのバージョンなど）

ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]
Rails 5.2.5
gem 3.0.8

Answer 1

エラーメッセージを見ると、users_controller の show メソッドが呼ばれていて、以下の処理で取得した @変数名 が、表示用の show.html.erb に渡されます。

  def show
    @user = User.find(params[:id])
    @books = @user.books
    @book = Book.new
  end

~~show.html.erb のソースコードを見ると、

<%= render 'books/form', book: @book %>　や　<%= render 'books/index',books: @books %>

といったファイルがさらに差し込まれているので、最終的に show.html.erb から、_info.html.erb が呼ばれているようですね。~~

20210911 修正

show.html.erb の上のほうで、 <%= render 'info', user: @user %> とあるので、ここで呼び出していましたね。

この_info.html.erb に期待しているローカル変数 (users) が渡ってきていないので、エラーになっているようです。

<%= render 'info', users: @user.followers %> とか、
<%= render 'info', users: @user.following %> とかにするといかがでしょう？

あるいは、Users#show のところで、@books や @book を取り出しているように、先にフォロー関係にあるレコードを取り出して @users とか @followers として、_info.html.erb に渡してあげるとか。

<%= render 'info', users: @followers %>

失礼いたしました。

Comments

[TAKU.1]

新しくrelationships_controllerで定義をした方がよろしいのでしょうか。それともusers_controller で定義し直す方がよろしいのでしょうか。
show.html.erbへの影響が出ないようにするにはどのように定義をすればいいのかわからないので教えていただけると助かります。

[suama]

こんにちは。
失礼しました、見落としていました！

上の方に、<%= render 'info', user: @user %> がありますね！
ここが問題でしたね。

users/_info.html.erb の、ローカル変数として users が必要ですが、呼び出し側の show.html.erb では、<%= render 'info', user: @user %> としているため、user というローカル変数、しかも単数でしか渡っていません。

単数のモデルなので、foreach といったループ処理ができないので、エラーになっていると思います。

users/_info.html.erb の、ローカル変数としての users は、なにかしら コレクション（複数のUserのレコード）をご想定かと思います。

Users#show で特定のユーザーを @user で取り出しつつ、その @user のフォロワー（複数）を users/_info.html.erb で表示するという内容でしょうか。

であれば、

<%= render 'info', user: @user %>

ではなくて、

<%= render 'info', users: @user.followers %>

とか、

<%= render 'info', users: @user.following %>

とかでしょうか。
いかがでしょう。

[TAKU.1]

users_controllerに@users = User.allを追加してusers_show.htmlにusers: @usersを追加して解決できました。返事が遅くなりましたが、ありがとうございます！