计算机专业面试题：业务上BUG一条及解答

在一家电商平台上，用户可以购买商品。平台有一个订单管理系统，用于处理用户的订单。是一个简化版的订单管理系统的一部分代码，用于处理订单的创建和更新。请找出代码中存在的业务逻辑BUG，并解释原因。

python

class Order:

def __init__(self, order_id, user_id, product_id, quantity, status='pending'):

self.order_id = order_id

self.user_id = user_id

self.product_id = product_id

self.quantity = quantity

self.status = status

class OrderManager:

def __init__(self):

self.orders = []

def create_order(self, order_id, user_id, product_id, quantity):

for order in self.orders:

if order.order_id == order_id:

raise ValueError("Order ID already exists.")

new_order = Order(order_id, user_id, product_id, quantity)

self.orders.append(new_order)

def update_order_status(self, order_id, new_status):

for order in self.orders:

if order.order_id == order_id:

order.status = new_status

return

raise ValueError("Order ID not found.")

# 示例使用

order_manager = OrderManager()

order_manager.create_order(1, 100, 200, 2)

order_manager.update_order_status(1, 'shipped')

分析

在这个代码示例中，我们需要分析可能存在的业务逻辑BUG。

1. 当尝试创建一个已存在的订单ID时，系统会抛出一个`ValueError`异常，这是合理的。

2. 当尝试更新一个订单的状态时，订单ID不存在，系统也会抛出一个`ValueError`异常。

存在一个潜在的业务逻辑BUG：

– 在尝试更新订单状态时，输入了一个不正确的订单ID，系统不会提供任何反馈，用户可能会认为操作成功。

BUG解释

在`update_order_status`方法中，我们遍历`orders`列表来查找匹配的订单ID。找到了匹配的订单，我们更新其状态并返回。没有找到，我们抛出一个`ValueError`异常。

在于，用户输入了一个不存在的订单ID，系统会静默地忽略这个操作，而不是提供任何反馈。这在业务逻辑上是不合理的，因为用户可能需要知道他们的操作是否成功。

解答

为了修复这个BUG，我们可以修改`update_order_status`方法，使其在找不到订单时返回一个明确的提示，而不是抛出异常。是修改后的代码：

python

class OrderManager:

# … (其他方法保持不变)

def update_order_status(self, order_id, new_status):

for order in self.orders:

if order.order_id == order_id:

order.status = new_status

return True # 返回True表示操作成功

return False # 返回False表示操作失败，订单ID不存在

# 示例使用

order_manager = OrderManager()

order_manager.create_order(1, 100, 200, 2)

success = order_manager.update_order_status(1, 'shipped')

if success:

print("Order status updated successfully.")

else:

print("Failed to update order status. Order ID not found.")

# 尝试更新一个不存在的订单ID

success = order_manager.update_order_status(999, 'shipped')

if success:

print("Order status updated successfully.")

else:

print("Failed to update order status. Order ID not found.")

通过这种，用户可以明确地知道他们的操作是否成功，从而避免了潜在的业务逻辑错误。