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

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

新規登録して質問してみよう
ただいま回答率
86.12%
Ruby on Rails 3.1

Ruby on Rails バージョン3.1.0はRuby on Railsの特定のバージョンです。2011年8月30日にリリースされました。

受付中

ProgateのQuest(Ruby Rails)でのエラーコードの修正

layout
layout

総合スコア0

Ruby on Rails 3.1

Ruby on Rails バージョン3.1.0はRuby on Railsの特定のバージョンです。2011年8月30日にリリースされました。

1回答

0リアクション

0クリップ

185閲覧

投稿2022/10/02 02:38

前提

オンラインプログラミング学習サイト、Progateの道場レッスンQuestの1ページ目をしています。
回答がないため、正解が分かりません。

実現したいこと

・ログインしていない状態で
/users/1/likes
にアクセスしたときに
/login
にリダイレクトする

・リダイレクトしたときに
ログインが必要です
というFlashメッセージを表示する

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

NoMethodError in Users#likes Showing /home/progate/tweet_app/app/views/users/likes.html.erb where line #7 raised: undefined method `id' for nil:NilClass Extracted source (around line #7): <h2><%= @user.name %></h2> <p><%= @user.email %></p> <% if @user.id == @current_user.id %>(エラーとされているのはここの部分) <%= link_to("編集", "/users/#{@user.id}/edit") %> <% end %> </div>

該当のソースコード

Ruby on rails

試したこと

NoMethodErrorを調べたところ、コントロールにメソッドを定義していないと出ていたのですが

・Application_Controller

class ApplicationController < ActionController::Base
before_action :set_current_user

def set_current_user
@current_user = User.find_by(id: session[:user_id])
end

def authenticate_user
if @current_user == nil
flash[:notice] = "ログインが必要です"
redirect_to("/login")
end
end

def forbid_login_user
if @current_user
flash[:notice] = "すでにログインしています"
redirect_to("/posts/index")
end
end

end

・User_Controller

class UsersController < ApplicationController
before_action :authenticate_user, {only: [:index, :show, :edit, :update]}
before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]}
before_action :ensure_correct_user, {only: [:edit, :update]}

def index
@users = User.all
end

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

def new
@user = User.new
end

def create
@user = User.new(
name: params[:name],
email: params[:email],
image_name: "default_user.jpg",
password: params[:password]
)
if @user.save
session[:user_id] = @user.id
flash[:notice] = "ユーザー登録が完了しました"
redirect_to("/users/#{@user.id}")
else
render("users/new")
end
end

def edit
@user = User.find_by(id: params[:id])
end

def update
@user = User.find_by(id: params[:id])
@user.name = params[:name]
@user.email = params[:email]

if params[:image] @user.image_name = "#{@user.id}.jpg" image = params[:image] File.binwrite("public/user_images/#{@user.image_name}", image.read) end if @user.save flash[:notice] = "ユーザー情報を編集しました" redirect_to("/users/#{@user.id}") else render("users/edit") end

end

def login_form
end

def login
@user = User.find_by(email: params[:email])
if @user && @user.authenticate(params[:password])
session[:user_id] = @user.id
flash[:notice] = "ログインしました"
redirect_to("/posts/index")
else
@error_message = "メールアドレスまたはパスワードが間違っています"
@email = params[:email]
@password = params[:password]
render("users/login_form")
end
end

def logout
session[:user_id] = nil
flash[:notice] = "ログアウトしました"
redirect_to("/login")
end

def likes
@user = User.find_by(id: params[:id])
@likes = Like.where(user_id: @user.id)
end

def ensure_correct_user
if @current_user.id != params[:id].to_i
flash[:notice] = "権限がありません"
redirect_to("/posts/index")
end
end

end

で、
def forbid_login_user
if @current_user
flash[:notice] = "すでにログインしています"
redirect_to("/posts/index")
end
end
がUser_Controllerが継承しているAuthonticate_Controllerで定義されているため、
このままでも要求された動きをするはずなのですが
どこが間違っているのでしょうか。

ちなみにルーティングも
get "users/:id/likes" => "users#likes"
です。

何か必要な事項などがありましたらお教えください。

以下のような質問にはリアクションをつけましょう

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

リアクションが多い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

気になる質問をクリップする

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

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

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

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

適切な質問に修正を依頼しましょう。

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
86.12%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

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

Ruby on Rails 3.1

Ruby on Rails バージョン3.1.0はRuby on Railsの特定のバージョンです。2011年8月30日にリリースされました。