Рубин на Rails: ActiveRecord :: StatementInvalid не может бросить ActiveRecord :: StatementInvalid

Question

У меня есть этот код: Рельсы

def newfood 
    memberproduct = MemberProduct.new 
    memberproduct.product_id = Product.where(:barcode_number => params[:barcode_number]).id 
    memberproduct.expiration_date = params[:expiration_date] 
    memberproduct.member_id = current_member.id 
    memberproduct.save 
end 

мне нужен идентификатор продукта. 3-я строка неверна.

У меня есть MemberProduct стола с product_id поля, в expiration_date поле и member_id поля (current_member происходит от devise) У меня есть Product таблицы с полем barcode_number и name поля.

Я получаю эту ошибку:

ActiveRecord::StatementInvalid in FoodController#newfood TypeError: can't cast ActiveRecord::Relation::ActiveRecord_Relation_Product to string: INSERT INTO "member_products" ("created_at", "expiration_date", "member_id", "product_id", "updated_at") VALUES (?, ?, ?, ?, ?)

Что я делаю неправильно?

Answer 1

Попробуйте

memberproduct.product = Product.where(:barcode_number => params[:barcode_number]).first 

memberproduct.product_id является столбец базы данных, где Rails хранит идентификатор продукта, связанный с вашим memberproduct. Обычно они не используются напрямую; вместо этого имя ассоциации.

Так обе эти работы:

def newfood 
    memberproduct     = MemberProduct.new 
    product      = Product.where(:barcode_number => params[:barcode_number]).first 
    memberproduct.product   = product 
    memberproduct.expiration_date = params[:expiration_date] 
    memberproduct.member   = current_member 

    memberproduct.save 
end 

и

def newfood 
    memberproduct     = MemberProduct.new 
    product      = Product.where(:barcode_number => params[:barcode_number]).first 
    memberproduct.product_id  = product.id 
    memberproduct.expiration_date = params[:expiration_date] 
    memberproduct.member_id  = current_member.id 

    memberproduct.save 
end 

но первая форма является более распространенным явлением. Если вы назначаете такой объект, как product или member, Rails достаточно умен, чтобы запросить объект для его идентификатора и использовать его автоматически.

Также, Product.where потенциально возвращает несколько результатов. Поскольку вы только ожидаете, добавьте .first, чтобы вернуть только первое совпадение.

Answer 2

В зависимости от версии Rails, вы должны быть в состоянии:

Rails 3: Product.find_by_barcode_number(params[:barcode_number])

Rails 4: Product.find_by(barcode_number: params[:barcode_number])

Вы можете упростить свои действия, как так:

mprod = current_member.build_member_product(expiration_date: params[:expiration_date]) 
mprod.product = Product.find_by(barcode_number: params[:barcode_number]) 
mprod.save 

Хотя вы, вероятно, хотите иметь дело с проверками и т. Д. (if mprod.save .. else)