I don't understand why not met 5. and 6. requirements

Question about the task

Resolved

After many attempts, I don’t understand why it always reject requirements 5 and 6. Maybe I didn’t well understand something from the requirements, I don’t know. The program seems to be working.

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; //write your code here if (continent.equals(Continent.EUROPE)) { hen = new EuropeanHen(continent, 24); } else if (continent.equals(Continent.NORTHAMERICA)) { hen = new NorthAmericanHen(continent, 22); } else if (continent.equals(Continent.ASIA)) { hen = new AsianHen(continent, 25); } else if (continent.equals(Continent.AFRICA)) { hen = new AfricanHen(continent, 28); } return hen; } } }

Comments (7)

Popular
New
Old

You must be signed in to leave a comment

dhanush

Level 30 , Chennai, India

20 February 2021, 03:31

Read the requirement number 5 carefully, classes must implement the getMonthlyEggCount method, which should return the number of eggs per month laid by the corresponding type of chicken. you should not return the field and you also don't need to create a constructor for the chickens. you also need to rewrite the Solution class's getHen method also without the parameters in the chickens class. In the getDescription method you should call the continent using the Continent Interface.

Gellert Varga

Level 23 , Szekesfehervar, Hungary

20 February 2021, 09:52

I have read this requirement many times: "classes must implement the getMonthlyEggCount method, which should return the number of eggs per month laid by the corresponding type of chicken." 1) Please define to me (point to) what my getMonthlyEggCount method returns instead of above cited requirement. (Because I really can't figure out the difference.) 2) I tried also without a new field (monthlyEggCount) so that the getMonthlyEggCount() method returned only a simple fictional number, but the verifier didn't accept it. So, this way was rejected too:

public int getMonthlyEggCount() {
     return 22;
}

and in an other type of chicken: return 25; etc.

Nouser

Level 36 , Germany

20 February 2021, 10:28

solution

I understand requirement 1 to be final, so that you should add just the two mentioned methods and nothing else. Of course your solution works. However the valiator may check for specific things you won't expect. That may be the usage of the constants EUROPE, ASIA.. in the getDescription methods or the validator calls for its tests a default constructor that you do not have. And giving the class' user the possibility to change the continent via the constructor isn't a really good idea. Then you might end up with a Hen of type AfricanHen from continent Europe laying as many eggs as the class user wishes. In my book that should be a constant (defined inside the class). As said, technically your solution is good as you bundle state inside the parent class. It is just not asked to do that here. But of course I might be wrong with my assumptions.

Guadalupe Gagnon

Level 37 , Tampa, United States

20 February 2021, 16:18

solution

Nouser is correct. The validation system will test your code to ensure that the directions were followed. One of the biggest reason people fail tasks is because they add too much to their code that the directions don't call for. In the code above, the directions do not say to add a constructor, so when the test code tries to create one of your objects:

AfricanHen test = new AfricanHen();

its going to get an error saying that there is no default constructor. That would be a fail. So, to fix this code you are going to need to make sure that only the directions are followed and nothing more: #1 get rid of the constructors in the African, European, NAmerican, and Asain Hen. #2 get rid of the fields for egg count and continent in the Hen Class #3 rewrite the getMonthlyEggCount() method in the 4 child Hens to return an int, just like the code you shared 2 posts above. #4 Rewrite the getDescription() in the 4 child Hens so it doesn't use the continent field that was removed. #5 Rewrite the getHen() method so that it uses default constructors.

Gellert Varga

Level 23 , Szekesfehervar, Hungary

20 February 2021, 21:28

Thanks Nouser, for your comments! Now I understand what the problem was. Using these, the task was successfully validated now.:)

Gellert Varga

Level 23 , Szekesfehervar, Hungary

20 February 2021, 21:28

Thanks Guadalupe, for your comments! Now I understand what the problem was. You and Nouser both wrote very good things:) Using these, the task was successfully validated now.

John

Level 17 , Mansfield, Philippines

2 June 2021, 14:00

anybody who can tell me the right use of String.format -_-