JavaRush/Помощь по Java задачам/Добрый вечер. Не могу пройти нулл Пожалуйста, подскажите ...

35 уровень

21.11.2021
453просмотров
12комментариев

Добрый вечер. Не могу пройти нулл Пожалуйста, подскажите ТОЛЬКО НЕ РЕШЕНИЕ!!!

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

Обсуждается

Добрый вечер. Не могу пройти нулл Пожалуйста, подскажите ТОЛЬКО НЕ РЕШЕНИЕ!!! Понимаю, что куча иф и т.д., но много чего пробовал никак. Куда смотреть ?

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); } }

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

21 ноября 2021, 20:31

полезный

И почему ImageReaderFactory наследуется от ImageReader? Это примерно как считать мясокомбинат разновидностью колбасы.

Ksenia VolkovaJava Developer в DXC

Master

21 ноября 2021, 20:27

if (jpg.equals(null) || jpg == null || jpg.equals(""))

Вот так - jpg.equals(null) - на null проверять нельзя. Если в переменной jpg окажется null, то программа упадет с ошибкой на этой проверке. Вот эта проверка - jpg.equals("") - не имеет смысла. Потому что тип переменной jpg - ImageTypes. А значит она никак не может быть равна строке. В задании сказано, что исключение IllegalArgumentException должно выбрасываться. Нигде не сказано, что его надо перехватывать (да и странно было бы выбрасывать исключение, чтобы тут же его поймать). А у тебя это исключение нигде не выбрасывается, но зачем-то ты его ловишь.

БелК в труселях

Уровень 35

21 ноября 2021, 20:47

ВЫ МОЙ СПАСИТЕЛЬ! 1 Вопрос почему так ImageReaderFactory наследуется от ImageReader? Это лучшее что я придумал. И как мне вернуть другой класс в ретерн? ImageReader imageReaderFactory= new ImageReaderFactory(); Я понял что только так или как еще? 2. Долго возился без Вас никак бы не смог я думал надо исключение схватить по привычки. if ( jpg == null ) { throw new IllegalArgumentException("Неизвестный тип картинки"); } и еще засунул else if(jpg==null||!(imageReaderFactory instanceof BmpReader) ||!(imageReaderFactory instanceof JpgReader) || !(imageReaderFactory instanceof PngReader)) { throw new IllegalArgumentException("Неизвестный тип картинки"); } и прошло Я понимаю, что бред с нулл у меня, но около 25 попытки меня понесло. ВАМ ОГРОМНОЕ СПАСИБО!

Ksenia VolkovaJava Developer в DXC

Master

21 ноября 2021, 20:57

То есть ты унаследовал ImageReaderFactory от ImageReader , чтобы написать вот это?

ImageReader  imageReaderFactory= new ImageReaderFactory();

А для чего нужна эта строка? какой ее смысл и назначение?

БелК в труселях

Уровень 35

21 ноября 2021, 21:06

Смысл для того что бы в ретюрн отправлять класс. Я когда с Вами решил посмотрел решение там создали private ImageReaderFactory() { } хотя для чего он непонятно? Я понимаю что можно без него но хотел схватить исключение. ЕЩЕ РАЗ СПАСИБО!

Ksenia VolkovaJava Developer в DXC

Master

21 ноября 2021, 21:12

Так я понять никак не могу - для чего ты создаешь объект ImageReaderFactory? Он же не нужен ни разу. Вернуть надо ImageReader (то есть BmpReader или JpgReader или PngReader). Объект, кторый ты создаешь в той строке, не играет никакой роли в коде. Но ты для создания этого объекта придумываешь извращенную систему наследования... Твой else if с проверками instanceof тоже не нужен. И проверять jpg на null там уже не имеет смысла.

БелК в труселях

Уровень 35

22 ноября 2021, 05:21

Доброе утро! Да я понимаю что проверки не нужны елс иф , а создание системы было когда я пытался как то отправить классы (то есть BmpReader или JpgReader или PngReader). Пробовал разные варианты. Сейчас после Ваших объяснений и анализа решения я понимаю, что это не нужно. ЕЩЕ раз СПАСИБО!

Александр Анисимов

Уровень 41

26 ноября 2021, 09:00

Ксения а подскажите почему null не дает проверить в switch? Сначала приходится if проверку на null делать? и почему default не достаточно и нужно все таки дополнительно проверять через if на null?

Ksenia VolkovaJava Developer в DXC

Master

26 ноября 2021, 09:07

Потому что swtich сравнивает строки с помощью equals, и в случае с null будет NPE

Александр Анисимов

Уровень 41

26 ноября 2021, 09:19

понятно, спасибо, получается он до default не доходит даже если мы null передаем?

Ksenia VolkovaJava Developer в DXC

Master

26 ноября 2021, 09:21

Нет, потому что если передать null, то исключение возникнет на входе в switch

Александр Анисимов

Уровень 41

26 ноября 2021, 10:33

спасибо за ответ)