背景
在计算机专业的面试中，业务上BUG的考察是一项重要的环节。这类要求者不仅能够识别出还需要能够给出合理的解决方案。是一道常见的业务上BUG面试题，我们将对其进行详细解析。
假设有一个在线购物平台，用户可以在平台上购买商品。系统设计了一个“秒杀”活动，活动期间商品价格会大幅降低。现有一个需求，要求在活动期间，当商品库存不足时，能够立即通知库存管理模块减少库存，并给出库存不足的警告信息。
代码片段是负责处理秒杀活动的部分代码，但存在一个BUG，请找出这个BUG并给出修改。
java
public class SeckillService {
private int stock; // 商品库存
public SeckillService(int stock) {
this.stock = stock;
}
public synchronized boolean purchase() {
if (stock > 0) {
stock–;
System.out.println("购买成功，剩余库存：" + stock);
return true;
} else {
System.out.println("库存不足");
return false;
}
}
public synchronized void reduceStock() {
stock–;
}
}
分析
在这段代码中，`purchase()` 和 `reduceStock()` 方法都被声明为 `synchronized`，这意味着这两个方法在同一时刻只能由一个线程访问。存在一个BUG，即在 `reduceStock()` 方法中直接减少了库存，而没有进行库存检查。
BUG解释
当多个线程调用 `reduceStock()` 方法时，可能会导致情况：
1. 线程A进入 `reduceStock()` 方法，发现库存大于0，执行 `stock–` 操作。
2. 线程B也进入 `reduceStock()` 方法，执行 `stock–` 操作，库存可能已小于0，但由于 `purchase()` 方法已经获取了锁，线程B的 `stock–` 操作不会被阻塞。
3. 线程A执行完毕 `purchase()` 方法，释放锁。线程B继续执行 `reduceStock()` 方法，执行 `stock–` 操作，导致库存出现负数。
解决方案
为了解决这个我们需要在 `reduceStock()` 方法中加入库存检查。是修改后的代码：
java
public class SeckillService {
private int stock; // 商品库存
public SeckillService(int stock) {
this.stock = stock;
}
public synchronized boolean purchase() {
if (stock > 0) {
stock–;
System.out.println("购买成功，剩余库存：" + stock);
return true;
} else {
System.out.println("库存不足");
return false;
}
}
public synchronized void reduceStock() {
if (stock > 0) {
stock–;
} else {
System.out.println("库存不足，无法减少库存");
}
}
}
通过这种，当库存不足时，`reduceStock()` 方法将不会减少库存，并输出一条库存不足的警告信息。
在面试中，遇到业务上BUG的时，我们需要仔细分析找出BUG产生的原因，并提出合理的解决方案。通过解决这类可以展现出者对业务逻辑和代码实现的深入理解能力。在上述例子中，我们通过在 `reduceStock()` 方法中加入库存检查，成功地解决了BUG。在实际工作中，我们也应该注重代码质量和系统稳定性，确保业务逻辑的正确性和健壮性。
