本文详解如何在 java 的 `basket` 类中正确实现两个静态方法：计算所有商品的平均单价（总金额 ÷ 商品总数）和每篮子的平均价格（总金额 ÷ 篮子总数），并修复原代码中变量命名、逻辑错误与类型安全问题。
在面向对象编程中，静态方法常用于聚合统计（如全局平均值），但其设计需兼顾线程安全性、数据一致性与语义清晰性。原代码中存在多个关键问题：
- allprice / allcount / averagebasket 命名不符合 Java 命名规范（应为 allPrice, allCount, basketCount）；
- average() 方法未返回值且未处理除零异常；
- add() 方法中误将单个商品价格 price 直接累加到 allprice，而实际应累加 price * count（已正确），但 allCount 仅在 increaseTovar(1) 中递增 1 次——这隐含假设每次 add() 只添加 1 种商品的 1 件，与方法签名 add(String, int, int, double) 中的 count 参数矛盾；
- 缺少对空篮子或零商品数的保护，直接整数除法会导致 ArithmeticException 或精度丢失。
✅ 正确做法是：
-
统一静态计数器命名与职责
private static int basketCount = 0; // 创建的 Basket 实例总数 private static int allItemCount = 0; // 所有篮子中商品总件数（非种类数） private static int allPriceSum = 0; // 所有商品总价（单位：分/整数避免浮点误差）
-
提供类型安全、带防护的静态访问方法
public static double getAveragePricePerItem() { return allItemCount == 0 ? 0.0 : (double) allPriceSum / allItemCount; } public static double getAveragePricePerBasket() { return basketCount == 0 ? 0.0 : (double) allPriceSum / basketCount; } -
修正 add() 中的商品计数逻辑
原 add(...) 方法调用 increaseTovar(1) 仅增加 1 件，但参数 count 表示添加数量，应改为：public void add(String name, int price, int count, double weight) { // ... error checks ... increaseItemCount(count); // ✅ 传入实际件数 // ... 其他逻辑 ... allPriceSum += price * count; // ✅ 已正确 } public static void increaseItemCount(int itemCount) { allItemCount += itemCount; } -
在构造器中正确更新 basketCount
public Basket() { basketCount++; // ✅ 替换原 increaseCount(1) items = "Список товаров:"; this.limit = 1000000; } -
完整使用示例（Main.java）
public class Main { public static void main(String[] args) { Basket basket1 = new Basket(); basket1.add("Milk", 40, 2, 305.4); // 2件牛奶，共80元 Basket basket2 = new Basket(); basket2.add("Bread", 60, 1, 555.0); // 1件面包，60元 System.out.printf("平均单价: %.2f 元/件%n", Basket.getAveragePricePerItem()); // 140 / 3 ≈ 46.67 System.out.printf("平均每篮: %.2f 元/篮%n", Basket.getAveragePricePerBasket()); // 140 / 2 = 70.00 } }
⚠️ 注意事项：
- 静态字段在多线程环境下非线程安全，如需并发支持，应改用 AtomicInteger 或同步块；
- 金额运算建议使用 BigDecimal 或以“分”为单位的 long 类型，避免 double 浮点精度误差；
- getAveragePricePerItem() 和 getAveragePricePerBasket() 应始终返回 double，避免整数截断；
- 不要依赖 average() 这类无返回值、无参数的“计算型”静态方法——它破坏封装性且易被忽略调用。
通过以上重构，Basket 类的静态统计功能变得语义明确、健壮可靠，符合工业级 Java 编程实践。
