Из-за чего программа выдаёт ошибку и как это исправить?

Question

[Кирилл Баженов]

public class infa {
    public static void main(String[] args) {
        Job job = new Job();

        job.R();
    }
}



import java.util.Random;

public class Job {

    void R () {

        Random random = new Random();


        String[][] job = new String[][] {

                {"Сколько бит в байте","Cколько бит в килобайте","Сколько бит в мегабайте"},
                {"Сколько байт в килобайте","Сколько байт в мегабайте"},
                {"Сколько килобайт в мегабайте"}
        };

        int p = random.nextInt(job.length+1);

        System.out.println(job[p][p]);
    }


}

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 3
at Job.R(Job.java:23)
at infa.main(infa.java:5)

Comments

[Кирилл Баженов]

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 3
at Job.R(Job.java:23)
at infa.main(infa.java:5)

[MaxLich]

можно было бы убрать дубли в стрингах ("сколько" и"в" вынести куда-то отдельно). И название метода - R - это ад. Не надо так методы называть!

Answer 1

[GavriKos]

int p = random.nextInt(job.length+1);
Откуда +1?

Answer 2

[mr_serg77]

У Вас проблема в понимании английского, ну а по факту кода - с индексом.

Comments

[Кирилл Баженов]

так как исправить?

[mr_serg77]

Кирилл Баженов, используйте корректный диапазон индексов.

Answer 3

[ComradV]

1. как правильно написано в первом комментарии, +1 там быть не должно.
2. Вообще, у вас первый элемент массива - массив из 3 элементов, второй - из двух, первый - из одного. Поэтому если вам сгенерировал генератор p=2, у вас получится элемент 2 из 2 массива. Во 2 по индексу массиве (в третьем по счёту) только 1 элемент, поэтому вы "вывалитесь" за границу массива.
Если вам нужен случайный элемент двумерного непрямоугольного массива, можно предложить технику:
int p = random.nextInt(job.length);
int q = random.nextInt(job[p].length);
System.out.println(job[p][q]);
3. Методы - с маленькой буквы
4. Если этот класс больше ничего не делает, разумно сделать метод статическим и не создавать лишний объект, а писать Job.r();