Java中calculateTotalPay与calculateAveragePay方法空指针异常求助

数据老陈

2026-7-19

问题原因与修复方案

核心问题：成员变量未正确初始化

你的java.lang.NullPointerException根源是类成员变量weeklyHours从未被赋值，始终为null。在构造函数中，你创建的是局部变量weeklyHours，完全没有对类的成员变量进行初始化：

// 错误写法：仅创建了局部变量，和类成员变量无关
double[] weeklyHours = new double[DEFAULT_WEEKS];

修复步骤

1. 修正构造函数，初始化成员变量

去掉构造函数里的局部变量声明，直接给类的weeklyHours成员赋值：

public Employee(String name, double salary) {
    this.name = name;
    this.salary = salary;
    // 直接给类成员变量赋值，而非创建新的局部变量
    this.weeklyHours = new double[DEFAULT_WEEKS];
}

public Employee(String name, double salary, int numWeeks) {
    this.name = name;
    this.salary = salary;
    this.weeklyHours = new double[numWeeks];
}

2. 修复`calculateTotalPay`方法的逻辑错误

原方法中循环被注释，导致index变量未定义，且无法计算所有周的工时总和。恢复循环逻辑，同时修正总和的变量类型（避免整数截断）：

public double calculateTotalPay() {
    double totalAmount = 0; // 改为double类型，匹配weeklyHours的元素类型
    for (int index = 0; index < weeklyHours.length; index++) {
        totalAmount += weeklyHours[index];
    }
    return totalAmount;
}

3. 完善`calculateAveragePay`方法

取消注释调用calculateTotalPay()的代码，同时可添加空值/空数组判断，避免后续异常：

public double calculateAveragePay() {
    // 防御性判断，避免空指针或除以0异常
    if (weeklyHours == null || weeklyHours.length == 0) {
        return 0; // 也可根据业务需求抛出异常
    }
    double totalPay = calculateTotalPay();
    return totalPay / weeklyHours.length;
}