Java почему вылетает java.lang.NullPointerException?

Question

[Millerish]

Добрый вечер!

Разбираюсь в java. Почему у меня вылетает java.lang.NullPointerException, я же проверяю на Null? Как правильно?

import java.util.Arrays;
import java.util.Scanner;

public class l9 {

    public static void main(String[] args){
        class Arr{


            int[] mass;

            void sayLen(){
                if (mass == null) {
                    System.out.println("0");
                } else {
                    int len = this.mass.length;
                    System.out.println(len);
                }
            }

            void addToMass(int e){
                if (mass == null) {
                    this.mass[0] =e;
                } else {
                    this.mass[mass.length+1] = e;
                }

            }

        }

        Arr arr = new Arr();
        Scanner in = new Scanner(System.in);

        while (true){
            try {
                int a = in.nextInt();
                System.out.println(a);
                arr.addToMass(a);
                arr.sayLen();
            }
            catch (Throwable e){
                System.out.println(e);
                break;
            }

        }

    }
}

Answer 1

[Evhen]

А массив кто создавать будет?

int[] mass; - это ссылка на массив которая равна null, это означает, что любые обращения к ней, типа mass.length или mass[0] будут приводить NullPointerException. Вы пытаетесь узнать длину или получить доступ к нулевому элементу массива которого нет.

this.mass[mass.length+1] = e; - это вы что пытаетесь сделать? Присвоить значение ячейки массива, которая находится за пределами размера массива.... получите еще одно исключение.

Массив - структура фиксированной длины, т.е. уменьшать или увеличивать её размер нельзя, только заново пересоздавать и копировать элементы из старого в новый.

Индекс массива может принимать значение от 0 до length - 1.

В общем внимательно дочитайте всю главу о массивах...

Comments

[Millerish]

Вот в примере mas2[1] = 20; или так нельзя?

[Evhen]

Millerish:

int[] mas2 = null;

// mas2[1] = 20; nullpointerexception

mas2 = new int[5];

mas2[0] = 10;
mas2[1] = 20;
mas2[2] = 30;
mas2[3] = 40;
mas2[4] = 50;

// mas2[5] = 50; ArrayIndexOutOfBoundsException

[Millerish]

ок, но ArrayIndexOutOfBoundsException вылетает при первой итерации:

int[] mass = new int[5];

void sayLen(){
if (mass == null) {
System.out.println("0");
} else {
int len = this.mass.length;
System.out.println(len);
}
}
void addToMass(int e){
if (mass == null) {
this.mass[0] = e;
} else {
this.mass[mass.length+1] = e;
}
}

1
1
java.lang.ArrayIndexOutOfBoundsException: 6
at l9$1Arr.addToMass(l9.java:24)
at l9.main(l9.java:38)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)

Process finished with exit code 0

[Evhen]

Millerish: this.mass[mass.length+1] = e; - вы вообще читаете, что я вам писал? о_О!!! Массив длиной 5. Индекс может быть от 0 до 4. Вам в исключении написали, что вы вышли за рамки массива, пытаясь присвоить 6-й ячейке, 6-й!!! Карл!

[Millerish]

спасибо!

Answer 2

[Дмитрий Александров]

mass не инициализирован. Вообще лучше используй коллекции =D

List<Integer> mass;

            void sayLen(){
                if (mass == null) {
                    System.out.println("0");
                } else {
                    int len = this.mass.size();
                    System.out.println(len);
                }
            }
            void addToMass(int e){
                if (mass == null) {
                    mass = new ArrayList<>();
                    this.mass.add(e);
                } else {
                    this.mass.add(e);
                }

            }

        }

Comments

[Millerish]

Спасибо, с arraylist все стало сразу ок)))

Answer 3

[Даниил Демидко]

В Java массив - тоже класс, а значит по него должна быть выделена память в куче.