rails ポイントの平均を出すところでundefined method `to_i' for のエラーがでます

Question

初学者です。入力画面で、相手への評価ポイントを入れて、今までのポイントと合わせて平均を求めるところで、
undefined method `to_i' for <ActionController::Parameters {"point"=>"5"} permitted: true>:ActionController::Parameters Did you mean? to_h to_s
というエラーが出ています。
どう直したらいいのか分かりません。すいませんが、お知恵を貸してください。

<%= form_tag("/posts/#{@post.id}/finishedafter",method: :post) do %>
            <p>タイトル</p>
            <h3><%=@post.title%></h3>
            <p>取引相手</p>
            <h3><%=@post.postedname%></h3>
            <p>評価０～５（最高は５です）</p>
            <div class="select">
              <select  name="point" id="point" required >
                 <option selected value= "" >選択してください</option>
                 <option value= "0" >0</option>
	             <option value= "1" >1</option>
                 <option value= "2" >2</option>
	             <option value= "3" >3</option>
                 <option value= "4" >4</option>
	             <option value= "5" >5</option>
              </select>
            </div>
           <div class="new-btn">
            <label><input class="btn btn-danger btn-lg " type="submit" value="取引を完了する"></label>
           </div>
        <% end %>

という形で送り。

post.controller

def finishedafter
    newpoint = params.permit(:point)      
    @post = Post.find_by(id:params[:id])　　　＜＝相手の投稿
    @user = User.find_by(id:@post.user_id)　　＜＝相手のデータ

    @user.finishcount += 1　　　　　　　　　　　＜＝平均を求めるための件数
    @user.point.to_f = (@user.point + newpoint.to_i) / @user.finishcount　
　　（相手の評価ポイント　＝　相手のポイント　＋　今回のポイント　/　件数）
    @user.save
    redirect_to("/")
  end

create_table "users", force: :cascade do |t|
    t.string "user_name"
    t.string "password_digest"
    t.string "email"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "imagename"
    t.integer "point"
    t.boolean "admin", default: false
    t.integer "finishcount"
  end

Answer 1

変数にどんな値（どのclassのインスタンス）が入っているのか、を意識するようにしてください。変数名から期待されるものが入るわけではないです。
newpoint = params.permit(:point) で newpoint には ポイントは入っていません {point: 5} というActionController::Parametersが入っています、
5を得るには newpoint = params[:point].to_fにしてください。
それで @user.point = (@user.point + newpoint.to_i) / @user.finishcount はエラーはでなくなります。

ただ、この式では 平均値はでません。
４つの評価の平均値が ３ で今回新たに 評価5が来るとこの式では(3+5)/5 で1.6になってしまいます。(3*4+5)/5で3.4が正しい新しい平均値です

Comments

[tori315]

ありがとうございます。エラーは出なくなったのですが、
def finishedafter
@newpoint = params[:point].to_f
@post = Post.find_by(id: params[:id])
@user = User.find_by(id: @post.user_id)　　
@pp = (@user.point.to_f * @user.finishcount) + @newpoint
@user.finishcount += 1
@user.point = @pp.round(2) / @user.finishcount
@user.save
@post.switch = 2
@post.save
redirect_to("/")
end

@user = User.find_by(id: @post.user_id)の部分でユーザーが呼び出されず、dbのuserの内容が変わらないのですが、なぜ呼び出されないのか分かりません。（postの方はデータが反映されます）どのようなことが原因として考えられるのでしょうか。
dbのpostsテーブルには、t.integer "user_id"のカラムはあります。

[winterboum]

カラムは有っても値が入っていないのでは？

[tori315]

user_idには、userテーブルのuser.idと同じ数値が入っているのですが、なぜか
@user = User.find_by(id: @post.user_id)が働いていないようなのです。。

[winterboum]

寝落ちしてしまって、コメントあるのに今気が付きました。
しかし　@userが取れない？　面妖な、、、
いや、違うな
@user = User.find_by(id: @post.user_id) に失敗すると @user には nil がはいります。
そうすると @user.point、 @user.finishcount、@user.saveで、Nill class にはそんなmethodはない　エラーで落ちます。
落ちては居ないようですがから、読めてます。
すると、@user.saveの失敗だな。
rails c で同じ操作をおこなって、 @user.save が成功しているかどうか見てください。
@user.errors.full_messages を見ると失敗の原因がわかります

[tori315]

ありがとうございます。
すいません、これでやり方が正しいのか分からないのですが、consoleでやってみたら、@user.saveで
falseが出ました。

