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

假设你正在开发一个在线书店系统，该系统允许用户浏览和购买书籍。在用户购买书籍的过程中，系统出现了一个BUG，导致部分用户在支付成功后无法收到购买确认邮件。是系统相关的代码片段，请你找出BUG所在，并解释原因。

python

class Order:

def __init__(self, user_id, book_id, quantity):

self.user_id = user_id

self.book_id = book_id

self.quantity = quantity

self.status = 'pending'

def complete_order(self):

self.status = 'completed'

self.send_confirmation_email()

def send_confirmation_email(self):

email_service = EmailService()

email_service.send_email(self.user_id, f"Your order for {self.quantity} books is completed.")

class EmailService:

def send_email(self, recipient_id, message):

# 假设这里是一个模拟发送邮件的函数

print(f"Sending email to {recipient_id}: {message}")

# 发送邮件失败，这里不抛出异常，而是静默处理

if not self.mock_email_success(recipient_id):

print("Failed to send email, retrying…")

self.send_email(recipient_id, message) # 递归调用发送邮件

def mock_email_success(self, recipient_id):

# 模拟邮件发送是否成功的函数

# 假设只有部分用户ID会导致邮件发送失败

return recipient_id % 2 == 0 # recipient_id是偶数，模拟邮件发送失败

# 测试代码

order = Order(1, 101, 1)

order.complete_order()

分析

在上面的代码中，我们定义了`Order`类和`EmailService`类。`Order`类有一个方法`complete_order`，该方将订单状态设置为已完成，并调用`send_confirmation_email`方法发送确认邮件。`EmailService`类中的`send_email`方法负责发送邮件，邮件发送失败，它会尝试重新发送邮件。

BUG分析

在这个中，BUG出`EmailService`类的`send_email`方法中。具体来说，`mock_email_success`方法模拟邮件发送是否成功，它返回一个布尔值。邮件发送失败，`send_email`方递归调用自己，试图重新发送邮件。

在于，邮件发送失败，`send_email`方无限递归调用自己，因为`mock_email_success`方法在每次调用时都有可能返回`False`，从而导致无限循环。

BUG修复

为了修复这个BUG，我们需要避免无限递归。一种方法是引入一个递归深度限制，当达到最大递归深度时，停止递归并记录错误。

是修复后的代码：

python

class EmailService:

def send_email(self, recipient_id, message, max_retries=3):

# 假设这里是一个模拟发送邮件的函数

print(f"Sending email to {recipient_id}: {message}")

if not self.mock_email_success(recipient_id):

print("Failed to send email, retrying…")

if max_retries > 0:

self.send_email(recipient_id, message, max_retries – 1)

else:

print("Maximum retries reached, email not sent.")

def mock_email_success(self, recipient_id):

# 模拟邮件发送是否成功的函数

# 假设只有部分用户ID会导致邮件发送失败

return recipient_id % 2 == 0 # recipient_id是偶数，模拟邮件发送失败

在这个修复版本中，我们为`send_email`方法添加了一个`max_retries`参数，用来限制递归调用的次数。达到最大重试次数，方法将停止递归，并打印一条消息表示邮件未能发送。

通过这个面试我们学习了如何在代码中处理潜在的无穷递归以及如何通过限制递归深度来避免程序崩溃。这是一个典型的业务上BUG，需要深入理解代码逻辑，找出并修复潜在的错误。