Как сделать что бы проверял на null

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

Решен

Осталось добавить проверку на 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; /* Factory method pattern */ public class Solution { public static void main(String[] args) { ImageReader reader = ImageReaderFactory.getImageReader(ImageTypes.JPG); } }

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

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

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

Anonymous #2372013

Уровень 26

4 июля 2021, 17:45

Проверку на null нужно выполнять в классе ImageReaderFactory: если (тип == ИзображенийТипы.БМП) { изображенийЧитатель = новый БМПЧитатель(); } иначе если (тип == ИзображенийТипы.JPG) { изображенийЧитатель = новый JpgЧитатель(); } иначе если (тип == ИзображенийТипы.PNG) { изображенийЧитатель = новый PngЧитатель(); }иначе{ выброс новый НезаконныйАргументИсключение("Неизвестный тип картинки"); } возврат изображенийЧитатель; }

Роман

Уровень 24

4 июля 2021, 16:16

Как вариант еще занести все данные в поле Map<ImageTypes, ImageReader> imageReaderInstance и потом проверять приходящий параметр сразу на null, через Optional -

return Optional.ofNullable(s)
            .filter(imageTypes -> imageReaderInstance.contains(imageTypes))
            .ifPresent(imageTypes -> imageReaderInstance.get(imageTypes ))
            .orElseThrow(() -> throw new IllegalArgumentException("Неизвестный тип картинки"));

Роман

Уровень 24

4 июля 2021, 15:59

Может лучше сделать проверку if, else if, этих трех параметров и если ни одно их них не сработало (к примеру null)

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

т.е. до этого условия дойдет только если ни один их трех типов не подошел И enum можно сравнивать по ссылке (под капотом так и будет все равно в данном случае), в таком случае вот тут

s.equals

NPE не будет, либо объект, где потенциально может быть null заноси в скобки -

ImageTypes.PNG.equals(s)