构造函数不创建对象-Java

Question

大约一个星期以来，我一直在自己的一个程序中解决此问题，但我不知道自己在做什么错。 对于此分配，我基本上是用3种不同类型的雇员（佣金，薪水，小时数）组成一个Employee对象数组。 这三个子类都扩展了Employee类，它是抽象的。

这是我的代码，它为每种类类型调用构造函数：

    public boolean addEmployee(int empType, String first, String last, char mi, char gen, int empNum, boolean full, double amount)
    {
    employeeCount += 1;

    if (employeeCount > EMPLOYEES_MAX) {
        System.out.println("ERROR: ARRAY FULL. UNABLE TO ADD NEW EMPLOYEE");
        return false;
    }

    else {
        switch (empType) {
            case 1:
            employees[employeeCount] = new HourlyEmployee(first, last, mi, gen, empNum, full, amount);
            break;

            case 2:
            employees[employeeCount] = new SalaryEmployee(first, last, mi, gen, empNum, full, amount);
            break;

            case 3:
            employees[employeeCount] = new CommissionEmployee(first, last, mi, gen, empNum, full, amount);
            break;
        }
        return true;
    }
}

这是薪水员工的构造函数，其他两个与此基本相同。

public class SalaryEmployee extends Employee
{
private double salary;

public SalaryEmployee(String first, String last, char midInit, char gen, int empNum, boolean full, double sal)
{
    super(first, last, midInit, gen, empNum, full);
    salary = sal;
}

我编写的由该构造函数调用的超级构造函数如下：

abstract class Employee
{
protected String firstName;
protected String lastName;
protected char middleInitial;
protected boolean fullTime;
protected char gender;
protected int employeeNum;

public Employee(String first, String last, char midInit, char gen, int empNum, boolean full)
{
    firstName = first;
    lastName = last;
    middleInitial = midInit;
    employeeNum = empNum;
    fullTime = full;

    switch (gen) {
        case 'M': case 'F': 
        case 'm': case 'f':
        gender = gen;
        break;

        default:
        gender = 'F';
    }
}

就像我之前说过的，我不知道为什么它不创建应有的对象。 每当我打印输出数组或对其进行排序时，总是得到null或null指针异常。

任何帮助将不胜感激，非常感谢大家！

编辑：

好的，这里是我的Employee数组初始化的地方。

public class EmployeeList
{
private final int EMPLOYEES_MAX = 50;
private Employee employees[];
private int employeeCount;

public EmployeeList()
{
    employees = new Employee[EMPLOYEES_MAX];
    employeeCount = 0;
}

当我运行它时，由于未运行默认方法，就像仍在构造函数中运行一样，由于某种奇怪的原因，我只会得到null。 继承人的输出：

从以下选项中选择：1添加员工2按类型处理员工子菜单3删除员工4排序员工5计算每周支付6计算奖金7年度薪水8重设周0退出选项：1

1. 每小时
2. 薪水
3. 佣金输入选项：1

输入姓氏：Doe

输入名字：Jane

输入中间名首字母：M

输入性别：F

输入员工编号：1

全职？ （y / n）：y

输入工资：14

员工已添加到列表

空值

主菜单

null来自我的主类，该主类调用listAll（）方法，该方法应列出数组中的所有内容。 但只会返回null。

再次编辑：

因此，这里是应该列出所有内容的方法。

