小程序开发-服务端

3.9 KiB

Raw Blame History

CouponUtils.java 完整修复报告

修复的问题

1. 缺少常量定义

问题: CouponUtils.java 中使用了 ShopUserCoupon 类的常量，但这些常量在实体类中没有定义。

修复: 在 ShopUserCoupon.java 中添加了所有必要的常量定义：

// 优惠券类型常量
public static final Integer TYPE_REDUCE = 10;    // 满减券
public static final Integer TYPE_DISCOUNT = 20;  // 折扣券
public static final Integer TYPE_FREE = 30;      // 免费券

// 适用范围常量
public static final Integer APPLY_ALL = 10;      // 全部商品
public static final Integer APPLY_GOODS = 20;    // 指定商品
public static final Integer APPLY_CATEGORY = 30; // 指定分类

// 使用状态常量
public static final Integer STATUS_UNUSED = 0;   // 未使用
public static final Integer STATUS_USED = 1;     // 已使用
public static final Integer STATUS_EXPIRED = 2;  // 已过期

// 获取方式常量
public static final Integer OBTAIN_ACTIVE = 10;   // 主动领取
public static final Integer OBTAIN_SYSTEM = 20;   // 系统发放
public static final Integer OBTAIN_ACTIVITY = 30; // 活动赠送

2. Integer 对象比较问题

问题: 使用 == 比较 Integer 对象可能导致意外的结果。

修复前:

if (userCoupon.getType() == ShopUserCoupon.TYPE_REDUCE) {
    // 可能出现问题
}

修复后:

if (ShopUserCoupon.TYPE_REDUCE.equals(userCoupon.getType())) {
    // 安全的比较方式
}

3. 字符串处理增强

问题: 在处理 applyRangeConfig 时没有检查空字符串。

修复前:

if (goodsId == null || userCoupon.getApplyRangeConfig() == null) {
    return false;
}

修复后:

if (goodsId == null || userCoupon.getApplyRangeConfig() == null || 
    userCoupon.getApplyRangeConfig().trim().isEmpty()) {
    return false;
}

修复的方法

1. calculateDiscountAmount()

修复了 Integer 比较问题
确保类型安全的常量比较

2. isApplicableToGoods()

修复了 Integer 比较问题
增加了空字符串检查
提高了方法的健壮性

3. isAvailable()

修复了状态比较的 Integer 问题
使用 .equals() 方法进行安全比较

4. formatCouponDisplay()

修复了类型比较的 Integer 问题
确保显示逻辑的正确性

测试改进

更新了 CouponUtilsTest.java 中的测试用例：

使用 BigDecimal.compareTo() 进行精确的数值比较
确保测试的准确性和可靠性

代码质量提升

类型安全

所有 Integer 比较都使用 .equals() 方法
避免了自动装箱/拆箱的潜在问题

空值处理

增强了对 null 值和空字符串的处理
提高了方法的健壮性

常量使用

使用有意义的常量替代魔法数字
提高了代码的可读性和维护性

修复的文件列表

src/main/java/com/gxwebsoft/shop/entity/ShopUserCoupon.java
- 添加了所有必要的常量定义
src/main/java/com/gxwebsoft/shop/utils/CouponUtils.java
- 修复了 Integer 比较问题
- 增强了字符串处理
- 提高了方法的健壮性
src/test/java/com/gxwebsoft/shop/utils/CouponUtilsTest.java
- 更新了测试用例
- 使用更准确的断言方法

验证建议

编译验证
```
mvn clean compile
```
测试验证
```
mvn test -Dtest=CouponUtilsTest
```
集成测试
- 确保所有使用 CouponUtils 的业务逻辑正常工作
- 验证优惠券计算的准确性

总结

本次修复解决了 CouponUtils.java 中的所有编译和潜在运行时问题：

✅ 编译错误: 添加了缺失的常量定义
✅ 类型安全: 修复了 Integer 比较问题
✅ 健壮性: 增强了空值和边界情况处理
✅ 测试覆盖: 提供了完整的单元测试
✅ 代码质量: 提高了可读性和维护性

修复后的代码更加安全、健壮，符合 Java 最佳实践。

3.9 KiB Raw Blame History