Question

您好我为我在网站上建立的购物车创建此条件时遇到了一些麻烦，以便在零件出售时显示总数。在我的架构中，我有零件，line_items，其中包含零件和推车的ID，以及推车。零件具有折扣属性。如果零件有折扣，它将显示折扣和零件的新价格。我的line_items有一个名为line_item_discount的方法，如果一个部分包含折扣，它将创建新的部分总和。虽然它显示了零件，折扣和新价格，但购物车总数并未更新。

我在这里创建了一个名为total_price_with_discount的方法

class Cart < ActiveRecord::Base
  has_many :order_items
  belongs_to :user
  has_many :line_items, dependent: :destroy

  def add_part(part_id)
    current_part = line_items.find_by(part_id: part_id)
    if current_part
      current_part.quantity += 1
    else
      current_part = line_items.build(part_id: part_id)
    end
    current_part
  end

  def total_price
    line_items.to_a.sum { |item| item.total_price}
  end

  def total_price_with_discount
    line_items.to_a.sum { |item| item.total_price.line_item_discount}
  end

现在我卡在了_cart部分里面我试图创建一个条件，如果一个零件有折扣，它将使用total_price_with_discount方法，但如果一个零件没有折扣，它将使用total_price。我已经尝试了很多方法来创建条件，但我不断收到这样的消息

由于某种原因，购物车没有line_items或它出现的部件的实例。

以下是我的购物车，零件和line_items的表格

create_table "carts", force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer  "user_id"
    t.integer  "quantity"
    t.decimal  "subtotal"
  end

create_table "parts", force: :cascade do |t|
    t.string  "name"
    t.text    "description"
    t.integer "category_id"
    t.integer "price"
    t.boolean "active"
    t.integer "discount"
    t.string  "image"
    t.integer "quantity"
  end
  create_table "line_items", force: :cascade do |t|
    t.integer  "part_id"
    t.integer  "cart_id"
    t.datetime "created_at",             null: false
    t.datetime "updated_at",             null: false
    t.integer  "quantity",   default: 1
  end

我的零件模型

class Part < ActiveRecord::Base
  has_many :order_items
  has_many :line_items

  before_destroy :ensure_not_referenced_by_any_line_item


  def ensure_not_referenced_by_any_line_item
    if line_items.empty?
      return true
    else
      errors.add(:base, 'Line Items present')
      return false
    end
end


  def subtotal
    parts.collect { |part| part.valid? ? (part.quantity * part.unit_price) : 0}.sum
  end


  def apply_discount
    price - (discount.to_f/100 * price)
  end

end

我的line_items模型

class LineItem < ActiveRecord::Base
  belongs_to :part
  belongs_to :cart

  def total_price
    part.price * quantity
  end

  def line_item_discount
    part.price - (part.discount.to_f/100 * part.price) * quantity
  end

end

这是抛出错误的部分视图

<h2>Your Cart</h2> <table>
    <%= render(cart.line_items) %>
<tr class="total_line">
<td colspan="2">Total</td>
<%unless cart.line_items.part.discount?%>
<td class="total_cell"><%= number_to_currency(cart.total_price) %></td>
<%end%>

<%if cart.line_items.part.discount?%>
<td class="total_cell"><%= number_to_currency(cart.total_price_with_discount) %></td>
<%end%>
    </tr>
  </table>

<%= button_to 'Empty cart', cart, method: :delete, data: { confirm: 'Are you sure?' } %>

感谢您对此提供任何帮助和建议

Answer 1

在您的视图中，您正在调用cart.line_items.part，但line_items是多个LineItem对象的集合，.part不是该集合对象上的有效方法。

正如您在错误中看到的那样，part缺少ActiveRecord::Associations::CollectionProxy方法。

您应该在LineItem上创建范围，例如：

scope :with_discounted_part, { joins(:part).where.not(part: { discount: nil }) }

然后在您看来，您可以这样做：

<% if cart.line_items.with_discounted_part %>

Answer 2

我要添加的Cart模型：

has_many :parts, through: :line_items

将范围添加到Part模型：

scope :with_discounts, -> { where.not(discount: nil) }

然后将视图更改为：

<td class="total_cell">
  <%if cart.parts.with_discount.any?%>
    <%= number_to_currency(cart.total_price_with_discount) %>
  <%else%>
    <%= number_to_currency(cart.total_price) %>
  <%end%>
</td>

更新：除了上述内容之外，我相信下面的代码效率更高，但我会提供这两个选项并允许您选择。

下面我们总是在line_item_discount模型中的total_price方法中使用LineItem：

def total_price
  (part.price * quantity) - line_item_discount
end

def line_item_discount
  return 0 if part.discount.blank?
  (part.discount.to_f/100 * part.price) * quantity
end

然后你甚至不需要视图中的if语句，total_price将以任何方式工作：

<td class="total_cell"><%= number_to_currency(cart.total_price) %></td>

然后，您可以从total_price_with_discount模型中删除Cart方法。

我们还可以在total_price模型中调整Cart方法： （适用于任何代码选择）

def total_price
  line_items.sum(:total_price)
end

Rails未定义局部变量或方法`line_item'

2 个答案: