合并相似对象的方法(数量不同)

时间:2013-06-28 18:28:54

标签: ruby-on-rails methods model attributes updating

我需要一个包含has_many line_items的购物车对象的帮助。如果购物车中的line_item已创建并且它与购物车中已有的line_item具有相同的确切属性,我只想更新预先存在的line_items数量,而不是创建具有单独数量的重复对象。

我在模型中编写了一些方法来尝试完成这项工作,但它不起作用。以下是我的代码:

模型

class LineItem < ActiveRecord::Base
  attr_accessible :cart_id, :product_id, :quantity, :unit_price, :product, :cart, :color_id, :size_id, :extra_id
  belongs_to :cart
  belongs_to :product
  belongs_to :color
  belongs_to :size
  belongs_to :extra
  validates :quantity, :presence => true

  def update_quantity(qty)
    quantity += qty
    quantity.save
  end
  def exists_in_collect?(items)
    if items.include?(product)
        if color == items.color && size == items.sizes && extra == items.extra
            return true
        end
    else
        return false
    end
  end
end

class Cart < ActiveRecord::Base
  attr_accessible :purchased_at
  has_many :line_items
  has_one :order

  def where_line_item_with(prod_id)
    line_items.where(:product_id => prod_id)
  end
end

控制器

class LineItemsController < ApplicationController
  def new
    @line_item = LineItem.new
  end
  def create
    @line_item = LineItem.new(params[:line_item].merge(:cart => current_cart))
    if @line_item.exists_in_collect?(current_cart.line_items)
      current_cart.where_line_item_with(product.id).update_quantity(@line_item.quantity)
      @line_item.destroy!
    else
    @line_item.save!
    @line_item.update_attributes!(:unit_price => @line_item.item_price)
    end


    redirect_to current_cart_url
  end
  def update
    @line_item = LineItem.find(params[:id])
    @line_item.update_attributes(params[:line_item])
    redirect_to current_cart_url
  end

完全赞赏任何见解。

1 个答案:

答案 0 :(得分:1)

1.您应该将where_line_item_with(prod_id)更改为以下内容:

def where_line_item_with(prod_id)
  line_items.where(:product_id => prod_id).first
end

由于where返回一个数组而您无法在数组上执行update_quantity(@line_item.quantity)

2.在exists_in_collect?(items) - 这里您的目的是查找购物车的商品是否包含与新商品相似的商品。它应该更新如下:

def exists_in_collect?(items)
  items.each do |item|
    if color == item.color && size == item.sizes && extra == item.extra && product == item.product
        return true
    end  
  end
  return false
end