Railsの私はレールに新しいですし、私は指定された値で、特定のデータベースフィールドを更新する一般的な方法を記述するためのタスクを持っているデータベースフィールド

Question

を更新するための一般的な方法。そして、私はアプリのどこからでもメソッドを呼び出すことができるはずです。自分のアプリケーションのコントローラで（私はセキュリティ上の欠陥について理解し、その上..しかし、私はとにかくそれを行うように頼まれた）は、私はこれを達成するためにどのように...これは動作しませんもちろん

def update_my_model_status(model,id,field, value) 
    @model = model.find(id) 
    @model.update(field: value) 
    end 

を試してみましたか？これを行う正しい方法は何ですか？メソッドへの引数としてモデルを渡す方法が可能ならば？あなたがRailsのを使用しない理由のRailsを使用している場合

def update_my_model_status(model,id,field, value) 
    @model_var = model.capitalize.constantize.find(id) 
    @model_var.update_attributes(field: value) 
    end 

Answer 1

：

Answer 2

代わりにちょうどあなたが使用する必要がありますupdateupdate_attributesを使用する：

def update_my_model_status(model,id,field, value) 
    @model_var = model.find(id) 
    @model.update_attributes(field: value) 
    end 

http://api.rubyonrails.org/classes/ActiveRecord/Persistence.html#method-i-update

Answer 3

はこれを試してみてください？

はupdate_all比較：

MyModel.where(id: 1).update_all(banned: true)

または多分update_attribute：

my_model.update_attribute(:banned, true)

へ：

update_my_model_status(MyModel, 1, :banned, true)

最初の2つの方法は、短くても最後のものよりもかなり表現力があることに注意してください。何が起こっているのかがはっきりしています。それだけでなく、通りの向こうのどのRails開発者にとってもすぐに馴染みやすいものですが、カスタムのものは学習曲線があります。これは、不要なメソッドの追加されたコードと組み合わされて、アプリケーションの保守コストを増加させます。さらに、Railsメソッドは十分にテストされ、文書化されています。最後に、Railsメソッドの方が優れています。たとえば、プロトタイプは属性の検証を単純に使用しますが、チェックはしません（予期しない動作が発生する可能性があります）。これは、カスタムメソッドを書くために細かいですが、私はあなたのモデルを更新するこの方法に対して助言する