質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.33%

  • Ruby

    8175questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7665questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

関連付けしたモデルのデータ削除に関して undefined method for nil:NilClass

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 230

stwebyy

score 5

現在、各ユーザーに毎に情報を登録できるアプリを開発しています。
ユーザー機能はdeviseを使用しています。
今実装したい内容が、ユーザーを削除すると、そのユーザーが登録した情報も一緒に消えるという機能を実装したいと考えています。
そこで、userモデルとcompanyモデルを1対多の関係で関連付けをしました。
また、companyコントローラーにdestroyアクションを追加し、開いている情報ページの単体の削除機能は実装いたしました。
次にuserコントローラーにもdestroyアクションを追加し、削除を行ったところ、

undefined method `name' for nil:NilClass


と出てしまいました。
モデルの記述も問題ないと思われるため、各コントローラーのdestroyアクションの記述が問題だと思うのですが、解決方法がわかりません。
原因を調べるためにログを確認すると以下の内容が出ていました。

Started DELETE "/users/2" for 127.0.0.1 at 2018-05-02 09:21:40 +0900
Processing by UsersController#destroy as HTML
  Parameters: {"authenticity_token"=>"I7m3KAKxOINmRXurMXIR9euoW5hoIDT5ye8OSXggyfo=", "id"=>"2"}
  User Load (1.3ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 2 ORDER BY `users`.`id` ASC LIMIT 1
  User Load (0.4ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 2 LIMIT 1
   (0.1ms)  BEGIN

  Company Load (0.4ms)  SELECT `companies`.* FROM `companies` WHERE `companies`.`user_id` = 2
  SQL (1.4ms)  DELETE FROM `companies` WHERE `companies`.`id` = 3
   (15.3ms)  ROLLBACK
Completed 500 Internal Server Error in 46ms

NoMethodError (undefined method `name' for nil:NilClass):
  app/controllers/users_controller.rb:9:in `destroy'

このcompaniesは、
user.id = companies.user_id
となっているため、user.id = 2のデータを削除するのであれば、最後のsql文が

DELETE FROM `companies` WHERE `companies`.`user_id` = 2


とならなくてはいけないはずですが、controllerをいじっても変わってくれません・・・。
5〜6時間以上ググったりしてみたのですが、解決の見込みが立っておりません。
どなたか原因、対策を教えていただけますでしょうか?
よろしくお願いいたします。

user.rb

class User < ActiveRecord::Base
  attr_accessor :login
  has_many :companies, dependent: :destroy
  validates :name, presence: true, length: { maximum: 50 }
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable,
         authentication_keys: [:login]

  def self.find_for_database_authentication(warden_conditions)
    conditions = warden_conditions.dup
    if login = conditions.delete(:login)
      where(conditions.to_h).where(["lower(name) = :value OR lower(email) = :value", { :value => login.downcase }]).first
    elsif conditions.has_key?(:name) || conditions.has_key?(:email)
      where(conditions.to_h).first
    end
  end
end
company.rb

class Company < ActiveRecord::Base
  belongs_to :user
  validates :nameco, presence: true, length: { maximum: 40 }
  validates :business, :wish, presence: true, length: { maximum: 300 }
  validates :recruit, presence: true, length: { maximum: 100 }
  validates :income, :employee, numericality: { greater_than:0 }
  validates :wishpoint, presence: true, numericality: { less_than: 6, greater_than:0 }
  validates :remarks, length: { maximum: 200 }
end
users.controller.rb

class UsersController < ApplicationController
  before_action :authenticate_user!
  def show
    @user=User.find(params[:id])
  end

  def destroy
    # TODO ここでエラー発生
    User.find(params[:id]).destroy
    flash[:success] = "削除しました。"
    redirect_to root_path
  end
end
companies.controller

class CompaniesController < ApplicationController
 before_action :authenticate_user!
 helper_method :sort_column, :sort_direction
  def new
    @company = Company.new
    @submit = false
  end

(略)
.
.
.
.


  def destroy
    Company.find(params[:id]).destroy
    flash[:success] = "削除しました。"
    redirect_to companies_path
  end

  def show
    @company = Company.find(params[:id])
    @id = current_user.id

  end

  def index
    @companies = Company.where(user_id: current_user.id).order(sort_column + ' ' + sort_direction)
  end

.
.
.
.
(略)

  private

    def company_params
        params.require(:company).permit(:salesname, :nameco, :business, :recruit, :income, :employee, :wish, :wishpoint, :advance, :remarks, :user_id)
    end

.
.
.
.
(略)
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

エラーの原因はちょっとわからないのですが、SQLがDELETE FROM `companies` WHERE `companies`.`id` = 3となるのは正しい動作のようです。
dependent: :destroyとした場合、Userを削除するとCompany#destroyが毎回呼ばれるようです。dependent: :delete_allとすると、ActiveRecordを通さず直接SQLが発行されるので、仰るようなSQLになると思います(試してませんが)。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • Ruby

    8175questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7665questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。