irb(main):006:0> @user = User.find_by(id: 50)
User Load (10.5ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? LIMIT ? [["id", 50], ["LIMIT", 1]]
=> #<User id: 50, user_name: "田中", password_digest: "$2a$12$hklWHscxLXb1BQwyIn9XhOBIbGsRDR1ibII6G1KgEDp...", email: "ss@ss", created_at: "2019-12-27 13:57:28", updated_at: "2019-12-27 13:57:28", imagename: "default_user.jpg", admin: false, finishcount: 0, point: 0.0>
irb(main):007:0> @pp = (@user.point.to_f * @user.finishcount)+4
=> 4.0
irb(main):008:0> @user.finishcount += 1
=> 1
irb(main):009:0> @user.point = @pp.round(2) / @user.finishcount
=> 4.0
irb(main):010:0> @user.save
(0.1ms) begin transaction
User Exists (0.5ms) SELECT 1 AS one FROM "users" WHERE "users"."email" = ? AND "users"."id" != ? LIMIT ? [["email", "ss@ss"], ["id", 50], ["LIMIT", 1]]
(0.1ms) rollback transaction
=> false
irb(main):011:0>

@user.errors.full_messages を見ると失敗の原因がわかります。←これはどのようにしたらいいのでしょうか。

[winterboum]

@user.save　でエラーになったところで　@user.errors.full_messages　としてください
その結果と共に、model Userの定義も載せてください

[tori315]

これでいいのでしょか？よろしくお願いします。

irb(main):011:0> @user.errors.full_messages
=> ["Passwordを入力してください", "Passwordは6文字以上で入力してください"]

class User < ApplicationRecord
# Include default devise modules. Others available are:
# :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
validates :email,{uniqueness:true,presence:true}

validates :user_name,{presence: true}
validates :user_name,length:{maximum: 40}

validates :password,{presence: true}
validates :password,length:{minimum: 6}
validates :password,length:{maximum: 18}
#has_many :posts => :delete
has_many :posts, dependent: :destroy
has_many :accepts, dependent: :destroy
has_secure_password

has_many :messages, dependent: :destroy
has_many :entries, dependent: :destroy

def self.search(search)
if search
User.where(['user_name LIKE?',"%#{search}%"])
else
User.all
end
end

end

[winterboum]

それで、Userがsaveされない原因が出ていますが、わかりますか

[tori315]

パスワードが必要ということで、
@user.update(user_params)
を @user.saveの前に入れて、

def user_params
params.permit(:user_name,:email,:password)
end
をpostコントローラにつくり、

viewで
<input type="hidden" name="user_name"　value="<%=@user.user_name%>">
<input type="hidden" name="email" value="<%=@user.email%>">
<input type="hidden" name="password" value="<%= @user.password %>">
を入れて、パスワードが送れるようにしようとしましたが、ダメでした。
すいません。わかりません。

[winterboum]

おしい。
パスワードを必須とするのはloginとユーザ登録の場合で、それ以外ではパスワードを要求するのはおかしいですよね。
ですので、validationに手を入れましょう
passwordのvalidation３つ全てに if: :present? を追加してください。

[tori315]

そういうことだったんですか。validationが効いたままなんですが、以下の書き方が違うのか他の原因があるのでしょうか。

validates :password,presence: true,if: :present?
validates :password,length:{maximum: 18},if: :present?
validates :password,length:{minimum: 6},if: :present?

[winterboum]

if: Proc.new { |a| a.password.present? }
に変えてみてください

[tori315]

ちょっとトラブルで、だいぶ先に戻ってしまいました。。
すいません。またちょっと作り直していきます。

[tori315]

上記のやり方で解決しました。
助かりました。今年もまたお力を貸してください。
ありがとうございました。

Answer 2

まず仕様を確認したいです
想像するに、post.pointにいろんな人がその企業を評価した平均値が入るというものだと思うのですが
これは整数でいいのですか？小数点以下は不要ですか？
小数点以下も必要ならpointがinteger型になっている点がおかしいです

つぎにpostが作られるところのコードを張ってください
Post.newでpostデータが作られたときにちゃんと、pointの初期値が入ってますか？
入っていなければ、nilになるので注意してください

Comments

[tori315]

そうですね。integerはfloatにデータ型を変えました。
pointはuserで作りました。

def new
@user = User.new
end

def create
@user = User.new(user_params)
@user.imagename = "default_user.jpg"
@user.point = 0
@user.finishcount = 0
@user.save
if @user.save
session[:user_id] = @user.id
flash[:notice] = "登録に成功しました"
redirect_to("/users/#{@user.id}") #show
else
if User.find_by(email:params[:email])
@error_message = "そのEmailアドレスは使われています"
else
@error_message = "パスワードは６～１８字です"
end
render("/users/new")
end
end

エラーの内容が、
undefined method `to_f' for <ActionController::Parameters {"point"=>"5"} permitted: true>:ActionController::Parameters Did you mean? to_h to_s
なのですが、

@user.point = (@user.point.to_f + newpoint.to_f) / @user.finishcount
を、
@user.point = (@user.point + newpoint / @user.finishcount
に変えると、
ActionController::Parameters can't be coerced into Integer

とでます。

Answer 3

@user.point.to_f = (@user.point + newpoint.to_i) / @user.finishcount　

この部分を

@user.point= (@user.point.to_f + newpoint.to_f) / @user.finishcount　

としてみるとどうなりますか？

Comments

[tori315]

回答ありがとうございます。
そうしたら、コントローラのところの、
@user.finishcount += 1;
の部分で、
undefined method `+' for nil:NilClassというエラーがでてしまいました。

[story_aniki]

おそらく、user.finishcount がnilだったのでしょう。Rubyではnilもオブジェクトなので、演算を含め存在しないメソッドを呼ぼうとすればNoMethodErrorになります
事前にnilの場合を弾いておく、あるいはnilなら適当な値を代入するなどしてください

[tori315]

そこは、クリアできたのですが、やはり
@user.point= (@user.point.to_f + newpoint.to_f) / @user.finishcount
の部分で

undefined method `to_f' for <ActionController::Parameters {"point"=>"5"} permitted: true>:ActionController::Parameters Did you mean? to_h to_s

というエラーがでてしまいます。