深入剖析计算机专业面试中常见的业务逻辑BUG及解决方案

义展示内容 2025年02月6日

文章详情

在计算机专业的面试中，业务逻辑BUG是一道常见的考察题目。这类不仅考验者对编程知识的掌握，还考察其对业务流程的理解和解决的能力。本文将针对一个典型的业务逻辑BUG进行深入剖析，并提供解决方案。

假设我们正在开发一个在线书店系统，系统需要实现用户购买书籍的功能。是一个简化的业务逻辑代码片段：

java

public class BookStore {

private List

books;
private List users;
public BookStore() {
books = new ArrayList<>();
users = new ArrayList<>();
}
public void purchaseBook(String userId, String bookId) {
User user = users.stream()
.filter(u -> u.getId().equals(userId))
.findFirst()
.orElse(null);
if (user == null) {
System.out.println("User not found.");
return;
}
Book book = books.stream()
.filter(b -> b.getId().equals(bookId))
.findFirst()
.orElse(null);
if (book == null) {
System.out.println("Book not found.");
return;
}
if (user.getBooks().contains(book)) {
System.out.println("User already owns this book.");
return;
}
user.getBooks().add(book);
System.out.println("Book purchased successfully.");
}
}

在这个代码片段中，我们有一个`BookStore`类，它有两个属性：`books`和`users`。`purchaseBook`方法用于处理用户购买书籍的业务逻辑。在于，当用户尝试购买一本已经拥有的书籍时，系统会输出“User already owns this book.”，但应该输出“Book purchased successfully.”。

分析

在这个中，业务逻辑BUG的根源在于对用户是否拥有某本书的判断。具体来说，`user.getBooks().contains(book)`这一行代码存在。该代码假设`user.getBooks()`返回的是一个`Book`对象的列表，但它返回的是一个`String`类型的列表，包含了书籍的ID。

解决方案

为了解决这个我们需要修改`purchaseBook`方法，使其能够正确地判断用户是否已经拥有某本书。是修改后的代码：
java
public class BookStore {
private List books;
private List users;
public BookStore() {
books = new ArrayList<>();
users = new ArrayList<>();
}
public void purchaseBook(String userId, String bookId) {
User user = users.stream()
.filter(u -> u.getId().equals(userId))
.findFirst()
.orElse(null);
if (user == null) {
System.out.println("User not found.");
return;
}
Book book = books.stream()
.filter(b -> b.getId().equals(bookId))
.findFirst()
.orElse(null);
if (book == null) {
System.out.println("Book not found.");
return;
}
if (user.getBooks().stream().anyMatch(b -> b.getId().equals(bookId))) {
System.out.println("User already owns this book.");
return;
}
user.getBooks().add(book);
System.out.println("Book purchased successfully.");
}
}