0

だから、私のコードを説明するには - >

タイプ Employee の抽象クラスがあり、名前などの独自のフィールドがあります。

次に、Employee から拡張されたクラス Admin、Sales、および Manager があります。これらにはそれぞれ、admin の fixedBonus、Sales の percentBonus、マネージャーの従業員の配列リストなどの独自のフィールドがあります。

次に、Employees の ArrayList を含むドライバー クラスを作成します。これには、Admins、Sales、および Managers が含まれます。

私の問題は、Arraylist 要素で発生します。ArrayList から Admin クラスの従業員を取得し、ゲッター getFixedBonus() を使用して fixedBonus フィールドにアクセスしようとしたとします。継承されたものだけでなく、クラス固有のゲッターとセッターを使用できるようにするにはどうすればよいですか? 問題が発生しているコードのコピーを次に示します。具体的には、問題は switch ステートメントのケース 5 と 6 にあります。

private Employee editAnEmployee(Employee emp) {
    boolean exit = false;
    boolean validChoice = false;
    int choice = 0;
    Boolean sure = false; // used in the switch statement
    String newName;
    double newValue;
    Employee admin = new AdminWorker("0", "0", 0, 0, 0); // these are used
    Employee sales = new SalesWorker("0", "0", 0, 0, 0, 0); // to check the
    Employee manager = new Manager("0", "0", 0, 0); // class of the
                                                    // employee
                                                    // being sent
                                                    // in, to edit
                                                    // the fields
                                                    // the employee
                                                    // may hold
    do {
        do {
            validChoice = true;
            emp.toString();
            StdOut.println("Which field would you like to edit?");
            StdOut.println("1) First Name.");
            StdOut.println("2) last Name.");
            StdOut.println("3) Hourly rate.");
            StdOut.println("4) Hours worked in the last week.");
            if (emp.getClass().equals(admin)) {
                StdOut.println("5) Fixed Bonus.");
            } else if (emp.getClass().equals(sales)) {
                StdOut.println("5) Percentage bonus on sales.");
                StdOut.println("6) Value of sales made in the last week.");
            } else if (emp.getClass().equals(manager)) {
                StdOut.println("5) Department.");
            }
            StdOut.println("0) Exit.");
            choice = StdIn.readInt();
            if (choice < 0 || choice > 4) {
                if (emp.getClass().equals(admin) && choice <= 5) {
                    validChoice = true;
                } else if (emp.getClass().equals(sales) && choice <= 6) {
                    validChoice = true;
                } else if (emp.getClass().equals(manager)) {
                    validChoice = true;
                } else {
                    StdOut.println("You entered an invalid number! Try Again...");
                    validChoice = false;
                }
            }
        } while (!validChoice);
        switch (choice) {
        case 0:
            StdOut.println("You are now exiting...");
            exit = true;
            break;
        case 1:
            StdOut.println("The employees first name is: " + emp.getFirstName() + ". What would you like the first name to be now?");
            newName = StdIn.readString();
            StdOut.println("Are you sure you want to change " + emp.getFirstName() + " to " + newName +"?(y/n)");
            sure = ynChoice();
            if(sure) emp.setFirstName(newName);
            break;
        case 2:
            StdOut.println("The employees last name is: " + emp.getLastName() + ". What would you like the last name to be now?");
            newName = StdIn.readString();
            StdOut.println("Are you sure you want to change " + emp.getLastName() + " to " + newName +"?(y/n)");
            sure = ynChoice();
            if(sure) emp.setLastName(newName);
            break;
        case 3:
            StdOut.println("The employees hourly rate is: " + emp.getHourlyRate() + ". What would you like the hourly rate to be now?");
            newValue = StdIn.readDouble();
            StdOut.println("Are you sure you want to change " + emp.getHourlyRate() + " to " + newValue +"?(y/n)");
            sure = ynChoice();
            if(sure) emp.setHourlyRate(newValue);
            break;
        case 4:
            StdOut.println("The employee has worked: " + emp.getHoursWorked() + " hours in the last week. What would you like that to be now?");
            newValue = StdIn.readDouble();
            StdOut.println("Are you sure you want to change " + emp.getHoursWorked() + " to " + newValue +"?(y/n)");
            sure = ynChoice();
            if(sure) emp.setHoursWorked(newValue);
            break;
        case 5:
            if (emp.getClass().equals(admin)) {
                StdOut.println("The employees fixed bonus is: " + emp.getFixedBonus() + ". What would you like that to be now?");
                newValue = StdIn.readDouble();
                StdOut.println("Are you sure you want to change " + emp.getFixedBonus() + " to " + newValue +"?(y/n)");
                sure = ynChoice();
                if(sure) emp.setHoursWorked(newValue);
            } else if (emp.getClass().equals(sales)) {
                StdOut.println("The employees percentage bonus is: " + emp.getPercentageBonus() + ". What would you like that to be now?");
                newValue = StdIn.readDouble();
                StdOut.println("Are you sure you want to change " + emp.getPercentageBonus() + " to " + newValue +"?(y/n)");
                sure = ynChoice();
                if(sure) emp.setHoursWorked(newValue);
            } else if (emp.getClass().equals(manager)) {
                StdOut.println("The employees fixed bonus is: " + emp.getFixedBonus() + ". What would you like that to be now?");
                newValue = StdIn.readDouble();
                StdOut.println("Are you sure you want to change " + emp.getFixedBonus() + " to " + newValue +"?(y/n)");
                sure = ynChoice();
                if(sure) emp.setHoursWorked(newValue);
            }
            break;
        case 6:
            StdOut.println("The employees sales in the last week are: " + emp.getLastWeeksSales() + ". What would you like that to be now?");
            newValue = StdIn.readDouble();
            StdOut.println("Are you sure you want to change " + emp.getLastWeeksSales() + " to " + newValue +"?(y/n)");
            sure = ynChoice();
            if(sure) emp.setHoursWorked(newValue);
            break;
        }
    } while (!exit);
    return emp;
}
4

