не проходит валидатор

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

Factory method pattern

Java Core, 6 уровень, 13 лекция

Решен

подскажите, что не так? программа работает правильно

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

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

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

Иван Лаврентьев

Уровень 22

23 декабря 2017, 22:17

все, нашел ошибку - создал ImageReaderFactory не в корне, в пакете commons сам код - верный всем спасибо

Иван Лаврентьев

Уровень 22

23 декабря 2017, 22:04

особенно вот это доставляет

Создай публичный класс ImageReaderFactory

Иван Лаврентьев

Уровень 22

23 декабря 2017, 22:01

переписал вот так

public class ImageReaderFactory {
//    static ImageReader imageReader;

    public static ImageReader getImageReader(ImageTypes imageTypes) {
//        if (imageTypes!=null){
//            try {
                if (imageTypes == ImageTypes.BMP)
                    return new BmpReader();
                else if (imageTypes == ImageTypes.PNG)
                    return new PngReader();
                else if (imageTypes == ImageTypes.JPG)
                    return new JpgReader();
                else throw new IllegalArgumentException("Неизвестный тип картинки");
//            } catch (IllegalArgumentException e) {
//                System.out.println("Неизвестный тип картинки");
//            }
        }
//        else {
//            throw new NullPointerException();
//        }
//        return imageReader;

}

работает точно так же, валидатор точто так же не принимает

Иван Лаврентьев

Уровень 22

23 декабря 2017, 21:28

он скрыт в закладках, вот код

package com.javarush.task.task16.task1631.common;

public class ImageReaderFactory {
    static ImageReader imageReader;

    public static ImageReader getImageReader(ImageTypes imageTypes) {
        if (imageTypes!=null){
            try {
                if (imageTypes == ImageTypes.BMP)
                    imageReader = new BmpReader();
                else if (imageTypes == ImageTypes.PNG)
                    imageReader = new PngReader();
                else if (imageTypes == ImageTypes.JPG)
                    imageReader = new JpgReader();
                else throw new IllegalArgumentException();
            } catch (IllegalArgumentException e) {
                System.out.println("Неверный тип картинки");
            }
        }
        else {
            throw new NullPointerException();
        }
        return imageReader;
    }
}

Александр Неклеса Backend Developer

23 декабря 2017, 21:48

4.2. Если передан неправильный параметр, то выбрасывать исключение IllegalArgumentException("Неизвестный тип картинки") У тебя другое сообщение "Неверный тип картинки" вместо "Неизвестный тип картинки" Не нужно кидать NullPointerException(). убери try/catch ты из твоего метода вылетают только NullPointerException, а IllegalArgumentException ты перехватил в catch попробуй так

if (imageTypes == ImageTypes.BMP)
    return new BmpReader();
else if (imageTypes == ImageTypes.PNG)
    return = new PngReader();
else if (imageTypes == ImageTypes.JPG)
    return = new JpgReader();
else throw new IllegalArgumentException("Неизвестный тип картинки");

зачем тебе static ImageReader imageReader; ?

Виталий Мельник

Уровень 31

23 декабря 2017, 21:26

А где класс ImageReaderFactory?

-2