背景
在计算机专业的面试中,面试官经常会提出一些与实际业务相关的编程以考察者的实际编程能力和解决能力。是一个常见的业务上BUG一条我们将对其进行深入分析和解答。
:
假设有一个在线书店的订单系统,系统中有类和函数:
java
class Order {
private int orderId;
private String customerName;
private List
books;
public Order(int orderId, String customerName, List books) {
this.orderId = orderId;
this.customerName = customerName;
this.books = books;
}
public int getTotalPrice() {
int totalPrice = 0;
for (Book book : books) {
totalPrice += book.getPrice();
}
return totalPrice;
}
}
class Book {
private String title;
private double price;
public Book(String title, double price) {
this.title = title;
this.price = price;
}
public double getPrice() {
return price;
}
}
public class OrderSystem {
public static void main(String[] args) {
List books = new ArrayList<>();
books.add(new Book("Java Programming", 49.99));
books.add(new Book("Python Basics", 39.99));
books.add(new Book("C++ Primer", 59.99));
Order order = new Order(12345, "John Doe", books);
System.out.println("Total Price: $" + order.getTotalPrice());
}
}
在上述代码中,有一个潜在的业务逻辑错误,请找出并解释这个。
分析
在上述代码中,`Order` 类负责创建订单,并计算订单中所有书籍的总价。`Book` 类代表一本书,包含书名和价格。`OrderSystem` 类中的 `main` 方法创建了一个订单,并打印出总价格。
潜在的业务逻辑错误可能在于订单价格的计算。在 `Order` 类的 `getTotalPrice` 方法中,它通过遍历 `books` 列表并累加每本书的价格来计算总价。这里存在一个订单中的书籍列表为空,即没有添加任何书籍, `getTotalPrice` 方法将返回0,这在业务逻辑上可能是不正确的。
解答
为了修复这个我们需要确保在订单中没有书籍时,`getTotalPrice` 方法能够返回一个合理的默认值。是修改后的代码:
java
class Order {
private int orderId;
private String customerName;
private List books;
public Order(int orderId, String customerName, List books) {
this.orderId = orderId;
this.customerName = customerName;
this.books = books;
}
public int getTotalPrice() {
if (books == null || books.isEmpty()) {
// 返回默认价格,0或-1,取决于业务需求
return 0;
}
int totalPrice = 0;
for (Book book : books) {
totalPrice += book.getPrice();
}
return totalPrice;
}
}
class Book {
private String title;
private double price;
public Book(String title, double price) {
this.title = title;
this.price = price;
}
public double getPrice() {
return price;
}
}
public class OrderSystem {
public static void main(String[] args) {
List books = new ArrayList<>();
books.add(new Book("Java Programming", 49.99));
books.add(new Book("Python Basics", 39.99));
books.add(new Book("C++ Primer", 59.99));
Order order = new Order(12345, "John Doe", books);
System.out.println("Total Price: $" + order.getTotalPrice());
}
}
在这个修改中,我们在 `getTotalPrice` 方法中添加了一个检查,以确保 `books` 列表不为空。列表为空,我们返回一个默认值,这里选择返回0。这样,即使订单中没有书籍,用户也会得到一个预期的价格。
在解决这类时,关键是要理解业务逻辑,并确保代码能够正确地反映这些逻辑。通过仔细检查代码中的每个部分,我们可以发现并修复潜在的错误,从而提高系统的可靠性和用户体验。在面试中,这种能力是非常重要的,因为它展示了者对业务逻辑的理解和编程实践。
还没有评论呢,快来抢沙发~