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

在一家软件开发公司进行面试时，面试官给出了业务场景和BUG

业务场景：

假设我们正在开发一个在线书店系统，该系统允许用户浏览书籍、添加书籍到购物车、结账购买。系统中的购物车功能需要确保用户在添加书籍到购物车时，能够正确地更新购物车中的书籍数量，当用户删除书籍时，购物车中的数量也要相应减少。

BUG：

在测试过程中，我们发现了一个BUG。当用户添加一本已经存在于购物车中的书籍时，购物车中的书籍数量没有增加，而是保持不变。当用户删除一本不存在的书籍时，购物车中的书籍数量却减少了。

是购物车功能的伪代码实现：

python

class ShoppingCart:

def __init__(self):

self.items = {}

def add_item(self, item_id, quantity):

if item_id in self.items:

self.items[item_id] += quantity

else:

self.items[item_id] = quantity

def remove_item(self, item_id, quantity):

if item_id in self.items and self.items[item_id] >= quantity:

self.items[item_id] -= quantity

if self.items[item_id] == 0:

del self.items[item_id]

else:

raise ValueError("Item not found or insufficient quantity")

def get_quantity(self, item_id):

return self.items.get(item_id, 0)

：

请找出上述伪代码中的BUG，并说明原因。提供修正BUG后的代码。

分析与解答

我们需要理解BUG的具体表现。根据BUG出`add_item`和`remove_item`方法中。下面是对每个方法的分析：

1. `add_item`方法：

– `item_id`已经存在于`self.items`中，代码应该将`quantity`加到现有的数量上。

– `item_id`不存在，代码应该创建一个新的键值对，键为`item_id`，值为`quantity`。

2. `remove_item`方法：

– `item_id`存在，但数量小于要删除的`quantity`，应该抛出一个错误，表示数量不足。

– 数量足够，应该从现有数量中减去`quantity`。

– 减去`quantity`后数量变为0，应该删除这个`item_id`。

根据上述分析，我们可以发现`add_item`方法中存在的当`item_id`已经存在于`self.items`中时，代码没有正确地更新数量。正确的做法是将`quantity`加到现有的数量上，而不是直接赋值。

是修正BUG后的代码：

python

class ShoppingCart:

def __init__(self):

self.items = {}

def add_item(self, item_id, quantity):

if item_id in self.items:

self.items[item_id] += quantity

else:

self.items[item_id] = quantity

def remove_item(self, item_id, quantity):

if item_id in self.items and self.items[item_id] >= quantity:

self.items[item_id] -= quantity

if self.items[item_id] == 0:

del self.items[item_id]

else:

raise ValueError("Item not found or insufficient quantity")

def get_quantity(self, item_id):

return self.items.get(item_id, 0)

通过上述修正，`add_item`方法能够正确地处理已存在的`item_id`，确保数量正确更新。这样，BUG就被解决了。