1
//TestEmployeesProgram driver with menu & object array.
import java.util.*;
public class TestEmployeesProgram {

public static Scanner console = new Scanner(System.in);

public static void main(String[] args)
{
    final int MAX = 7;

    Employee employee[] = new Employee[MAX];

    int choice,k;
    String name;
    boolean notFound;

    employee[1] = new Manager("Joe Bloggs","gdr",4,32.5);
    employee[2] = new Admin("Mary Jennings","nnv",35.3,88.5,34.3);
    employee[3] = new Clerk("Brian Jones","bbl",42.4,78.5,23.5,45.3);
    employee[4] = new Manager("John Bloggs","gvr",5,33.5);
    employee[5] = new Admin("Bridget Jennings","nvv",45.3,98.5,36.3);
    employee[6] = new Clerk("Jack Jones","bbb",43.4,78.5,23.5,47.3);


    //Initial Read
    choice = showMenu();

    //Continue Until 4/Exit
    while (choice != MAX)
    {
        switch (choice)
        {
        case 1://Manager

            System.out.println();
            System.out.printf("%s %-16s %-10s %6s","Name","Id","Hours Worked","Pay");
            System.out.println("\n==================================================");

            for (k = 0; k < MAX; ++k)
            {
                if (employee[k] instanceof Manager){ //use of string method instance of.


                    System.out.println(employee[k].toString());
                }
            }
            break;

        case 2://Administration

            System.out.println();
            System.out.printf("%s %-16s %-10s %6s %-19s","Name","Id","Hours Worked","Pay","Admin Quota");
            System.out.println("\n==================================================");

            for (k = 0; k < MAX; ++k)
            {
                if (employee[k] instanceof Admin){
                System.out.println(employee[k].toString());

                }
            }
            break;

        case 3://Clerk

            System.out.println();
            System.out.printf("%s %-16s %-10s %6s %-19s","Name","Id","Hours Worked","Pay","Admin Quota","Units Sold");
            System.out.println("\n==================================================");

            for (k = 0; k < MAX; ++k)
            {
                if (employee[k] instanceof Clerk){
                System.out.println(employee[k].toString());
                }
            }
            break;

switch ステートメントのこのケースでは、NullPointerException. すべてをチェックして、正しく初期化されているかどうかを確認しましたが、問題が見つからないようです。また、名前が見つからない場合、検索機能は名前検索をバイパスしてデフォルトのメッセージを取得します。いくつかのガイダンスをいただければ幸いです。

        case 4://Name search

            System.out.print("Enter employee name: ");
            name = console.nextLine();

            k = -1;
            notFound = true;



            while ((k < MAX-1) && (notFound))
            {
                ++k;
                if (name == employee[k].getName()){

                    System.out.println();
                    System.out.printf("%s %-16s %-10s %6s %-19s","Name","Id","Hours Worked","Pay","Admin Quota","Units Sold");
                    System.out.println("\n==================================================");

                    System.out.println(employee[k].toString());
                    System.out.println();
                    notFound = false;
                }


            }//end of case 4 while.
             if (notFound){
                System.out.println("Employee name not found\n");
            }
            break;

        case 7://exit
            System.out.println("Program exiting...");
            System.exit(0);

        default:
            System.out.println("Invalid menu choice 1..3 of 7 to Exit");    
        }//end of switch

        //sub read 
        choice = showMenu();

    }//end of while 
}//end of main

//Menu method for employee selection.
public static int showMenu()
{

    int choice;
    System.out.println();

    System.out.println("Employee Program Menu");

    System.out.println("1.Show Manager pay details ");
    System.out.println("2.Show Admin pay details ");
    System.out.println("3.Show Clerk pay details ");
    System.out.println("4.Search by employee name ");
    System.out.println("7.Exit");

    System.out.print("Enter option: ");
    choice = console.nextInt();

    return choice;
}
}
4

3 に答える 3

3

0thindex に値を設定していないため、if (name == employee[0].getName()){}にアクセスしようとすると例外が発生しますemployee[0].getName()

配列はインデックス 0 で始まり、サイズ -1 で終わるため、値をインデックス 0 に設定する必要があります ...

employee[0] = new Manager("Joe Bloggs","gdr",4,32.5);
employee[1] = new Admin("Mary Jennings","nnv",35.3,88.5,34.3);
...
employee[6] = new Clerk("Jack Jones","bbb",43.4,78.5,23.5,47.3)
于 2012-11-16T17:49:10.410 に答える
0

編集 もう一度見てから、これをあちこちで行います。コードを調べて、配列が適切に初期化されていることを確認してください。また、switch ステートメントに名前付き定数を使用してみると、読みやすくなります (私のような学生であれば、マジック ナンバーの点数を失うことはありません)。

    final int MANAGER = 1;

    final int CLERK = 2; 

    switch(choice)
    {
          case MANAGER: foo;
              break;
           case CLERK: foo;
              break;
    }

これはあなたの問題コードです。制御フローが if ステートメントに到達すると、k はゼロに等しくなり、配列の 0 番目の要素がないため、Java はヌル ポインター例外をスローします。

また、今後の参考のために、コンパイラが表示する実際のエラーを調べてみてください。コンパイラは行番号を提供します。これにより、エラーを簡単に追跡できます。Java は、null ポインターに関しては非常に親切です。C++ は、不適切な場所で見つかった値を使用して実行を継続します。

        k = -1;
        notFound = true;



        while ((k < MAX-1) && (notFound))
        {
            ++k;
            if (name == employee[k].getName())
于 2012-11-16T17:55:49.427 に答える
0

Java では形が崩れているかもしれませんが、employee[0] に新しい Employee を設定するのを忘れていませんか?

ケース 4 では、k を -1 に等しくしてから 0 に増やしますが、employee[0] には Employee がまったくないためです....

于 2012-11-16T17:51:28.673 に答える