3 に答える 3

2

getFixedBonus()およびgetPercentBonus()メソッドを従業員に追加し、それぞれが 0 を返すようにする必要があります。その後、他のワーカー クラスがそれらをオーバーライドできます。

実際には、これを にしてinterface、すべてのクラス ( を含むEmployee) に を実装させることをお勧めしinterfaceます。extend次に、インターフェイスを実装するがデフォルト値を提供するbaseクラスを作成できます。

public static interface Staff {
  public double getFixedBonus ();
  public double getPercentageBonus ();
  public List<Staff> getMinions();
}

// Base class - no bonus by default.
public static class Worker implements Staff {

  @Override
  public double getFixedBonus() {
    return 0;
  }

  @Override
  public double getPercentageBonus() {
    return 0;
  }

  @Override
  public List<Staff> getMinions() {
    return Collections.EMPTY_LIST;
  }

}

public static class Administrator extends Worker implements Staff {
  double fixedBonus = 2.13;

  @Override
  public double getFixedBonus() {
    return fixedBonus;
  }

}

public static class Salesperson extends Worker implements Staff {
  double percentageBonus = 3.14;

  @Override
  public double getPercentageBonus() {
    return percentageBonus;
  }

}

public static class Manager extends Worker implements Staff {
  List<Staff> minions;

  @Override
  public List<Staff> getMinions() {
    return minions;
  }
}

instanceof使用とキャストの暗黒面に誘惑されないでください。あなたは自分の決断を後悔するでしょう。oop の全体的なポイントは、作業しているオブジェクトの種類がわからない場合でも、それらに取り組むことができるということです。

于 2013-04-16T22:56:00.490 に答える
0

クライアント コードをよりシンプルにするオプション(よりクリーン - instanceof とキャストが醜い)はEmployee、の特定のメソッドごとにデフォルトの実装を宣言することです。fixedBonus()EmployeeSales

于 2013-04-16T23:20:21.950 に答える