フォーム提出後にActiveRecordクエリを実行するにはどうすればよいですか？

Question

これは機能しますか？私はあなたのビューでこれらの値を表示することを計画している場合、私は知らないが、あなたはこのような何かを行うことができ

@logs = Logs.new(log_params) 
@logs.save 
@tt = Users.where(email: params[:email]).update(money: Users.find(current_user.id)['money'] - params[:money]) 
@tt.save 
@wr = Users.find(current_user.id).update(money: Users.where(email: params[:email])['money'] + params[:money]) 
@wr.save 

Answer 1

を転送コインのような何かをしたい：

sender = Users.where(email: params[:email]).first 
# This returns the user 
recipient = Users.find(current_user.id) 
# or just current_user if it inherit from the User class 
money_to_substract = recipient.money - params[:money] 
money_to_sum = recipient.money + params[:money] 

次に、あなたのtranssaction

：

User.transaction do 
    @logs = Logs.new(log_params) 
    @tt = sender.update(money: money_to_substract) 
    # update saves to the database so no need to call save 
    # @tt.save 
    @wr = recipient.update(money: money_to_sum) 
    # update saves to the database so no need to call save 
    # @wr.save 
end 

しかし、私見ビットdryierだろう、私はこのような何かをするだろう

モデル/ User.rb読んで、あなたのコードフォロースルーする事が容易になり、あなたのコントローラ

amount = params[:money].to_i 
User.transaction do 
    @logs = Logs.new(log_params) 
    @tt = sender.sends_money(amount) 
    @wr = recipient.receives_money(amount) 
end 

で

class User < ActiveRecord::Base 

    ... 

    def sends_money(amount) 
    money_to_substract = self.money - amount 
    update(money: money_to_substract) 
    end 

    def receives_money(amount) 
    money_to_sum = self.money + amount 
    update(money: money_to_sum) 
    end 

    ... 

end 

。

希望すると便利です。