再確認用のパスワードについて

Question

rails で再確認のパスワードフォームを作りたいです。

<コントローラー>
・
・
・
  def create
    @user = User.new(
      password: params[:password],
      id: params[:id],
      repassword: params[:repassword]
    )
    if @user.password == @user.repassword and @user.save
      redirect_to("/a/#{@user.id}")
    elsif @user.password != @user.repassword and @user.save
      flash[:notice] = "don't match password"
      render("new")
    else
      flash[:notice] = "please write email or password"
      render("new")
    end
  end
・
・
・

<ビュー>
・
・
・
    <p>password</p>
    <input type="password" name="password" value="<%=@user.password%>">
    <p>please check password.</p>
    <input type="password" name="repassword" value="<%=@user.repassword%>">
・
・
・

と、データベースに「passwordカラム」と「repasswordカラム」を作って、コントローラで、この二つに送られた文字列が一緒ならちゃんとしたページに飛んで、違うときはdon't match passwordを表示させて"new"に飛ぶように作ったはずです。

しかし、実際にはどっちにも通らず、elseのplease write email or passwordが出てしまいます。
試しに

if @user.password == @user.repassword and @user.save

を

 if @user.password == @user.repassword or @user.save

と入力すると理想のページ（"/a/#{@user.id}"）に飛びます。

おそらくは書き方が間違っているのかと思ったのですが、一体どこを間違っているのかわかりません。
どうか知恵をお貸しください。

Answer 1

最新のRailsではHTTPのPOSTで渡されるparameterはStrong Parametersにより明示的に許可しないと使えません。デバッグ用途ではbyebug gemを使うとbinding.pryを途中の行に挟むことができ、処理の途中で処理が止まって、その時点の状態がわかるので理解が進むかもしれません。

おそらくですが@userモデルにはpasswordフィールドが保存に必須で、それが与えられていないので保存に失敗、@user.saveがnilを返しているのかと。

余談ですが、formとmodelのパラメーターは必ずしも1対1で対応する必要はないので、普通はControllerで比較して、一つだけをUserモデルに保存します。もっと言うならpasswordを平文のまま保存するのはセキュリティー上よろしくないので、bcryptなどを使ってpasswordをhash化したものを保存した方がよいです。

def create
  @user = User.new(user_params)
end

def user_params
  params.require(:user).permit(:password, :repassword)
end

Answer 2

まずは、こちらをお読みください。

&&とandの違い - Qiita
Rubyのandと&&、orと||の違い -- ぺけみさお

Rubyにおいては、&&とand, ||とorは結果が異なる場合があります。
通常は、&&と||を使ったほうが混乱が少ないと思います。
というか私はand、orを使った事がありませんし、それで何か困った事はありません。

それはともかくとして、elseはできるだけ避けるほうがコードがシンプルになります。
元のコードを生かす形で直すとこんな感じです。
エラーがあればすぐreturnする、いわゆる「早期リターン」という考え方ですが、かなり有効です。
常にelseを排除すればいいのかというと、そうでもないのですが、そういう判断ができるようになるまでは基本的にすぐreturnしてelseをなくすほうが良いです。

def create
  @user = User.new(
  password: params[:password],
    id: params[:id],
    repassword: params[:repassword]
  )

  # パスワードが一致しなければ、リターン
  unless @user.password == @user.repassword
    flash[:notice] = "don't match password"
    render action: :new
    return
  end

  # この時点でパスワードが一致している事は保証できる

  # バリデーションエラーがある
  unless @user.valid?
    flash[:notice] = "please write email or password"
    render action: :new
    return
  end

  # この時点でバリデーションが通った事は保証できる

  unless @user.save
    flash[:notice] = "save failed"
    render action: :new
    return
  end

  # この時点でレコードが保存できた事が保証できるので単純にリダイレクトすればいい

  redirect_to("/a/#{@user.id}")
end

追記

このようにモデルを定義しているとすると、user.valid?を実行した後、
バリデーションのエラーがある場合、user.errorsにエラー内容が格納されます。

class User < ApplicationRecord
  validates :email, {presence: true, uniqueness: true}
  validates :password, {presence: true}
  validates :repassword, {presence: true}
end

エラー内容を全部知りたい場合、user.errors.full_messagesとすると配列でエラーメッセージが返ってきます。
flash[:notice]に全部入れたいので、改行(\n)で結合して格納する例を示します。

class XXXControler < ApplicationController

  def create
    @user = ...
    ...
    unless @user.valid?
      #flash[:notice] = "please write email or password"
      flash[:notice] = @user.errors.full_messages.join("\n")
      render action: :new
      return
    end
    ...
  end

end

Comments

[harisann]

回答ありがとうございます！
まず、ここでの質問をお許しください。
教えてくださったコードで書いてみたのですが、
please write email or passworｄ
が出るばかりで先に進めません。バリデーションがおかしいのかと思ってヴァリデーションに関するコードを記述したら先に進めました。
原因は何かと思って一個ずつ消して確認してみたところ、どのバリデーションの何か一つ存在しているだけで引っかかってしまいました。
一体何が原因かわかりますか？
教えていただけると幸いです。
＜user.rb＞
・・・
validates :email, {presence: true, uniqueness: true}
validates :password, {presence: true}
validates :repassword, {presence: true}
・・・

[mingos]

コメントではコードブロックが動かないので、回答に追記します

[mingos]

回答に追記しました。
バリデーションのエラーの詳細が分かるようになったはずなので、もう解決していると思いますが、
気付いた点を書いておくと、
テーブルには、email, password, repasswordの３つのカラムがあるんですよね？
であれば、コントローラでは
@user = User.new(
email: params[:email],
password: params[:password],
repassword: params[:repassword]
)
と書かないといけません。
もちろん、フォームにもemailの入力テキストが必要です。
<input type="text" name="email" value="<%= @user.email %>" />

質問で提示してもらったコードでは、emailを渡していないので、emailのpresence:trueの部分で引っかかっているのではないかと予想しています。