Как правильно: проверить на null через if либо выбросить исключение в try-catch в Java?

Question

[Jake Taylor]

Имеется метод create для создания экземпляра класса CustomArray, который в качестве аргумента принимает массив примитивов типа int.
Создан также собственный класс исключений ArrayException.
Одни из условий для метода должны выступать:
-проверка аргумента метода на null;
- проверка аргумента метода на нулевой размер.

С точки зрения правильности реализации проверки на null аргумента метода как правильно реализовать метод, проверкой аргумента на null через условие if либо выбросить исключение в конструкции try-catch?

Сам метод:

public CustomArray create(int[] numbers) throws ArrayException {
}

Вариант с использованием if:

public CustomArray create(int[] numbers) throws ArrayException {
        if(numbers.length == 0) {
            throw new ArrayException("Array is empty!");
        } else if (numbers == null) {
            throw new ArrayException("An empty argument passed!");
        }
    }

Пример с try-catch:

public CustomArray create(int[] numbers) throws ArrayException {
        CustomArray customArray;

        try {
            customArray = new CustomArray(numbers);
        } catch (NullPointerException ex) {
            throw new ArrayException("Null pointer!");
        } 
        
        return customArray;
    }

Comments

[Антон Воскобоев]

Хэндлить NPE - не очень хорошая практика. Лучше простая проверка.

[Jake Taylor]

Антон Воскобоев, что такое хэндлить?
Где написано про ту самую практику? Мне интересно где это написано, чтобы в будущем оперировать данным аргументом.

[Антон Воскобоев]

Jake Taylor, лучше не допускать NPE и выполнить проверку на null. Написано ли где-то или это из личного и чужого опыта - уже не вспомню.

Answer 1

[Сергей Горностаев]

Исключения не нужны.

public Optional<CustomArray> create(int[] numbers) {
    return Optional.ofNullable(numbers)
        .filter(a -> a.length > 0)
        .map(CustomArray::new);
}

Comments

[Jake Taylor]

А если кто-нибудь отправит чудесным образом null?

[Сергей Горностаев]

Jake Taylor, то вернётся пустой Optional.

[Jake Taylor]

Сергей Горностаев, в моей ситуации нужно, чтобы в данной ситуации выбросилось исключение, дальнейшее выполнение невозможно.

[Сергей Горностаев]

Jake Taylor, зря, исключения не поддаются композиции. Но если уж надо, то я бы проверки из первого варианта перенёс в конструктор CustomArray, если он под вашим контролем. Тогда метод create можно было бы вообще выбросить.

[Jake Taylor]

Сергей Горностаев, почему нужен именно метод create - т.к. создается из расчета создания данной структуры проекта.

[Сергей Горностаев]

Jake Taylor, вызов конструктора - это не бизнес-логика, отдельный метод в сервисном слое для этого не нужен.

P.S. Чтение статей от джунов-пхпшников до добра не доведёт.

[Jake Taylor]

Сергей Горностаев, а где тогда брать информацию про слоистую архитектуру? Подскажи годный источник.

[Сергей Горностаев]

Jake Taylor, для начала можно "Чистый код" и "Чистую архитектуру" Мартина прочитать.

[Jake Taylor]

Сергей Горностаев, Благодарю.