计算机专业面试题：业务上BUG一条，挑战你的编程智慧

在一家电商平台上，有一个订单管理系统。该系统允许用户添加商品到购物车，下单购买。是一个简化的订单管理系统的代码片段，存在一个BUG。你的任务是找出这个BUG，并修复它。

python

class ShoppingCart:

def __init__(self):

self.items = []

def add_item(self, item):

self.items.append(item)

def remove_item(self, item):

if item in self.items:

self.items.remove(item)

def total_price(self):

total = 0

for item in self.items:

total += item.price

return total

class Order:

def __init__(self, cart):

self.cart = cart

def place_order(self):

if len(self.cart.items) == 0:

raise ValueError("购物车为空，无法下单")

for item in self.cart.items:

print(f"商品：{item.name}, 价格：{item.price}")

print("订单已生成")

class Item:

def __init__(self, name, price):

self.name = name

self.price = price

# 测试代码

cart = ShoppingCart()

cart.add_item(Item("苹果", 10))

cart.add_item(Item("香蕉", 20))

order = Order(cart)

order.place_order()

分析

在上面的代码中，`ShoppingCart` 类负责管理购物车中的商品，`Order` 类负责处理订单的生成。当你尝试下单时，系统会检查购物车中是否有商品，没有商品，则会抛出一个 `ValueError` 异常。

当你尝试添加一个商品到购物车后，再尝试移除它，再下单时，你会发现订单会生成，尽管购物车中没有任何商品。这是因为在移除商品时，虽然商品被从列表中移除了，但 `total_price` 方法在计算总价时仍然会计算这个已经被移除的商品的价格。

BUG修复

要修复这个BUG，我们需要确保在移除商品后，`total_price` 方法不会计算已经被移除的商品的价格。是修复后的代码：

python

class ShoppingCart:

def __init__(self):

self.items = []

def add_item(self, item):

self.items.append(item)

def remove_item(self, item):

if item in self.items:

self.items.remove(item)

# 修复点：移除商品后，重新计算总价

self.calculate_total_price()

def calculate_total_price(self):

total = 0

for item in self.items:

total += item.price

self.total_price = total

def total_price(self):

return self.total_price

class Order:

def __init__(self, cart):

self.cart = cart

def place_order(self):

if len(self.cart.items) == 0:

raise ValueError("购物车为空，无法下单")

for item in self.cart.items:

print(f"商品：{item.name}, 价格：{item.price}")

print("订单已生成")

class Item:

def __init__(self, name, price):

self.name = name

self.price = price

# 测试代码

cart = ShoppingCart()

cart.add_item(Item("苹果", 10))

cart.add_item(Item("香蕉", 20))

cart.remove_item(Item("苹果", 10)) # 修复前会计算苹果的价格

order = Order(cart)

order.place_order() # 修复后应该不会计算苹果的价格

在修复后的代码中，我们添加了一个新的方法 `calculate_total_price` 来重新计算总价，在 `remove_item` 方法中调用它。这样，每次移除商品后，都会重新计算购物车的总价，确保了订单中的商品总价是准确的。