计算机专业面试题：如何在Java中解决一个常见的业务逻辑BUG？

义展示内容 2025年02月7日

首页展示内容正文

文章详情

背景介绍

在计算机专业的工作中，调试和修复BUG是家常便饭。一个典型的场景是处理业务逻辑时出现的BUG，这些BUG往往与数据的一致性和正确性直接相关。是一个在Java中处理业务逻辑BUG的例子。

假设我们正在开发一个在线订单系统，系统中有一个方法用来计算订单的总金额。这个方法接收一个订单对象作为参数，该对象包含一个订单项列表，每个订单项包含商品的价格和数量。我们的目标是计算并返回订单的总金额。

java

public class Order {

private List

orderItems;
public double getTotalAmount() {
double total = 0;
for (OrderItem item : orderItems) {
total += item.getPrice() * item.getQuantity();
}
return total;
}
}
public class OrderItem {
private double price;
private int quantity;
public OrderItem(double price, int quantity) {
this.price = price;
this.quantity = quantity;
}
public double getPrice() {
return price;
}
public int getQuantity() {
return quantity;
}
}

BUG现象

在实际使用中，我们发现订单的总金额计算结果是不正确的。有一个订单包含订单项：
– 商品1：价格10，数量2
– 商品2：价格20，数量3
按照预期，总金额应该是70。计算结果却是60。

分析BUG原因

通过检查代码，我们发现BUG的原因可能在于`OrderItem`类的`price`和`quantity`字段可能不是的数值，而是通过某些外部操作得到的中间值。
假设`OrderItem`类的`price`和`quantity`字段在创建对象时被正确赋值，但在某个时刻被外部代码修改了，而我们的`getTotalAmount`方法没有考虑到这种情况。

修复BUG的方法

为了修复这个BUG，我们需要确保在计算总金额时使用的是订单项的最新数值。是修改后的代码：
java
public class Order {
private List orderItems;
public double getTotalAmount() {
double total = 0;
for (OrderItem item : orderItems) {
// 我们确保使用的是订单项的最新数值
total += item.getPrice() * item.getQuantity();
}
return total;
}
}
public class OrderItem {
private double price;
private int quantity;
public OrderItem(double price, int quantity) {
this.price = price;
this.quantity = quantity;
}
public synchronized double getPrice() {
return price;
}
public synchronized int getQuantity() {
return quantity;
}
// 假设这个方法是外部调用用来更新价格
public synchronized void setPrice(double price) {
this.price = price;
}
// 假设这个方法是外部调用用来更新数量
public synchronized void setQuantity(int quantity) {
this.quantity = quantity;
}
}