计算机专业面试题：业务逻辑中的BUG分析与解决

假设你正在参与一个在线图书销售系统的开发，该系统允许用户浏览书籍、添加购物车、结账等。系统的一个功能是允许用户对已购书籍进行评价。是一个简化的代码片段，用于处理用户对书籍的评价逻辑：

python

def add_book_review(book_id, user_id, review_score):

# 假设有一个数据库函数来获取书籍信息

book_info = get_book_info(book_id)

# 假设有一个数据库函数来获取用户信息

user_info = get_user_info(user_id)

# 检查用户是否已购买该书籍

if book_id not in user_info['purchased_books']:

raise ValueError("User has not purchased this book.")

# 检查评分是否在合理范围内

if not 1 <= review_score <= 5:

raise ValueError("Review score must be between 1 and 5.")

# 添加评价到数据库

add_review_to_database(book_id, user_id, review_score)

return "Review added successfully."

# 示例函数（假设）

def get_book_info(book_id):

# 返回书籍信息

return {'book_id': book_id, 'title': 'Example Book'}

def get_user_info(user_id):

# 返回用户信息

return {'user_id': user_id, 'purchased_books': [1, 2, 3, 4, 5]}

def add_review_to_database(book_id, user_id, review_score):

# 添加评价到数据库的逻辑

pass

在这个代码片段中，存在一个潜在的BUG。请分析这个BUG，并给出修改后的代码。

BUG分析

在上述代码中，存在潜在BUG：

1. 硬编码的书籍ID列表：在`get_user_info`函数中，用户信息包含一个硬编码的书籍ID列表`[1, 2, 3, 4, 5]`。这假设所有用户都只购买了一组特定的书籍，这在现实世界中是不可能的。

2. 不合理的异常处理：当用户尝试对未购买的书籍进行评价时，系统会抛出一个`ValueError`。更做法可能是返回一个错误消息而不是抛出异常，这样前端可以更好地处理错误。

3. 评价添加逻辑缺失：`add_review_to_database`函数是一个占位符，没有实现具体的数据库操作逻辑。

修改后的代码

是针对上述BUG修改后的代码：

python

def add_book_review(book_id, user_id, review_score):

# 假设有一个数据库函数来获取书籍信息

book_info = get_book_info(book_id)

# 假设有一个数据库函数来获取用户信息

user_info = get_user_info(user_id)

# 检查用户是否已购买该书籍

if book_id not in user_info.get('purchased_books', []):

return "Error: User has not purchased this book."

# 检查评分是否在合理范围内

if not 1 <= review_score <= 5:

return "Error: Review score must be between 1 and 5."

# 添加评价到数据库

try:

add_review_to_database(book_id, user_id, review_score)

return "Review added successfully."

except Exception as e:

return f"Error: An error occurred while adding the review. {str(e)}"

# 示例函数（假设）

def get_book_info(book_id):

# 返回书籍信息

return {'book_id': book_id, 'title': 'Example Book'}

def get_user_info(user_id):

# 返回用户信息

return {'user_id': user_id, 'purchased_books': []}

def add_review_to_database(book_id, user_id, review_score):

# 添加评价到数据库的逻辑

# 假设数据库操作成功

pass

在这个修改后的版本中，我们做了改动：

1. 使用`user_info.get('purchased_books', [])`来获取用户购买书籍列表，用户信息中没有`purchased_books`键，则默认为一个空列表。

2. 将异常处理改为返回错误消息，而不是抛出异常。

3. 添加了异常处理逻辑到`add_review_to_database`函数的调用中，以捕获并返回可能的数据库操作错误。

这样修改后的代码更加健壮，能够更好地处理现实世界中的情况。