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

前提・実現したいこと

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

###①SKILLを入力(編集)しても、ユーザ詳細画面で何も表示されない。
###②ユーザ詳細画面の「プロフィール変更」が、ログイン中以外のユーザでも表示されてしまう。

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

①②ともエラー画面等は現時点では表示されず、①についてはプロフィール部分は表示されるが、SKILL部分は入力しても何も表示されない。(ちなみにユーザ詳細変更画面で、プロフィール部分・SKILL部分の「更新」ボタンが2つあるが、どちらを押してもプロフィール部分のみしか更新されない)

③

ActiveRecord::RecordNotFound in SkillsController#new
Couldn't find Skill without an ID
  def new
    @skill = Skill.find(params[:id])
  end
  
  def create

該当のソースコード

Ruby
1routes.rb
2
3Rails.application.routes.draw do
4  root to: "tasks#index"
5
6  get 'login', to: 'sessions#new'
7  post 'login', to: 'sessions#create'
8  delete 'logout', to: 'sessions#destroy'
9
10  get 'signup', to: 'users#new'
11  resources :users, only: [:new, :create]
12
13  resources :tasks
14end

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
    @user.skills.build
  end

  def create
    @user = User.new(user_params)
    user.save

    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: [:edit, :show, :update, :destroy]

  def new
    @skill = Skill.find(params[:id])
  end

  def create
    @skills = current_user.skills.build(skill_params)
    if @skill.save
      flash[:success] = "Skill を登録しました。"
      redirect_to root_url
    else
      @skills = current_user.feed_skills.order(id: :desc).page(params[:page])
      flash.now[:danger] = "Skill の登録に失敗しました。"
      render "users/edit"
    end
  end

  def edit
    @skill = Skill.find(params[:id])
  end

  def update
    @skill = Skill.find(params[:id])

    if @skill.update(skill_params)
      flash[:success] = 'Skill は正常に更新されました'
      redirect_to @skill
    else
      flash.now[:danger] = 'Skill は更新されませんでした'
      render :edit
    end
  end

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

  private

  # Strong Parameter
  def skill_params
    params.require(:skill).permit(:title, :description)
  end

  def correct_user
    @skill = current_user.skills.find_by(id: params[:id])
    unless @skill
      redirect_to root_url
    end
  end
end

show.html.erb

<div class="row">
  <aside class="col-sm-12">
    <div class="text-left">
      <%= link_to 'プロフィール変更', edit_user_path(current_user.id) %>
    </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">
    <%= link_to '新規スキル投稿', new_skill_path(current_user.id) %>
    <%= render "skills/skills",skills:@skills %>
  </aside>
</div>

skills/new.html.erb

<%= form_with(model: @skill, local: true) 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 %>

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>

skill/_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
  accepts_nested_attributes_for :skills, allow_destroy: true
  has_many :likes
  has_many :like_posts, through: :likes, source: :post
  
  def like(post)
    self.likes.find_or_create_by(post_id: post.id)
  end
  
  def unlike(post)
    like = self.likes.find_by(post_id: post.id)
    like.destroy if like
  end
  
  def likes?(post)
    self.like_posts.include?(post)
  end
end

skill.rb

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

試したこと

①については、skill/edit.html.erbのformの部分、@userを@skillに修正したりしましたが、NoMethodError in Users#editが表示されてしまい、うまくいきません。
②については、show.html.erbの<%= linkto 'プロフィール変更'～の部分に<%= current_user == user %>～<% end %>を設定してみても、SyntaxError in UsersController#showが表示されてしまい、???です。

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

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

行動規範の内容に同意します

回答1件

パット見２つ問題があります。

formが2つあります。一回のsubmitでは一つのformしか情報が送られません。

一つにまとめてください。

def user_params に追加したfieldが載っていないので無視されます。

投稿2020/12/30 05:23

winterboum

総合スコア23347

Tarzan3154

2020/12/30 14:06

winterboum様ご回答ありがとうございます。 1.formを一つにまとめてくださいとのことですが、skillについては、skillsテーブルを作成し、プロフィール部分(usersテーブル)とは完全に別扱いなのですが、一つにまとめることが可能なのでしょうか？(可能であるなら、その方法は?) 2.それは、def user_params部分で、何か修正が必要になるということでしょうか？

winterboum

2020/12/31 03:24

別tableですか。そうしますとmodelの定義を見せていただかないとやりにくいです。

Tarzan3154

2021/01/07 06:02

回答に気付かずすみません。 user.rbおよびskill.rbのコードを追記しておりますので、ご確認をお願い致します。

winterboum

2021/01/07 06:55

そういう構造ですと https://qiita.com/ftyabu/items/cae912b211e985d74d58 が参考になるでしょう。 accepts_nested_attributes_for :books: を見落とさないように

Tarzan3154

2021/01/12 14:24

遅くなりすみません。ご指摘の点、user.rbおよびusers_controller.rbを修正してみましたが、特に変化なしでした。それから、skills#newを追加してusers#showで「新規スキル投稿」のリンクを入れてみましたが、リンクをクリックすると、③のエラーメッセージが表示され、うまくいきません。(ActiveRecord::RecordNotFound in SkillsController#newのやつです。) skill.rbおよびskills_controller.rbでは何か修正は必要になりますか？

行動規範の内容に同意します