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

在一家电商公司进行面试时，面试官提出了

假设你正在开发一个电商平台的订单管理系统。系统中有一个功能是“订单详情”，用户可以通过这个功能查看订单的详细信息。是一个简化版的代码片段，用于展示订单详情的获取逻辑。请指出代码中存在的业务上BUG，并解释原因。

python

class Order:

def __init__(self, order_id, customer_id, items):

self.order_id = order_id

self.customer_id = customer_id

self.items = items

class OrderService:

def __init__(self):

self.orders = []

def add_order(self, order):

self.orders.append(order)

def get_order_details(self, order_id):

for order in self.orders:

if order.order_id == order_id:

return order

return None

# 假设有一个订单服务实例

order_service = OrderService()

# 添加一些订单

order_service.add_order(Order(1, 100, ["Laptop", "Mouse"]))

order_service.add_order(Order(2, 101, ["Smartphone", "Charger"]))

# 获取订单详情

order_details = order_service.get_order_details(1)

if order_details:

print(f"Order ID: {order_details.order_id}, Customer ID: {order_details.customer_id}, Items: {order_details.items}")

else:

print("Order not found.")

分析

在上述代码中，存在一个明显的业务上BUG。这个BUG可能会导致用户在尝试获取某个订单的详情时，即使该订单确实存在于系统中，也可能得到一个错误的信息。

BUG分析

BUG出`get_order_details`方法中。该方法遍历`orders`列表，查找与`order_id`匹配的订单。找到匹配的订单，它将返回该订单对象。订单不存在，该方法将返回`None`。

在于，即使订单不存在，用户在调用`get_order_details`方法时，程序仍然会执行`if order_details:`条件判断。`order_details`是`None`，这个条件判断将会失败，会打印出“Order not found.”的消息。

这个BUG的在于它没有考虑到`order_details`可能为`None`的情况。当`order_details`为`None`时，直接返回`None`或者进行相应的错误处理是非常重要的。

BUG解答

为了修复这个BUG，我们需要在`get_order_details`方法中添加对`order_details`是否为`None`的检查。`order_details`为`None`，我们可以抛出一个异常或者返回一个错误信息，而不是直接返回`None`。

是修复后的代码：

python

class OrderService:

# … 其他方法保持不变 …

def get_order_details(self, order_id):

for order in self.orders:

if order.order_id == order_id:

return order

raise ValueError(f"Order with ID {order_id} not found.")

# 使用修复后的方法

try:

order_details = order_service.get_order_details(1)

if order_details:

print(f"Order ID: {order_details.order_id}, Customer ID: {order_details.customer_id}, Items: {order_details.items}")

else:

print("Order not found.")

except ValueError as e:

print(e)

在这个修复版本中，订单不存在，`get_order_details`方抛出一个`ValueError`异常，在调用该方法的地方使用`try-except`块来捕获这个异常，并打印出相应的错误信息。

通过这种，我们确保了当用户尝试获取不存在的订单详情时，程序能够给出明确的，而不是返回一个`None`值，这有助于提高用户体验和程序的健壮性。