我无法理解我收到的错误

时间:2015-01-30 15:26:19

标签: java

我似乎遇到了一个问题,我在终端中的代码中出现错误,出于某种原因,但似乎无法理解实际问题。这是我得到的错误:

  

线程中的异常" main" java.lang.ArrayIndexOutOfBoundsException:5           在Lottery.compareNumbers(LotteryApplication.java:59)           在LotteryApplication.main(LotteryApplication.java:106)

这是我的代码:

import java.util.Random;
import java.util.Scanner;

class Lottery {

    /**
     * The lottery numbers.
     */
    private int lotteryNumbers[];

    /**
     * Default Constructor.
     *
     * The class should use the Random class (from the Java API) to generate a
     * random number in the range of 0 through 9 for each element in the array.
     */
    public Lottery() {
        Random rand = new Random(System.currentTimeMillis());
        lotteryNumbers = new int[5];
        for (int i = 0; i < lotteryNumbers.length; i++) {
            lotteryNumbers[i] = rand.nextInt(10);
        }
    }

    /**
     * The class should also have a method that accepts an array of 5 integers
     * that represent a person's lottery picks. The method is to compare the
     * corresponding elements in the two arrays and return the number of digits
     * that match.
     */
    public int compareNumbers(int[] usersNumbers) {
    int match = 0;
    if (usersNumbers.length == lotteryNumbers.length) {
        for (int i = 0; i < lotteryNumbers.length; i++) {
            for(int j = 0; j < lotteryNumbers.length; i++) {
                if (usersNumbers[i] == lotteryNumbers[j]) {
                    //match++;
                    break;
                }
            }
        }
    }
    return match;
    }

    /**
     * In addition, the class should have a method that returns a copy of the
     * lotteryNumbers array.
     */
    public int[] getLotteryNumbers() {
        return lotteryNumbers;
    }
    }

    /**
    * Demonstrate the class in a program that asks the user to enter five numbers.
    * The program should display the number of digits that match the randomly
    * generate lottery numbers. If all of the digits match, display a message
    * proclaiming the user a grand prize winner.
    */
    public class LotteryApplication {
    public static void main(String[] args) {
        Lottery lottery = new Lottery();
        int lotteryNumbersCount = lottery.getLotteryNumbers().length;

        System.out.println("\t\t\t\tLottery Application\n");
        System.out.println("   " + "There are " + lotteryNumbersCount
                + " secret numbers in range of 0 through 9. "
                + "Try to guess them!\n");

        // Asks the user to enter five numbers.
        Scanner kb = new Scanner(System.in);
        int numbers[] = new int[lotteryNumbersCount];

        for (int i = 0; i < numbers.length; i++) {
            System.out.print(String.format("Enter Number %d: ", i + 1));
            numbers[i] = kb.nextInt();
        }

        // Display the number of digits that match the randomly generate
        // lottery numbers.

        int match = lottery.compareNumbers(numbers);

        if (match == lotteryNumbersCount) {

            // If all of the digits match, display a message proclaiming the
            // user a grand prize winner.
            System.out.println("\nWOHOO! ALL CORRECT! YOU WON THE BIG PRIZE!");

        } else {

            System.out.println("\nUh-oh! You hit " + match + " number(s).");

        }
    }
}

2 个答案:

答案 0 :(得分:1)

将内部for循环从for(int j = 0; j < lotteryNumbers.length; i++)更改为for(int j = 0; j < lotteryNumbers.length; j++),否则您将运行i++太多次并超出lotteryNumbers的范围

答案 1 :(得分:1)

在你的第二个for循环中,你计算i++。因此,您在两个for循环中都计算i++。但是你应该在i++和另一个循环j++中计算一次。也许是通过复制粘贴发生的。

if (usersNumbers.length == lotteryNumbers.length) {
        for (int i = 0; i < lotteryNumbers.length; i++) {
            //Here you have to set j++
            for(int j = 0; j < lotteryNumbers.length; j++) {
                if (usersNumbers[i] == lotteryNumbers[j]) {
                    //match++;
                    break;
                }
            }
        }
    }