Валидатору не нравятся Украинские куры...

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

Решен

Добре. Не могу понять, почему валидатор не хочет пропускать задачу. Ментор советует разобраться с Украинской курицей. - Метод getDescription должен возвращать строку - "Я - курица.". - Курица(UkrainianHen) не может нести отрицательное количество яиц в месяц! - Описание курицы UkrainianHen должно соответствовать формату "getDescription() родительского класса + Моя страна - Ukraine. Я несу N яиц в месяц.", где N - число яиц в месяц(возвращаемое методом getCountOfEggsPerMonth. Например: Я - курица. Моя страна - Ukraine. Я несу 5 яиц в месяц. Но ведь все это выполняется. Чито не так?

Давай напишем Фабрику (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 class Solution { public static void main(String[] args) { Hen hen = HenFactory.getHen(Country.BELARUS); hen.getCountOfEggsPerMonth(); } static class HenFactory { static Hen getHen(String country) { Hen hen = null; if (country.equals(Country.BELARUS)){ hen = new BelarusianHen(country);//напишите тут ваш код } else if (country.equals(Country.RUSSIA)){ hen = new RussianHen(country); } else if (country.equals(Country.UKRAINE)){ hen = new UkrainianHen(country); } else if (country.equals(Country.MOLDOVA)){ hen = new MoldovanHen(country); } return hen; } } }

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

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

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

Сергей System Engineer в -

2 апреля 2020, 19:04

Описание ошибки не имеет к проблеме никакого отношения. Особенно про отрицательные яйца. вот так не правильно:

hen = new BelarusianHen(country);

Класс сам знает какая строка должна быть. Переписал - прошла. - Курица(UkrainianHen) не может нести отрицательное количество яиц в месяц! - Описание курицы UkrainianHen должно соответствовать формату "getDescription() родительского класса + Моя страна - Ukraine. Я несу N яиц в месяц.", где N - число яиц в месяц(возвращаемое методом getCountOfEggsPerMonth. Например: Я - курица. Моя страна - Ukraine. Я несу 5 яиц в месяц.

Denys

Уровень 34

27 февраля 2019, 19:02

1. Уберите поле country из класса Hen, соответственно и конструктор. Следовательно и во всех классах наследников уберите лишний код. Тогда (например класс UkrainianHen):

@Override
public String getDescription() {
    return super.getDescription() + ". Моя страна - " + Country.UKRAINE + ". Я несу " + getCountOfEggsPerMonth() + " яиц в месяц.";
}

2. Модификатор public лишний в абстрактных классах (это так, ко всему, проблема в вышеуказанном пункте). HenFactory:

static Hen getHen(String country) {
    Hen hen = null;
    if (country.equals(Country.BELARUS))
        hen = new BelarusianHen();
    if (country.equals(Country.MOLDOVA))
        hen = new MoldovanHen();
    if (country.equals(Country.RUSSIA))
        hen = new RussianHen();
    if (country.equals(Country.UKRAINE))
        hen = new UkrainianHen();
    return hen;
}

Как-то не совсем логично. Если класс называется (Ukrainian)Hen то глупо получается передавать в конструктор страну курицы, вы ей уже сказали что она украинская курица.

NewBuy

Уровень 29

Expert

28 февраля 2019, 10:11

Спасибо. Переспав ночь, тоже пришел к такому выводу, хотя конечно претензии валидатора очень сбивают с толку. А второй фрагмент кода, про HenFactory не совсем понял, почему в строках с условием у вас фигурирует Hen.BELARUS и т.д., а не Country.BELARUS?

Denys

Уровень 34

28 февраля 2019, 18:02

То из-за моей спешки. Естественно там должно быть Country.BELARUS и т.д. Поправил, спасибо. Пожалуйста. Успехов в обучении!)