0

参考までに私のコードを同封します。乗算機能のために以下のコードを実行しているときに、以下のエラーが発生します。

スレッド「メイン」での例外 java.lang.ArrayIndexOutOfBoundsException: -4 at choice2.main(choice2.java:99)

参考までに - プログラムは、このプログラムの他のすべての機能に対して正常に動作しています。

コード :

import java.util.*;

class choice2 {
    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        @SuppressWarnings("resource")
        Scanner S = new Scanner(System.in);

        int ch, subch;
        String Target = null;
        int codePos = 0, a, b, c = 0;

        String[] Arith = {"Add", "Sub"};
        String[] Mult = {"Multiplication", "Division", "Modulas" };

        System.out.println("1.Arithmatic");
        System.out.println("2.Mult/Div/Mod");
        System.out.println("0 to Exit from Menu");
        System.out.println("Enter your Choice");

        ch = S.nextInt();

        switch (ch) {
        case 1:
            System.out.println("1.Addition");
            System.out.println("2.Subtraction");
            System.out.println("9.Goback to Main Menu");
            subch = S.nextInt();

            if (subch == 1)
                Target = "Add";
            else
                Target = "Sub";

            codePos = Arrays.binarySearch(Arith, Target);

            switch (Arith[codePos]) {

            case "Add":
                System.out.println("Enter value for A");
                a = S.nextInt();
                System.out.println("Enter value for B");
                b = S.nextInt();

                c = a + b;

                System.out.println("The result is " + c);

                break;

            case "Sub":
                System.out.println("Enter value for A");
                a = S.nextInt();
                System.out.println("Enter value for B");
                b = S.nextInt();

                c = a - b;

                System.out.println("The result is " + c);

                break;
            }
            break;

        case 2:
            System.out.println("1.Multiplication");
            System.out.println("2.Division");
            System.out.println("3.Modulas");
            System.out.println("9.Goback to Main Menu");

            subch = S.nextInt();

            switch (subch) {

            case 1:
                Target = "Multiplication";
                break;

            case 2:
                Target = "Division";
                break;

            case 3:
                Target = "Modulas";
                break;

            default:
                System.out.println("Invalid Value");
            }

            System.out.println(codePos);
            System.out.println(Target);

//          codePos = Arrays.binarySearch(Mult, Target);
            codePos = Arrays.binarySearch(Mult, 0, Mult.length, Target);
            switch (Mult[codePos]) {

            case "Multiplication":
                System.out.println("Enter value for A");
                a = S.nextInt();
                System.out.println("Enter value for B");
                b = S.nextInt();

                c = a * b;

                System.out.println("The result is " + c);

                break;

            case "Division":
                System.out.println("Enter value for A");
                a = S.nextInt();
                System.out.println("Enter value for B");
                b = S.nextInt();

                c = a / b;

                System.out.println("The result is " + c);

                break;

            case "Modulas":
                System.out.println("Enter value for A");
                a = S.nextInt();
                System.out.println("Enter value for B");
                b = S.nextInt();

                c = a % b;

                System.out.println("The result is " + c);

                break;
            }
        }
        System.out.println("Exit from program");
    }
}
4

4 に答える 4

3

Arrays.binarySearch値が見つからない場合は負の数を返します。また、バイナリ検索を使用するには、配列をソートする必要がありますが、ソートされていないため、おそらくエラーが発生しています。次のように配列を宣言します。

String[] Mult = {"Division", "Modulas", "Multiplication" };
于 2013-07-15T17:54:06.230 に答える
1

配列をソートする必要があるという他の回答は正しいですが、ポイントを逃しています。次の行を検討してください。

codePos = Arrays.binarySearch(Mult, 0, Mult.length, Target);
switch (Mult[codePos]) {
  case "Multiplication":
  .....
}

インデックスを取得するために文字列ターゲットを使用して配列を検索し、一致した配列から文字列を取得するのはなぜですか? それは基本的に何も達成しません（バグの可能性を除いて）。ただ行く方がはるかに簡単です

switch (Target) {
  case "Multiplication":
   ...
  default: // handle any mistypings here...
}
于 2013-07-15T18:17:32.360 に答える
1

この線

switch (Mult[codePos]) {

入力配列がバイナリ検索用に順序付けられていないため、前の検索からそのままスローしArrayIndexOutOfBoundsExceptionてcodePosいます-4String

codePos = Arrays.binarySearch(Arith, Target);

ドキュメントから

配列に含まれている場合は検索キーのインデックス。それ以外の場合は (-(挿入ポイント) - 1)。挿入ポイントは、キーが配列に挿入されるポイントとして定義されます: キーより大きい最初の要素のインデックス、または a.length

したがってcodePos、

- (array.length + 1) = -4

二分検索の値は、値の順序付けに依存しています。したがって、あなたが必要です

String[] Mult = { "Division", "Modulas", "Multiplication", };
于 2013-07-15T17:54:32.507 に答える
0

ここ

 codePos = Arrays.binarySearch(Mult, 0, Mult.length, Target);// will retuen codePos as -4.

だからあなたは変えることができます

 codePos = Arrays.binarySearch(Mult, 0, Mult.length, Target)

に

            List<String>   list=Arrays.asList(Mult);
            codePos = list.indexOf(Target);
于 2013-07-15T18:09:47.827 に答える