ruby on rails - after_save callback: TypeError: Bid can't be coerced into Fixnum -

-

i have after_save callback in bid model, keep getting error: typeerror @ /items/11/bids bid can't coerced fixnum

below bid model:

class bid < activerecord::base   belongs_to :user   belongs_to :item    validates :amount, presence: true   validate :check_if_highest_bid    after_save :bid_logic     def check_if_highest_bid     errors.add(:amount, "you must enter higher bid") unless self.item.price < self.amount   end     def bid_logic     add_previous_bidders_bid     update_items_current_price     subtract_current_users_price   end     def add_previous_bidders_bid       price = self.item.price       bid = bid.find_by_amount(price)       unless bid.nil?       user = user.find(bid.user_id)       user.budget += bid       user.save       end    end    def update_items_current_price     self.item.price = self.amount     self.item.save   end    def subtract_current_users_price     user = user.find(self.user_id)     user.budget -= self.amount     user.save   end  end

the idea in add_previous_bidders_bid method when past users gets out bid, money goes users budget thats set @ registration.

here schema:

create_table "bids", force: true |t|     t.datetime "created_at"     t.datetime "updated_at"     t.integer  "item_id"     t.integer  "user_id"     t.integer  "amount"   end    create_table "items", force: true |t|     t.datetime "created_at"     t.datetime "updated_at"     t.integer  "user_id"     t.string   "title"     t.integer  "price"   end    create_table "users", force: true |t|     t.string   "email",                  default: "", null: false     t.string   "encrypted_password",     default: "", null: false     t.string   "reset_password_token"     t.datetime "reset_password_sent_at"     t.datetime "remember_created_at"     t.integer  "sign_in_count",          default: 0     t.datetime "current_sign_in_at"     t.datetime "last_sign_in_at"     t.string   "current_sign_in_ip"     t.string   "last_sign_in_ip"     t.datetime "created_at"     t.datetime "updated_at"     t.integer  "budget"   end    add_index "users", ["email"], name: "index_users_on_email", unique: true   add_index "users", ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true  end

any tips?

try user.budget += bid.amount instead of user.budget += bid - doesn't know bid...


Comments

Post a Comment

Popular posts from this blog

c# - How Configure Devart dotConnect for SQLite Code First? -

-
can configure following sample code ? when run following code, error : no entity framework provider found ado.net provider invariant name 'devart.data.sqlite' setting in machine.config <system.data> <dbproviderfactories> <add name="dotconnect sqlite" invariant="devart.data.sqlite" description="devart dotconnect sqlite" type="devart.data.sqlite.sqliteproviderfactory, devart.data.sqlite, version=4.6.287.0, culture=neutral, publickeytoken=09af7300eec23701" /> </dbproviderfactories> </system.data> after add below block code in app.config <?xml version="1.0"?> <configuration>   <configsections> <!-- more information on entity framework configuration, visit http://go.microsoft.com/fwlink/?linkid=237468 --> <section name="entityframework" type="system.data.entity.internal.configfile.entityframeworksection, entit...
Read more

java - Copying object fields -

-
what best way of copying classes when need have 2 independent variables? have simple class: public class myclass { boolean = false; string b= "not empty"; } do need make method : assign(myclass data ) { a= data.a; b= data.b; } is there automatic method copy (duplicate) objects in java? do need make method : pretty close. instead of making method, should make constructor. such constructors called copy constructor , , create them this: myclass(myclass data) { = data.a; b = data.b; } and create copy of instance, use constructor this: myclass obj1 = new myclass(); myclass obj2 = new myclass(obj1); copy constructor can tedious : using copy constructor create deep-copy can tedious, when class has mutable fields. in case, assignment create copy of reference, , not object itself. have create copy of fields (if want deep copy). can go recursive. a better way create deep copy serialize , deserialize object. why not use clone()? ...
Read more

c++ - Clear the memory after returning a vector in a function -

-
i have function processes , stores lot of data in , returns results vector of class. amount of data stored in function tremendous , want clear storage memory of function after finished job. necessary (does function automatically clear memory) or should clear memory function? update: vector<customers> process(char* const *filename, vector<int> id) { vector<customers> list_of_customers; (perform actions) return list_of_customers; } variables defined locally within function automatically released @ end of function scope. destructors objects called, should free memory allocated objects (if objects coded correctly). example of such object std::vector . anything you've allocated new must have corresponding delete release storage. try avoid doing own allocation , use raii instead, i.e. containers or smart pointers.
Read more