Getting the right output, but not passing validation. Help please :)

Question about the task

Chicken factory

Java Core, Level 4, Lesson 6

Resolved

Can someone, please, explain to me why my code is not passing validation?

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 NAHen = HenFactory.getHen(Continent.NORTHAMERICA); Hen EUHen = HenFactory.getHen(Continent.EUROPE); Hen AsHen = HenFactory.getHen(Continent.ASIA); Hen AfHen = HenFactory.getHen(Continent.AFRICA); int eggsMonthly; if ((eggsMonthly = NAHen.getMonthlyEggCount()) > 0) System.out.println(NAHen.getDescription()); if ((eggsMonthly = EUHen.getMonthlyEggCount()) > 0) System.out.println(EUHen.getDescription()); if ((eggsMonthly = AsHen.getMonthlyEggCount()) > 0) System.out.println(AsHen.getDescription()); if ((eggsMonthly = AfHen.getMonthlyEggCount()) > 0) System.out.println(AfHen.getDescription()); } static class HenFactory { static Hen getHen(String continent) { Hen hen = null; //write your code here if ("NorthAmerica".equalsIgnoreCase(continent)) { hen = new NorthAmericanHen(continent); } else if ("Europe".equalsIgnoreCase(continent)) { hen = new EuropeanHen(continent); } else if ("Asia".equalsIgnoreCase(continent)) { hen = new AsianHen(continent); } else if ("Africa".equalsIgnoreCase(continent)) { hen = new AfricanHen(continent); } return hen; } } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Michael Martin

Level 19 , Arlington, United States

23 November 2018, 14:40

The getDescription() method in the Hen class has an extra space after the period in the return string. Remove that extra space, should fix one of your problems.

Nemanja Curufinwë Kuzmanovic

Level 18 , Belgrade, Serbia

25 November 2018, 12:58

Thank you, but I figured out what the problem was. I was using a country instance variable, but it was not needed. All that was needed was to hard code the country of origin. Space was the problem as well, but that one I do not understand why. I thought it easier to put one space there than to put space in three separate places to do the same thing.