文章详情

背景

在计算机专业面试中,面试官往往会针对者的专业技能和解决能力进行考察。是一个典型的业务上BUG旨在考察者对实际编程的理解和处理能力。

在一个电商系统中,用户可以在购物车中添加商品。购物车中的商品应该有一个库存数量字段,表示该商品在仓库中的剩余数量。当用户添加商品到购物车时,库存数量大于等于用户请求的数量,则允许添加,否则不允许添加,并返回错误信息。是一个简化的代码片段,用于处理用户添加商品到购物车的逻辑:

python

class ShoppingCart:

def __init__(self):

self.items = {}

def add_item(self, item_id, quantity):

if item_id in self.items:

if self.items[item_id]['quantity'] >= quantity:

self.items[item_id]['quantity'] -= quantity

return True

else:

return False

else:

return False

# 示例

cart = ShoppingCart()

cart.add_item('item1', 5) # 假设商品'item1'的库存是10

print(cart.add_item('item1', 3)) # 应该返回True

print(cart.add_item('item1', 15)) # 应该返回False

在上述代码中,存在一个业务上的BUG。请该BUG,并给出修改后的代码,确保系统的正确性。

BUG

在上述代码中,当用户尝试添加商品到购物车时,库存数量不足以满足用户请求的数量,`add_item`方返回`False`。这个BUG在于,即使库存不足,该方法也不会减少商品在购物车中的库存数量。这意味着,用户连续尝试添加同一个商品,即使库存不足,购物车中的库存数量也不会更新,从而导致库存信息的错误。

修改后的代码及解释

下面是修改后的代码,包括对BUG的修复:

python

class ShoppingCart:

def __init__(self):

self.items = {}

def add_item(self, item_id, quantity):

if item_id in self.items:

if self.items[item_id]['quantity'] >= quantity:

self.items[item_id]['quantity'] -= quantity

return True

else:

# 库存不足,不减少购物车中的库存数量

return False

else:

# 商品不存在于购物车中,不执行任何操作

return False

# 示例

cart = ShoppingCart()

cart.add_item('item1', 5) # 假设商品'item1'的库存是10

print(cart.add_item('item1', 3)) # 应该返回True

print(cart.add_item('item1', 15)) # 应该返回False

# 检查购物车中的库存数量

print(cart.items) # 输出应该显示{'item1': {'quantity': 7}}

在修改后的代码中,我们保留了原有的逻辑,即当库存数量不足以满足用户请求时,方法返回`False`。我们确保了当库存足够时,购物车中的库存数量会相应减少。这样,即使用户连续尝试添加同一个商品,库存信息也会得到正确的更新。

通过上述面试官不仅考察了者对代码逻辑的理解,还考察了其对业务规则的把握。在解决这类时,者需要能够清晰地识别所在,并给出合理的解决方案。面试官也会关注者的代码风格和代码的可读性,这些都是计算机专业面试中非常重要的方面。