NullPointerException

Вопрос по задаче

Решен

Вопрос возник при написании метода getHen. Если писать так, как написано сейчас, то JVM выдает NullPointerException. Если его перехватывать, то программа не компилируется, так как уходит переменная hen. Конечно, можно создавать в методе getHen в блоках if других куриц, используя hen = new RussianHen() (как пример). Но будет ли тогда ругатся валидатор? Хочется решить задачу с первого раза)))

Давай напишем Фабрику (Factory) по производству кур (Hen):

Создай класс Hen.

Сделай его абстрактным.
Добавь в класс абстрактный метод int getCountOfEggsPerMonth().
Добавь в класс метод String getDescription(), который возвращает строку "Я - курица.".

Создай класс RussianHen, который наследуется от Hen.
Создай класс UkrainianHen, который наследуется от Hen.
Создай класс MoldovanHen, который наследуется от Hen.
Создай класс BelarusianHen, который наследуется от Hen.
В каждом из четырех последних классов напиши свою реализацию метода getCountOfEggsPerMonth.
Методы должны возвращать количество яиц в месяц от данного типа куриц.
В каждом из четырех последних классов напиши свою реализацию метода getDescription.
Методы должны возвращать строку вида:
<getDescription() родительского класса> + <" Моя страна - Sssss. Я несу N яиц в месяц.">
где Sssss - название страны
где N - количество яиц в месяц
В классе HenFactory реализуй метод getHen, который возвращает соответствующую стране породу кур.
Все созданные классы должны быть в отдельных файлах.

Требования:

Класс Hen должен быть абстрактным.
Класс Hen должен содержать абстрактный метод int getCountOfEggsPerMonth().
В классе Hen должен быть реализован метод String getDescription(), который возвращает строку "Я - курица.".
Классы RussianHen, UkrainianHen, MoldovanHen и BelarusianHen должны наследоваться от класса Hen и быть созданы в отдельных файлах.
Классы RussianHen, UkrainianHen, MoldovanHen и BelarusianHen должны реализовывать метод getCountOfEggsPerMonth, который должен возвращать количество яиц в месяц от данного типа куриц.
Классы RussianHen, UkrainianHen, MoldovanHen и BelarusianHen должны переопределять метод getDescription родительского класса таким образом, чтобы возвращаемая ими строка имела вид: <getDescription() родительского класса> + < Моя страна - Sssss. Я несу N яиц в месяц.> где Sssss - название страны, а N - количество яиц в месяц.
Метод getHen должен быть реализован в классе HenFactory и возвращать тип кур для переданной в него страны.

package com.javarush.task.task14.task1408; public interface Country { String UKRAINE = "Ukraine"; String RUSSIA = "Russia"; String MOLDOVA = "Moldova"; String BELARUS = "Belarus"; }

Комментарии (4)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Сергеев Виктор

Уровень 40

Master

23 июня 2019, 15:27

Конечно, можно создавать в методе getHen в блоках if других куриц, используя hen = new RussianHen() (как пример) это не правильно. NPE это частая ошибка, автор её назвал "ошибка на 1000000". Тут есть 3 вариант решения, которая я знаю: 1. создать Null объект. Т.е. объект курицы, который будет возвращаться по дефолту, если страна не найдена. 2. Создать свое исключение CountryNotFoundException или как-то так. Выбрасывать его, когда страна не найдена. 3. Возвращать null, но в месте вызова проверять if (obj != null)

Olga Enaldieva

Уровень 20

24 июня 2019, 04:57

Как я понял, первый вариант мне не подходит, так как у меня создан дефолтный объект курицы

Hen hen = null;

и при этом все равно возникает ошибка. Второй способ посложнее, + нужно почитать намного литературы. Я, конечно, умею создавать исключения, но вот что делать с его перехватом? К тому же это исключение будет равносильно 3-му способу. Следовательно проще сделать 3-м способом. Единственое, что я не понял, так это куда вставлять if (obj != null). При создании объекта Hen hen = HenFactory.getHen(Country.BELARUS);? Или лучше на месте return в методе getHen()?

Olga Enaldieva

Уровень 20

24 июня 2019, 05:05

Виктор, сегодня я с новыми свежими мыслями выявил ошибку. Изменил код:

static Hen getHen(String country) throws NullPointerException {
           Hen hen = null;
           if (country.equals("Belarus"))  {System.out.println("Провека выполнена"); hen = (BelarusianHen) hen; if (hen == null) System.out.println("Что-то пошло не так"); }
           if (country.equals("Russia"))  System.out.println("Провека выполнена"); //hen = (RussianHen) hen;
           if (country.equals("Ukraine"))  System.out.println("Провека выполнена"); //hen = (UkrainianHen) hen;
           if (country.equals("Moldova")) System.out.println("Провека выполнена"); //hen = (MoldovanHen) hen;
           return hen;
       }

Оказывается, объект hen просто не меняет свой тип, и остается ссылкой на null. Почему так происходит?

Olga Enaldieva

Уровень 20

24 июня 2019, 07:14

решение

Я разобрался с вопросом. hen не изменялось, потому что объекта как такового не было. А значит нечему было изменяться. Поэтому hen = new RussianHen() оказалось единственным верным решением