link_toを使ってviewからコントローラーに値を渡したいのですが、うまくいかないので説明お願いします。

Question

###解決したい事
一覧ページから詳細ページに移動したいのですが、うまくいきません。

###ソースコード
users/show.html.erb

<div class="container">
  <h2 style="text-align: center;"><%= @user.name %>の投稿一覧</h2>
  <div class="center-block">
    <%= link_to "写真詳細ページ", photo_path(@photo.id) %>
   <% @photos.each do |photo| %>
   <div><a href="<%= photo_path %>">写真の詳細ページ</a></div>
     <div style="border: 2px solid;">
       <%= image_tag photo.image.url if photo.image? %>
       <%= photo.content %>
     </div>
   <% end %>
  </div>
</div>

photos/show.html.erb

<p id="notice"><%= notice %></p>
<div class="row">
  <div class="offset-md-3 col-md-5">
    <p>
      <%= image_tag @photo.image.url if @photo.image? %>
    </p>
    <p>
      <strong>クイズの問題:</strong>
      <%= @photo.content %>
    </p>
      <%= link_to "回答する", new_photo_answer_path(@photo) %>
      
      <ul class="list-unstyled">
        <% @answers.each do |answer| %>
          <li class="media">
            <div class="media-body">
              <div>
                <%= answer.user.name %>
              </div>
              <div>
                <%= answer.content %>
              </div>
            </div>
          </li>
        <% end %>
      </ul>
    </div>
</div>

<%= link_to '戻る', photos_path %> |
<%= link_to '編集', edit_photo_path(@photo) %> 

users_controller.rb

class UsersController < ApplicationController
  before_action :require_user_logged_in, only: [:show]
  def index
  end

  def show
    @user = User.find(params[:id])
     @photos = @user.photos.order(id: :desc).page(params[:page])
  end

  def new
    @user = User.new
  end

  def create
    @user = User.new(user_params)
    
    if @user.save
      flash[:success] = 'ユーザを登録しました。'
      redirect_to @user
    else
      flash.now[:danger] = 'ユーザの登録に失敗しました。'
      render :new
    end
  end
  
  private
  
  def user_params
    params.require(:user).permit(:name, :email, :password, :password_confirmation)
  end
end

photos_controller.rb

class PhotosController < ApplicationController
  before_action :set_photo, only: %i[ show edit update destroy ]

  # GET /photos or /photos.json
  def index
    @photos = Photo.all
  end

  # GET /photos/1 or /photos/1.json
  def show
    @photo = Photo.find(params[:id])
    @answers = @photo.answers
    @answer = current_user.answers.new
  end

  # GET /photos/new
  def new
    @photo = Photo.new
  end

  # GET /photos/1/edit
  def edit
  end

  # POST /photos or /photos.json
  def create
    @photo = current_user.photos.new(photo_params)
    
    if @photo.save
      flash[:success] = '投稿に成功しました。'
      redirect_to photos_path
    else
      flash.now[:danger] = "投稿に失敗しました。"
      render :new
    end
  end

  # PATCH/PUT /photos/1 or /photos/1.json
  def update
    respond_to do |format|
      if @photo.update(photo_params)
        format.html { redirect_to @photo, notice: "Photo was successfully updated." }
        format.json { render :show, status: :ok, location: @photo }
      else
        format.html { render :edit, status: :unprocessable_entity }
        format.json { render json: @photo.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /photos/1 or /photos/1.json
  def destroy
    @photo.destroy
    respond_to do |format|
      format.html { redirect_to photos_url, notice: "Photo was successfully destroyed." }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
     def set_photo
      @photo = Photo.find(params[:id])
     end

    # Only allow a list of trusted parameters through.
    def photo_params
      params.require(:photo).permit(:image, :content)
    end
end

routes.rb

Rails.application.routes.draw do
  
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  root to: 'toppages#index'
  
  get 'login', to: 'sessions#new'
  post 'login', to: 'sessions#create'
  delete 'logout', to: 'sessions#destroy'
  
  get 'signup', to: 'users#new'
  resources :photos do
    resources :answers, only: [:create, :new]
  end
  resources :users, only: [:index, :show, :create]
end

users/show/html.erbからphotos/show.html.erbに移動したいのですが、エラーが出てしまいうまくいきません。

######エラーメッセージ

エラーメッセージ

ActionView::Template::Error (undefined method `id' for nil:NilClass):
    1: <div class="container">
    2:   <h2 style="text-align: center;"><%= @user.name %>の投稿一覧</h2>
    3:   <div class="center-block">
    4:     <%= link_to "写真詳細ページ", photo_path(@photo.id) %>
    5:    <% @photos.each do |photo| %>
    6:    <div><a href="<%= photo_path %>">写真の詳細ページ</a></div>
    7:      <div style="border: 2px solid;">
  
app/views/users/show.html.erb:4:in `_app_views_users_show_html_erb___4171669809949038165_46961492423160'

###補足
原因はおそらくidが受け渡されてないからだと思いますが、どうやってもうまくいきません。
足りないコードがあれば載せます。
ご教授お願いします。

Answer 1

重要なところだけ抜き出すと

def show
@user = User.find(params[:id])
@photos = @user.photos.order(id: :desc).page(params[:page])
end

および

<%= link_to "写真詳細ページ", photo_path(@photo.id) %>

<% @photos.each do |photo| %>

<div><a href="<%= photo_path %>">写真の詳細ページ</a></div>

見比べるとわかると思いますが@photoはコントローラー内で宣言されていないので使えません

   <% @photos.each do |photo| %>
     <div><%= link_to "写真詳細ページ", photo_path(photo.id) %></div>
     <div style="border: 2px solid;">
       <%= image_tag photo.image.url if photo.image? %>
       <%= photo.content %>
     </div>
   <% end %>

Comments

[gen515]

コントローラーに@photo = Photo.find(params[:id])と宣言したのですが次はこのようなエラーが出ました。Couldn't find Photo with 'id'=3

[asm]

そりゃ、UsersController#showに渡されるidはuserのものなのでPhotoから探してもみつからないでしょう

Answer 2

@photoとは何でしょうか。

コントローラー内でこの値に何も代入していないので、値はnilです。

Comments

[gen515]

コントローラーに@photo = Photo.find(params[:id])と宣言したのですが次はこのようなエラーが出ました。Couldn't find Photo with 'id'=3

[maisumakun]

エラーメッセージのとおりです。UsersControllerに渡されるidはuserのidで、それでPhotoを引いても正しい結果は得られません。