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

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

ただいまの
回答率

87.80%

Railsでログイン時にエラーが出るようになった

解決済

回答 1

投稿

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

score 65

前提・実現したいこと

Railsでログイン時にエラーが出るようになりました(サインイン時も同様です)。

ログイン機能は下記のコードで過去に実装が終わっており、最近はRuby Deviseを使いフォロー機能を実装しておりました(最近ログイン機能は特にいじっておりません)。

しかしフォロー機能実装時にフォローボタンが表示されない問題が起こり、その原因がどうやらログイン機能にある事が分かった為にログアウト・ログインをし直すと、以前は出来ていたログインでエラーが発生しました。
参考質問:【Rails】フォロー機能実装時にフォローボタンが表示されない

久々にログアウト・ログインをし直して気付いた為、いつから(何が原因で)この不具合が裏で出ていたのかが分かりません。どなたかお知恵を拝借出来ましたら幸いです。

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

ArgumentError in UsersController#login
wrong number of arguments (given 0, expected 1)

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")

該当のソースコード(users_controller.rb)

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

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

  def following
    @title = "フォロー"
    @user = User.find(params[:id])
    @users = @user.followings
    render 'show_follow'
  end

  def followers
    @title = "フォロワー"
    @user = User.find(params[:id])
    @users = @user.followers
    render 'show_follow'
  end

end

該当のソースコード(login_form.html.erb)

<div class="main users-new">
  <div class="container">
    <div class="form-heading">ログイン</div>
    <div class="form users-form">
      <div class="form-body">
        <% if @error_message %>
          <div class="form-error">
            <%= @error_message %>
          </div>
        <% end %>
        <%= form_tag("/login") do %>
          <p>メールアドレス</p>
          <input name="email" value="<%= @email %>">
          <p>パスワード</p>
          <input type="password" name="password" value="<%= @password %>">
          <input type="submit" value="ログイン">
        <% end %>
      </div>
    </div>
  </div>
</div>

該当のソースコード(routes.rb)

Rails.application.routes.draw do
  devise_for :users
  post "likes/:post_id/create" => "likes#create"
  post "likes/:post_id/destroy" => "likes#destroy"

  get "login" => "users#login_form"
  post "login" => "users#login"
  post "logout" => "users#logout"

  post "users/:id/update" => "users#update"
  get "users/:id/edit" => "users#edit"
  post "users/create" => "users#create"
  get "signup" => "users#new"
  get 'users/index' => 'users#index'
  get "users/:id" => "users#show"
  get "users/:id/likes" => "users#likes"
  get "/" => "users#top"

  get 'posts/index'=> 'posts#index'
  get "posts/new" => "posts#new"
  get "posts/:id" => "posts#show"
  post "posts/create" => "posts#create"
  get "posts/:id/edit" => "posts#edit"
  post "posts/:id/update" => "posts#update"
  post "posts/:id/destroy" => "posts#destroy"
  get "posts/search" => "posts#search"  

  get 'about/about'
  get 'home/top' => 'home#top'
  get 'top' => 'home#top'
  get 'about' => 'about#about'

  resources :users do
    member do
      get :following, :followers, :users_tweets
    end
  end
  resources :relationships, only: [:create, :destroy]

  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end

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

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

wrong number of arguments (given 0, expected 1)

「引数が1つ必要だが、引数が何もない」というエラーが出ています。

loginアクションを実行したときに、下記に何の値が入っているか確認してみると、先に進めるのではないかと思います。

  • @users(User.find_byで取得した結果がちゃんと入っているか?)
  • params(フォームから送った値がちゃんと入っているか?)

↑の確認方法ですが、「Rails デバッグ」等で検索すると色々出てきます(自分はpry-railsというGemをよく使っています)。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/01/30 18:30

    その後、全て無事解決致しました。
    フォロー機能実装時に、参考にしたサイトにあるコードを頼り過ぎて鵜呑みにしてしまい、自分のController等でしっかり定義が出来ていなかった事が原因でした。ログインに関連するコードがフォロー機能にもあり、そこでログイン自体にも影響が出てしまったものと思われます。Deviseはアンインストールしましたが直接の原因ではなかったようです。

    今後は一行一行意味を考え、既存(自分)のコードときちんと関連付け、統一感を持たせる様心掛けます。
    怠慢さから来る原因で申し訳ありませんでしたが、また機会がありましたら是非宜しくお願い致します。

    キャンセル

  • 2020/02/04 21:51

    無事解決できたようでよかったです!Deviseすごく便利なので、機会があれば是非使ってみてください!

    キャンセル

  • 2020/02/10 08:33

    有難うございます!:)

    キャンセル

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

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

関連した質問

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