Че он хочет

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

Решен

Ментор:

Проверь, что метод getImageReader работает правильно если ему передать null

1. Внимательно посмотри, какие классы у тебя есть.
2. В отдельных файлах в пакете common создай классы JpgReader, PngReader, BmpReader, которые реализуют интерфейс ImageReader.
3. В отдельном файле в основном пакете создай класс ImageReaderFactory с одним методом.
3.1. Подумай, как он должен называться.
3.2. Подумай, какие модификаторы должны быть у этого метода.
4. Этот метод должен:
4.1. Для каждого значения из ImageTypes возвращать соответствующий Reader, например, для ImageTypes.JPG - JpgReader;
4.2. Если передан неправильный параметр (в т.ч. null), то выбрасывать исключение IllegalArgumentException("Неизвестный тип картинки").

Требования:

Создай в пакете common класс JpgReader, реализующий интерфейс ImageReader.
Создай в пакете common класс PngReader, реализующий интерфейс ImageReader.
Создай в пакете common класс BmpReader, реализующий интерфейс ImageReader.
Создай публичный класс ImageReaderFactory.
Добавь в класс ImageReaderFactory открытый статический метод getImageReader с параметром ImageTypes.
Метод getImageReader должен создавать и возвращать подходящий Reader.
Метод getImageReader должен кидать исключение IllegalArgumentException, если передан неправильный параметр.

package com.javarush.task.task16.task1631; import com.javarush.task.task16.task1631.common.ImageReader; import com.javarush.task.task16.task1631.common.ImageTypes; public class Solution { public static void main(String[] args) { ImageReader reader = ImageReaderFactory.getImageReader(ImageTypes.JPG); } }

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

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

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

Алексей Java Developer

Expert

12 октября 2019, 13:24

Хочет чтобы в случае если в getImageReader аргумент является null выбрасывалось исключение IllegalArgumentException. Ваше решение в этом случае выбрасывает NullPointerException.

Siraj Java Developer в Сбербанк

12 октября 2019, 13:28

Вот же:

else
                throw new IllegalArgumentException("Неизвестный тип картинки");

Алексей Java Developer

Expert

12 октября 2019, 13:29

А Вы передайте null в функцию)

Siraj Java Developer в Сбербанк

12 октября 2019, 13:33

К сожалению идея не запускает код, еще не пофиксил эту проблему.. поэтому даже не пробовал тестить..

Алексей Java Developer

Expert

12 октября 2019, 13:35

Идея не запускает код, если на этапе компиляции в проекте находятся синтаксические ошибки (проект от JavaRush включает в себя все задачи, которые Вы пробовали решать в Идее), либо если не указан JDK для проекта (на Вашем компе установлен JDK? Он указан для проекта?)

Алексей Java Developer

Expert

12 октября 2019, 13:38

Но вообще, если object == null, то object.equals (что-угодно) вылетит с NullPointerException.

Siraj Java Developer в Сбербанк

12 октября 2019, 14:41

так и не понял как решить в итоге поменял местами и прошло, но ничего не понял.

if (imageTypes.equals(ImageTypes.PNG))

Wladyslaw Java Developer

Master

12 октября 2019, 15:37

полезный

проверку на null нужно было сделать для параметра. Если в твой метод придет null - ты попытаешься разименовать ссылку и вызвать у нее метод, а она равна null. В итоге будет брошен NullPointerException А поменял ты, я так полагаю на что-то типа

if (ImageTypes.PNG.equals(imageTypes))

Так тоже пройдет ибо ты у нулевой ссылки ничего не вызываешь, а просто сравниваешь элементы энума с нуллом. В итоге будет брошен таки тот экспепшн который тебе нужен.