Can someone please help me see my mistake?

Question about the task

Resolved

It doesn't verify the last condition: "The HenFactory class's getHen method must return a NorthAmericanHen object if Continent.NORTHAMERICA is passed as an argument."

Write a factory (HenFactory) for making chickens (Hen):

1. Write a Hen class.
1.1. Make it abstract.
1.2. Add an int getMonthlyEggCount() method to the abstract class.
1.3. Add a String getDescription() method to the class. It should return "I am a chicken."
2. Create an NorthAmericanHen class that inherits Hen.
3. Create an EuropeanHen class that inherits Hen.
4. Create an AsianHen class that inherits Hen.
5. Create an AfricanHen class that inherits Hen.
6. In each of these four classes, write your own implementation of the getMonthlyEggCount method.
The method should return the number of eggs per month produced by that type of chicken.
7. In each of these four classes, write your own implementation of the getDescription method.

The method must return a string as follows:
<parent class>.getDescription() + " I come from <continent>. I lay <n> eggs a month."
where <continent> is the name of the continent
and <n> is the number of eggs per month.

8. In the HenFactory class, implement the getHen method. It should return a chicken breed corresponding to the continent that the chicken came from.
9. All the classes you create must be in separate files.

Requirements:

The Hen class must be abstract.
The Hen class must have an abstract int getMonthlyEggCount() method.
The Hen class must implement a String getDescription() method, which returns "I am a chicken."
The NorthAmericanHen, EuropeanHen, AsianHen, and AfricanHen classes must inherit the Hen class and be located in separate files.
The NorthAmericanHen, EuropeanHen, AsianHen, and AfricanHen classes must implement the getMonthlyEggCount method, which should return the number of eggs per month laid by the corresponding type of chicken.
The NorthAmericanHen, EuropeanHen, AsianHen, and AfricanHen must override the parent class's getDescription method so that the returned string looks like this: <parent class>.getDescription() + " I come from <continent>. I lay <n> eggs a month." where <continent> is the name of the continent, and <n> is the number of eggs per month.
The getHen method must be implemented in the HenFactory class and must return the chicken type that corresponds to the passed continent.

package com.codegym.task.task14.task1408; /* Chicken factory */ public class Solution { public static void main(String[] args) { Hen hen = HenFactory.getHen(Continent.AFRICA); hen.getMonthlyEggCount(); } static class HenFactory { static Hen getHen(String continent) { Hen hen = null; switch (continent) { case Continent.NORTHAMERICA: hen = new NorthAmericanHen(); break; case Continent.EUROPE: hen = new EuropeanHen(); break; case Continent.AFRICA: hen = new AfricanHen(); break; case Continent.ASIA: hen = new AsianHen(); break; } return hen; } } }

Comments (7)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

23 February 2019, 00:50

Probably because switching on strings can throw a nullpointer exception. Try wrapping lines 18-21 in a try-catch block with the catch block being empty.

Araceli

Level 19 , Buenos Aires, Argentina

23 February 2019, 13:13

Thanks for the suggestion Guadalupe! But it still won't verify. I saw other questions and many users used an if else statment, so I changed it (although it is more or less the same). And still it doesn't work. I'm going crazy. I don't know how many attempts I've already made.

Guadalupe Gagnon

Level 37 , Tampa, United States

23 February 2019, 16:26

What is your code for the he then method now?

Araceli

Level 19 , Buenos Aires, Argentina

23 February 2019, 19:16

Like this:

static Hen getHen(String continent) {
            Hen hen = null;

                if (continent.equals(Continent.NORTHAMERICA)) {
                    hen = new NorthAmericanHen();

                } else if (continent.equals(Continent.EUROPE)) {
                    hen = new EuropeanHen();

                } else if (continent.equals(Continent.AFRICA)) {
                    hen = new AfricanHen();

                } else if (continent.equals(Continent.ASIA)) {
                    hen = new AsianHen();

                }
                return hen;
            }

Guadalupe Gagnon

Level 37 , Tampa, United States

25 February 2019, 15:51

Try this:

return super.getDescription() + " I come from North America. I lay " + getMonthlyEggCount() + " eggs a month.";
                                                // ↑ make this NorthAmerica instead

Change North America to NorthAmerica

Araceli

Level 19 , Buenos Aires, Argentina

26 February 2019, 13:33

It did not work but your suggestion gave me an idea. Instead of using Continent.NORTHAMERICA in the Solution, I used "North America" and verified! It does not make sense to me but it works (?) So thank you!

static Hen getHen(String continent) {
            Hen hen = null;

                if (continent.equals("North America")) {
                    hen = new NorthAmericanHen();

                } else if (continent.equals(Continent.EUROPE)) {
                    hen = new EuropeanHen();

                } else if (continent.equals(Continent.AFRICA)) {
                    hen = new AfricanHen();

                } else if (continent.equals(Continent.ASIA)) {
                    hen = new AsianHen();

                }
                return hen;
            }

Guadalupe Gagnon

Level 37 , Tampa, United States

26 February 2019, 14:17

It looks like CodeGym fixed a bunch of spelling errors in this task just recently. Your code and my code both had "NorthAmerica" as the starting value of the Continents.NORTHAMERICA, and my code verified successfully when I completed this task a few months ago. I didn't realize that they fixed that error and the verification system uses "North America" now.