form_forについて、エラーを解決したい。

Question

form_forについてです。

#解決したい内容
最終的にやりたい内容はログイン機能でメールアドレスかパスワードが間違っていたらエラーメッセージを出すようにしたいです。
ログインはdeviseを使用しました。
今はエラーメッセージより正常にログインのビューを表示させたいです。
コントローラーをカスタマイズすることは出来ました。

sessions/new.html.haml

.new
  %h4 ログイン
  = form_for(@user) do |f| 
    - if @error_message
      .error
      = @error_message
    .field
      = f.label :email
      %br/
      = f.email_field :email, autofocus: true, autocomplete: "email", class: "text"
    .field
      = f.label :password
      %br/
      = f.password_field :password, autocomplete: "current-password", class: "text"
    - if devise_mapping.rememberable?
      .field
        = f.check_box :remember_me
        = f.label :remember_me

sessions.controller.rb

def new
    @user = User.find_by(email: params[:email], password: params[:password])
    if @user
      redirect_to :root
    else
      @error_message = "エラー"
      @email = params[:email]
      @password = params[:password]
      
      render "devise/sessions/new"
  end
end

コントローラーの2行目をfind_byにすると下記のエラーメッセージが出ます。
エラー個所はビューの3行目のform_forなのでその引数に当たる、@userがおかしいんだと思います。

ArgumentError in Users::Sessions#new
First argument in form cannot contain nil or be empty

なのでコントローラーの2行目をnewに変えてみました。

def new
    @user = User.new
    if @user
      redirect_to :root
    else
      @error_message = "エラー"
      @email = params[:email]
      @password = params[:password]
      
      render "devise/sessions/new"
  end
end

そしたらエラーはなくなるのですが、恐らく、コントローラーの4行目が動いてると思いますが、ルートにリダイレクトされます。
ログイン押しても瞬時にルートに遷移されます。
これをredirect_to "/users/sign_in"に変更すると

このページは動作していません
localhost でリダイレクトが繰り返し行われました。

となります。
これにずっと悩んでいるのでご教授いただけると幸いです。
よろしくお願いします。

Answer 1

form_forの第一引数にnilを渡すことはできません。

@user = User.find_by(email: params[:email], password: params[:password])

の結果がnilになっている場合、@user = User.newを実行するようなコードを追加してみましょう。

Answer 2

yume0409さんの見立てがあっていると思うので、それを前提に回答します。

ログイン処理は概ね以下の流れになると思います。
0. newアクションで@userインスタンスをセット
0. sessions/new.html.hamlでフォームを表示
0. ユーザー名とパスワードを入力してフォームを送信
0. createアクションにリクエストを送信
0. ログイン処理をして、エラーならエラーメッセージを返す
0. ログインが成功した場合はログイン後のページ、エラーの場合はsessions/new.html.hamlでエラーメッセージを表示

今のコードですと、newアクションでインスタンス変数のセットとログイン処理を一緒に実装しているので、うまく行っていないように見えます。

インスタンス変数の設定とログイン処理をわけて実装するとRailsらしい書き方になるので、SessionsControllerは以下のようにするといいと思います。
（コードは既存のコードを再利用したので、適宜修正する必要があるかもしれません。）

# SessionsController

def new
  @user = User.new
end

def create
  @user = User.find_by(email: params[:email], password: params[:password])
  
  if @user
    redirect_to :root
  else
    @error_message = "エラー"
    @email = params[:email]
    @password = params[:password]

    render "devise/sessions/new"
  end
end

Comments

[yume0409]

ありがとうございます。
エラーメッセージが出せるようになりましたが、メールアドレスとパスワードが合っている場合もエラーメッセージが出ます。
loginボタンを押してチャットメインへ遷移されない状態になっています。
コントローラーのcreateにif文がありますが、何か記述のミスはありますでしょうか・・・？

[shgtkshruch]

回答にも書きましたが、createアクションのコードはyume0409さんのコードを利用したものなので、ご自身の設定に応じて修正する必要があると思います。
エラーメッセージが表示されるということはログイン処理ができていないので、ログイン判定を修正してみてください。
form_forのエラーは解決できたと思うので、一旦この回答は解決済みにして、別の質問をたてたほうが課題を混同しないのでいいと思います。