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

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

ただいまの
回答率

88.33%

railsの起動時のName Errorについて

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,005

gunofu

score 8

 前提・実現したいこと

ここに質問の内容を詳しく書いてください。
初めての質問です。
現在、ruby on railsを勉強している初心者です。
画像投稿サイトのサインインの認証チェックを実行しようと思い以下を書いたところerrorに原因が分からず困っています。
よろしければご教授いただければ幸いです。

エラーメッセージ
Routing Error
undefined method `user_signed_in?' for UsersHelper:Module

 該当のソースコード

users_controller.rb
class UsersController < ApplicationController
  before_action :authorize, except: [:sign_up, :sign_up_process, :sign_in, :sign_in_process]
  before_action :redirect_to_top_if_signed_in, only: [:sign_up, :sign_in]
  def top
  end
  # ユーザー登録ページ
  def sign_up
    @user = User.new
    render layout: "application_not_login"
  end
  # サインインページ
  def sign_in
    @user = User.new
    render layout: "application_not_login"
  end
  def sign_out
    #ここに処理を実装
    user_sign_out
    #サインインページへ遷移
    redirect_to sign_in_path and return
  end
  # サインイン処理
  def sign_in_process
    # パスワードをmd5に変換
    password_md5 = User.generate_password(user_params[:password])
    # メールアドレスとパスワードをもとにデータベースからデータを取得
    user = User.find_by(email: user_params[:email], password: password_md5)
    if user
      # セッション処理
      user_sign_in(user)
      # トップ画面へ遷移する
      redirect_to top_path and return
    end
  end
  #ユーザー登録処理
  def sign_up_process
    user = User.new(user_params)
    if user.save
      #登録が成功したらサインインしてトップページへ
      user_sign_in(user)
      redirect_to top_path and return
    else
      #登録が失敗したらユーザー登録ページへ
      flash[:danger] = "ユーザー登録に失敗しました。"
      redirect_to('/')
    end
  end
  private
  def user_params
    params.require(:user).permit(:name, :email, :password)
  end
   #現在サインイン中のユーザー情報を返す
    def current_user
      if @current_user.nil?
        @current_user = User.find_by(id: session[:user_id])
      else
        @current_user
      end
    end
end
users_helper.rb
module UsersHelper
  # ユーザーがサインインしていればtrue, そうでなければfalseを返す
    def user_signed_in?
      current_user.present?
    end
    # 渡されたユーザーでサインインする
    def user_sign_in(user)
      session[:user_id] = user.id
    end
    #サインアウトする
    def user_sign_out
        session.delete(:user_id)
        @current_user = nil
    end
   # 現在サインイン中のユーザー情報を返す
    def current_user
      if @current_user.nil?
        @current_user = User.find_by(id: session[:user_id])
      else
        @current_user
      end
    end
    # 認証チェック
    def authorize
      redirect_to sign_in_path unless user_signed_in?
    end
    # サインイン済みならトップページに遷移する
    def redirect_to_top_if_signed_in
      redirect_to top_path and return if user_signed_in?
    end
end
models/user.rb
class User < ApplicationRecord
  # データの保存前に、パスワードを暗号化するメソッド(convert_password)を実行するよう設定
  before_save :convert_password
  #バリデーション
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i 
  validates :name, presence: true
  validates :email, presence: true, format: {with: VALID_EMAIL_REGEX}, uniqueness: true
  validates :password, presence: true, length:{minimun: 6}
  # パスワードを暗号化するメソッド
  def convert_password
    self.password = User.generate_password(user_params[:password])
  end
  # メールアドレスとパスワードをもとにデータベースからデータを取得
  user = User.find_by(email: user_params[:email], password: password_md5)
end

皆様、ご意見ありがとうございます。 返信遅くなり申し訳ありません。 users_helper.erの6行目のご指摘がありましたので、修正したところ以下のerrorメッセージに 変わりました。 イメージ説明

ちなみにroutes.rbは以下になります。

Rails.application.routes.draw do
  root 'users#sign_up'
  get'top', to:'users#top', as: :top
  get 'users/top'
  resources :posts
  get '/profile/(:id)', to:'users#show',as: :profile
  get '/profile/edit', to:'users#edit',as: :profile_edit
  get '/follower_list/(:id)', to:'users#follower_list',as: :follower_list
  get '/follow_list/(:id)', to:'users#follow_list',as: :follow_list
  get '/sign_up', to:'users#sign_up',as: :sign_up
  get '/sign_in', to:'users#sign_in',as: :sign_in
  get '/sign_out', to:'users#sign_out',as: :sign_out
  post '/sign_up', to:'users#sign_up_process'
  post '/sign_in', to:'users#sign_in_process'
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end


ご意見、ご教授お願い致します。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • psuke

    2018/08/23 23:20

    ```users_helper.rb```の下から6行目に```def ... end```で囲まれていないredirect_toの構文がありますが、これは意図的なものでしょうか?

    キャンセル

  • shiena

    2018/08/24 12:56

    ブラウザにエラー画面が出ているならそのキャプチャを、ログにエラーが出ているなら抜粋せずにそのまま載せてください。長くなってもいいのですが漏れがあると原因特定が難しいからです。

    キャンセル

回答 2

checkベストアンサー

0

UserモデルからUsersControllerのメソッドであるuser_paramsを使っているためエラー画面でundefined local variable or methodエラーになっています。
モデルからコントローラーのメソッドや変数は参照できないので、コントローラーからモデルを利用するように書くべきです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

routes.rbのコードはどのようになっていますか?
ルーティングの定義が誤っている恐れがあります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • ただいまの回答率 88.33%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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