Routing Errorが出るときと出ない時がある

Question

https://teratail.com/questions/229498前にこの質問をさせていただいて解決したと思ったのですが、フォローボタンを押してたときに「Routing Error　uninitialized constant RelationshipsController」と出てくるときと正常に動作するときがあります。

原因が全くわからないので教えていただけると幸いです。

Answer 1

原因が全くわからないので教えていただけると幸いです。

1つ考えられるものとして、「autoloadがうまくいっていない」ということがあります。RelationshipsControllerをrelationships_controller.rbという正しいファイルに書かず、別なファイルに書いてしまっていた場合、その「別なファイル」が事前にロードされていれば動く、という挙動になります。

Comments

[退会済みユーザー]

relationships_controller.rb　の中身なのですが正しくかけていないでしょうか

class RegistrationsController < Devise::RegistrationsController

protected

def update_resource(resource, params)
resource.update_without_current_password(params)
end
end
class RelationshipsController < ApplicationController

def create
@user = User.find(params[:relationship][:following_id])
current_user.follow!(@user)
redirect_to @user
end

def destroy
@user = Relationship.find(params[:id]).following
current_user.unfollow!(@user)
redirect_to @user
end
end

[maisumakun]

> 正しくかけていないでしょうか

RegistrationsControllerをrelationships_controller.rbの中に書くのが正しくないです。

[退会済みユーザー]

RegistrationsController はrelationships_controller.rbではなくrelation_ships_controller.rbに書くのが正しいとうことですか？

[maisumakun]

RegistrationsControllerは、registrations_controller.rbというファイルの中に書くべきものです。

[退会済みユーザー]

registrations_controller.rbに書いたのですが動作は前と同じでした。
まだ他に問題があるのでしょうか

[退会済みユーザー]

いろいろ試してみたのですがhttp://localhost:3000/users/editにアクセスしたあとフォローボタンを押すと正しく動作することがわかりました。

[maisumakun]

逆に、「RelationshipsController」というものは存在しますか？

[退会済みユーザー]

ありませんでした。

[maisumakun]

とすれば、「間違えてRelationshipsControllerを呼ぶようなリンクを作成してしまっている」ことが問題かと思います。

[退会済みユーザー]

リンクをRegistrationsControllerを呼ぶようにしたらいいということですか？

[退会済みユーザー]

http://localhost:3000/users/editにアクセスするとフォローボタンが動作するのはなぜでしょうか

Answer 2

前回の質問　というのは私が回答したやつだと思いますが、
このとき
registrations_controller.rb　に　class RelationshipsController
を書いていたのを指摘して　 relation_ships_controller.rb に書くように変更した。
そのあと
Unable to autoload constant RegistrationsController, expected /Users/〇〇/Site/〇〇/app/controllers/registrations_controller.rb to define it
がでたので、

registrations_controller.rb　の中身を壊してしまったのでは？と思われます。
registrations_controller.rb　には　RegistrationsController が定義されていなければならないのに、RelationshipsController が定義されています

とコメントして、解決した　とありましたね？
なのに今回maisumakunさんの質問に、「RelationshipsController」というものは存在しますか？　に対し「ありません」　とあるのですが、 ???

registrations_controller.rb に RegistrationsController　を
relation_ships_controller.rb に RelationshipsController を
定義するようになっていますか？

Comments

[退会済みユーザー]

registrations_controller.rb

class RegistrationsController < Devise::RegistrationsController

protected

def update_resource(resource, params)
resource.update_without_current_password(params)
end
end
end

relation_ships_controller.rb

class RelationshipsController < ApplicationController

def create
@user = User.find(params[:relationship][:following_id])
current_user.follow!(@user)
redirect_to @user
end

def destroy
@user = Relationship.find(params[:id]).following
current_user.unfollow!(@user)
redirect_to @user
end
end

になっていますがRouting Error　uninitialized constant RelationshipsControllerが出てしまいます。

[winterboum]

いかん、RelationShips　と思い込んでた
relationships_controller.rb に改名してください

[退会済みユーザー]

名前変えたら出来ました！

[winterboum]

ごめん、思い込みで振り回してしまいました
で、
今回を教訓に、class名とfile名をきちんと合わせるというのを確実に。
rails g controller を使って作っていればこういうことは起きないのですが