Rails 自己紹介ページに写真投稿＆いいね機能

Question

前提・実現したいこと

Railsで自己紹介ページに写真投稿＆いいねができる機能を付け、共有できるサービスを作成しておりました。ユーザ詳細ページの一部分にSKILL(自分の得意としていることや勉強していることを入力するスペース。titleとdescriptionを用意)部分を設け、ユーザ詳細変更画面で編集できるようにしていたのですが、show.html.erbにてskill部分を実装後、下記のエラー画面が表示されるのですが、原因をご教示お願いできないでしょうか？

###①ログイン後、ActiveRecord::StatementInvalid in UsersController#showが表示される。
###②ユーザ詳細画面にてNoMethodError in Users#showが表示される。

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

ActiveRecord::StatementInvalid in UsersController#show
Mysql2::Error: Unknown column 'users.user_id' in 'where clause': SELECT `users`.* FROM `users` WHERE `users`.`user_id` = 1 LIMIT 1
  def show
    @user = User.find(params[:id])
    @skill = @user.skills.page(params[:page])
  end

  def new

NoMethodError in Users#show
undefined method `title' for #<ActiveRecord::AssociationRelation []>
    <div class="center jumbotron">
      <div class="text-center">
        <%= @skill.title %>
      </div>
    </div>
    <div class="center jumbotron">

該当のソースコード

routes.rb

Rails.application.routes.draw do
  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 :users, only: [:index, :show, :new, :create, :edit] 
  
  put 'users/:id/edit', to: 'users#edit'
  
  put '/users/:id', to: 'users#update'
  get 'users/:id/edit', to: 'users#edit'
  
  resources :posts
  resources :skills, only: [:create, :edit, :destroy]
end
  

users_controller.rb

class UsersController < ApplicationController
  before_action :require_user_logged_in, only: [:index, :show]
  before_action :correct_user, only: [:edit]
  
  def index
    @users = User.order(id: :desc).page(params[:page]).per(50)
  end

  def show
    @user = User.find(params[:id])
    @skills = @user.skills.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

  def edit
    @user = User.find(params[:id])
  end
  
  def update
    @user = User.find(params[:id])
    
    if @user.update(user_params)
      flash[:success] = 'プロフィールを変更しました。'
      redirect_to @user
    else
      flash.now[:danger] = 'プロフィールが変更できませんでした。'
      render :edit
    end
  end
  
  private

  def user_params
    params.require(:user).permit(:name, :email, :password, :password_confirmation, :profession, :first_name, :last_name, :profile,)
  end
  
  def correct_user
  end
end

skills_controller.rb

class SkillsController < ApplicationController
  before_action :require_user_logged_in
  before_action :correct_user, only: [:destroy]
  
  def create
    @skills = current_user.skills.build(skill_params)
    if @skill.save
      flash[:success] = "スキルを登録しました。"
      redirect_to root_url
    else
      @skills = current_user.feed_skills.order(id: :desc).page(params[:page])
      flash.now[:danger] = "スキルの登録に失敗しました。"
      render "users/edit"
    end
  end

  def edit
  end

  def destroy
    @skill.destroy
    flash[:success] = "スキルを削除しました。"
    redirect_back(fallback_location: root_path)
  end
end

users/show.html.erb

<div class="row">
  <aside class="col-sm-12">
    <div class="text-left">
      <%= link_to 'プロフィール変更', edit_user_path(@user) %>
    </div>
  </aside>
  
  <aside class="col-sm-12">
    <table class="table table-bordered">
      <tr>
        <th>職業</th>
        <td><%= @user.profession %></td>
      </tr>
      <tr>
        <th>名前</th>
        <td><%= @user.last_name %> <%= @user.first_name %></td>
      </tr>
      <tr>
        <th>自己紹介</th>
        <td><%= @user.profile %></td>
      </tr>
    </table>
  </aside>
  
  <aside class="col-sm-12">
    <div class="center jumbotron">
      <div class="text-center">
        <h2>SKILL</h2>
      </div>
    </div>
  </aside>
  <aside class="col-sm-6">
    <div class="center jumbotron">
      <div class="text-center">
        <img class="rounded img-fluid" src="<%= gravatar_url(@user, { size: 250 }) %>" alt="">
      </div>
    </div>
  </aside>
  <aside class="col-sm-6">
    <div class="center jumbotron">
      <div class="text-center">
        <%= @skill.title %>
      </div>
    </div>
    <div class="center jumbotron">
      <div class="text-center">
        <%= @skill.description %>
      </div>
    </div>
  </aside>
</div>

users/edit.html.erb

<div class="text-center">
  <h1>プロフィール変更画面</h1>
</div>

<div class="row">
  <div class="col-sm-6 offset-sm-3">

    <%= form_with(model: @user, local: true, method: :put) do |f| %>
      <%= render 'layouts/error_messages', model: f.object %>

      <div class="form-group">
        <%= f.label :profession, 'Profession' %>
        <%= f.text_field :profession, class: 'form-control' %>
      </div>

      <div class="form-group">
        <%= f.label :first_name, 'First Name' %>
        <%= f.text_field :first_name, class: 'form-control' %>
      </div>
      
      <div class="form-group">
        <%= f.label :last_name, 'Last Name' %>
        <%= f.text_field :last_name, class: 'form-control' %>
      </div>
      
      <div class="form-group">
        <%= f.label :profile, 'Profile' %>
        <%= f.text_field :profile, class: 'form-control' %>
      </div>
      
  <div class="text-left">
    <h3>Security</h3>
  </div>
      
      <div class="form-group">
        <%= f.label :email, 'Email' %>
        <%= f.email_field :email, class: 'form-control' %>
      </div>
      <div class="form-group">
        <%= f.label :password, 'Password' %>
        <%= f.password_field :password, class: 'form-control' %>
      </div>

      <%= f.submit '更新', class: 'btn btn-primary btn-block' %>

  <div class="text-left">
    <h3>SKILL</h3>
  </div>
  
      <%= render 'skills/edit', skills: @skill %>
    <% end %>
  </div>
</div>

skills/_edit.html.erb

<%= form_with(model: @user, local: true, method: :put) do |f| %>
  <%= render 'layouts/error_messages', model: f.object %>

  <div class="form-group">
    <%= f.label :title, 'Skill Title' %>
    <%= f.text_field :title, class: 'form-control' %>
  </div>

  <div class="form-group">
    <%= f.label :description, 'Skill Descriotion' %>
    <%= f.text_field :description, class: 'form-control' %>
  </div>
  
  <%= f.submit '更新', class: 'btn btn-primary btn-block' %>
<% end %>

user.rb

class User < ApplicationRecord
  validates :name, presence: true, length: { maximum: 50 }
  validates :email, presence: true, length: { maximum: 255 },
                  format: { with: /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i },
                  uniqueness: { case_sensitive: false }
  validates :password, allow_blank: true, length: { maximum: 20 }
  validates :profession, allow_blank: true, length: { maximum: 20 }
  validates :first_name, allow_blank: true, length: { maximum: 10 }
  validates :last_name, allow_blank: true, length: { maximum: 10 }
  validates :profile, allow_blank: true, length: { maximum: 200 }
  has_secure_password
  
  has_many :posts
  has_many :skills
end

skill.rb

class Skill < ApplicationRecord
  belongs_to :user
  
  validates :title, length: { maximum: 10 }
  validates :description, length: { maximum: 150 }
end

20201026001900_create_skills

class CreateSkills < ActiveRecord::Migration[5.2]
  def change
    create_table :skills do |t|
      t.string :title
      t.text :description
      t.references :user, foreign_key: true

      t.timestamps
    end
  end
end

schema.rbの一部分

ActiveRecord::Schema.define(version: 2020_10_31_081638) do

  create_table "posts", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "image"
    t.bigint "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_posts_on_user_id"
  end

  create_table "skills", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "title"
    t.text "description"
    t.bigint "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_skills_on_user_id"
  end

  create_table "users", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.string "name"
    t.string "email"
    t.string "password_digest"
    t.string "profession"
    t.string "first_name"
    t.string "last_name"
    t.text "profile"
    t.timestamp "icon"
    t.string "new_email"
    t.string "new_password"
    t.string "image"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  add_foreign_key "posts", "users"
  add_foreign_key "skills", "users"
end

試したこと

users#showに何か記述が足りないのかと思いましたが、今のところ？？？です。

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

DB:MySQL
FW:Ruby on Rails5.2.2
コーディング環境:AWS Cloud9
バージョン管理システム:Git/GitHub

Answer 1

原因をご教示お願いできないでしょうか？

とのことなので原因を書いていきます。

原因は、@skillが定義されていないと思われます。

動いているのはこのアクションだと思うので、たしかに@skillが定義されていないことがわかります。

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

なので、解決するためにはこのアクションないで@skillを定義してあげてください。

Comments

[Tarzan3154]

ご回答ありがとうございます。
users#showにて、上記のとおり@skillsを定義してみましたが、上記のエラー表示が出てしまいます。
@skillsの定義はどのようにするのが正しいでしょうか？

[Tarzan3154]

併せて、skills_controller.rbも追記しています。

[hatsu]

もともとの質問を消してしまうと、回答との整合性が取れなくなるので、追記の形をとっていただくといいと思います。
もともとは@skillsではなく、@skillが定義されていなかったことが原因だったかと思います。
なので@skillを定義してあげてください。
どんなparamsから@skillを定義するのか、リレーションから引いてくるのかはわからないですが、@skill = Skill.newなのか@skill = Skill.find(〇〇)なのか？こんな感じです。

で、いま出ているMysql2::Error: Unknown column のエラーはUserとSkillのリレーションがうまくいっていなそうなのでまた別問題が起きていそうです。

[Tarzan3154]

user.rbおよびskill.rbを編集(has_oneとしていたところを一対多の関係に修正)したところ、②のエラー画面が表示されるようになりました。
UserとSkillのリレーションについては、私はおかしな点はないように思いましたが、現時点では原因は？？？です。
skillについてはマイグレーションファイルも追記しておりますが、何か修正点はありますでしょうか？

[hatsu]

②ということ「ユーザ詳細画面にてNoMethodError in Users#showが表示される。」のことですよね？
<div class="text-center">
<%= @skill.title %>
</div>
としていますが、Controllerでは@skillsの定義をしており@skillの定義はされていません。そのためにNoMethodErrorが起きていると思われます。

あと正しくMigrateされているかがわからないのでdb/schema.rbとかが載っているといいかもです。
rails sで起動しっぱなしのときに反映されないコードの変更もあるので、たまに再起動するといいと思います。initializerの変更とかなので今回は関係ないかもですが。

[Tarzan3154]

すみません、Controllerでは@skillの定義をするよう修正しておりますが、NoMethodErrorが表示されるのです。(ソースコードの変更ができていませんでした)
それから、db/schema.rbの一部分についても追記してますので、再度ご確認をお願い致します。

[hatsu]

@skillがたぶん空なんだと思います。Userに紐付いているSkillがないのかな、と。
いったん<%= @skill&.title %>とかにすればエラーはなくなります。
@skill = @user.skills.page(params[:page])とするなら@skillsにして、View側でとかでもいいかもです。
<% @skills.each do |skill| %>
<%= skill.title %>
<% end %>