    public void listAll()
{
    for(int x = 0; x < employeeCount; x++)
        System.out.println(employees[x]);
}

谢谢你的帮助！

Answer 1

第一个问题

您传入int empType ，然后打开employeeType

第二个问题

您传递了int empType ，但您的开关中的所有情况都是char ，例如case '1': int empType

编辑：您已编辑问题，以解决某些问题>：-/

第三个问题

在您的listAll()方法中，您试图打印不具有toString()方法的对象。 给您的抽象Employee类一个toString()方法，如下所示。

public String toString() {
    return lastName + ", " + firstName + " " + middleInitial;
}

现在，假设您希望子类具有更详细的表示形式，例如“ Smith，John C.每小时。 ”您可以通过向每个子类中添加ANOTHER toString方法来做到这一点，但是您必须重用已经存在的部分写了！ 例如，这将在您的每小时员工班级中进行。

public String toString() {
    return super.toString() + " is hourly.";
}

Answer 2

我想到了几件事情，但其中大多数都考虑了总体设计，而不是您的特定问题。

一对一

您的代码当前包含一个错误的外观。 您正在将employeeCount初始化为0但是在第一次插入之前将其递增为1 。 因此， 第一名员工进入了employees[1]的第二个阵列插槽。

但是您的输出循环是正确的，并且在第一次插入之后，它仅输出employees[0] ，该值仍然为null 。 只需移动employeeCount += 1; 到方法的结尾，您就可以了。

（在不清楚的情况下：Java计算从0到length-1数组索引。）

IndexOutOfBounds

还有另一个错误。 您正在比较employeeCount > EMPLOYEES_MAX 。 如果employeeCount = EMPLOYEES_MAX ，则它通过了测试，但是在employees[EMPLOYEES_MAX]处插入将失败，因为它超出了数组的范围。

您必须检查employeeCount >= EMPLOYEES_MAX 。 这似乎很违反直觉，因为在您的方法中，您将其用作“此插入之后的员工数” 。 如果从上方包括我的更正，则解释为“已插入的员工数” 。

Answer 3

将case '1'更改为case 1 ，依此类推，当您传入int时，显然不想打开char值。

Answer 4

到目前为止，我已经加入了您所写的内容，并且看起来行之有效（为提高代码可读性而引入了常量）：

public class EmployeeList {

public static final int EMP_TYPE_HOURLY = 1;
public static final int EMP_TYPE_SALARY = 2;
public static final int EMP_TYPE_COMMISSION = 3;
public static final char EMP_GEN_MALE = 'M';
public static final char EMP_GEN_FEMALE = 'F';

private int employeeCount = 0;
private static final int EMPLOYEES_MAX = 5;
private Employee[] employees = new Employee[EMPLOYEES_MAX];

public boolean addEmployee(int empType, String first, String last, char mi, char gen, int empNum, boolean full, double amount) {
    employeeCount += 1;

    if (employeeCount > EMPLOYEES_MAX) {
        System.out.println("ERROR: ARRAY FULL. UNABLE TO ADD NEW EMPLOYEE");
        return false;
    } else {
        switch (empType) {
            case EMP_TYPE_HOURLY:
                employees[employeeCount] = new HourlyEmployee(first, last, mi, gen, empNum, full, amount);
                break;

            case EMP_TYPE_SALARY:
                employees[employeeCount] = new SalaryEmployee(first, last, mi, gen, empNum, full, amount);
                break;

            case EMP_TYPE_COMMISSION:
                employees[employeeCount] = new CommissionEmployee(first, last, mi, gen, empNum, full, amount);
                break;
        }
        return true;
    }
}

public Employee[] getEmployees() {
    return employees;
}

public static void main(String[] args) {
    EmployeeList empList = new EmployeeList();

    empList.addEmployee(EMP_TYPE_HOURLY, null, null, 'A', EMP_GEN_MALE, 555, true, 100.0);
    empList.addEmployee(EMP_TYPE_HOURLY, null, null, 'A', EMP_GEN_FEMALE, 555, true, 100.0);
    empList.addEmployee(EMP_TYPE_SALARY, null, null, 'A', EMP_GEN_FEMALE, 555, true, 100.0);
    empList.addEmployee(EMP_TYPE_COMMISSION, null, null, 'A', EMP_GEN_FEMALE, 555, true, 100.0);

    for (int i = 0; i < EMPLOYEES_MAX; i++) {
        System.out.println("" + empList.getEmployees()[i]);
    }
}

}