Creating a Static Average Price Method

本文详解如何在 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 或精度丢失。

✅ 正确做法是:

  1. 统一静态计数器命名与职责

    private static int basketCount = 0;     // 创建的 Basket 实例总数
    private static int allItemCount = 0;     // 所有篮子中商品总件数(非种类数)
    private static int allPriceSum = 0;       // 所有商品总价(单位:分/整数避免浮点误差)
  2. 提供类型安全、带防护的静态访问方法

    public static double getAveragePricePerItem() {
        return allItemCount == 0 ? 0.0 : (double) allPriceSum / allItemCount;
    }
    
    public static double getAveragePricePerBasket() {
        return basketCount == 0 ? 0.0 : (double) allPriceSum / basketCo

    unt; }
  3. 修正 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;
    }
  4. 在构造器中正确更新 basketCount

    public Basket() {
        basketCount++; // ✅ 替换原 increaseCount(1)
        items = "Список товаров:";
        this.limit = 1000000;
    }
  5. 完整使用示例(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 编程实践。