文章详情

在一家软件开发公司进行面试时,面试官给出了业务场景和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就被解决了。