【Rails】 「if @user.save」というコードの挙動について

こちらでどうでしょうか

基本的にプログラミング言語では条件式にメソッドを入れると、そのメソッドが実行されてその戻り値が評価されます。
rubyには明るくないのですがsaveメソッドの戻り値は保存に成功したか否かを返すようです。

メソッドを評価するといっても実際に実行して戻り値が返ってこないと評価はできませんよね？
せっかく実行したのに元に戻したりはしないのでそのまま登録が完了してしまうのではないでしょうか。

以下Javaの話にはなりますが大抵のプログラミング言語で同じ動きをすると思います。

データベースに登録するのならtrueの場合に実行する処理の部分に@user.saveが記述されているべきなのではないかと思ってしまいます。

「このifで判定しているのは何か」を考えてみてください。
ここで判定しているのは「userの保存が成功したか失敗したか」です。

if @user.saveという書き方が分かりにくいのであれば、下記のコードでどうでしょうか？

@isSeveSuccessed = @user.save # userを保存する
if @isSeveSuccessed # userの保存が成功したか失敗したかを判定する
    # 成功した場合
    flash[:notice] = "登録しました"
    # どこかへリダイレクト
else
    # 失敗した場合
    flash.now[:alert] = "登録に失敗しました"
    render "new" end
end

成功の場合は次のような流れになります。

1. userを保存する
2. userの保存が成功したか失敗したかを判定する（成功した）
3. 「登録しました」のメッセージ
4. どこかへリダイレクト

trueの場合に実行する処理の部分に@user.saveを記述しなくてもいい理由

userの保存が成功しているのに、再度userの保存を行うことになるからです。

1. userを保存する
2. userの保存が成功したか失敗したかを判定する（成功した）
3. userを保存する（２回目）
4. 「登録しました」のメッセージ
5. どこかへリダイレクト

あくまでも条件式の部分に書かれたコードはtrueかfalseかを評価するための
もの

if文としてはそうですが、その条件式は実行されます。
条件式を実行した結果が評価されると考えてください。
次の結果はどちらも同じです。

def equalZero(num)
    return num == 0
end

@num = 0

if @num == 0
    puts 'numは0です'
end

if equalZero(@num)
    puts 'numは0です'
end