I don't understand the logic of this game

Question about the task

Under discussion

The new body part must be added to the logic for the attack and defend methods in the AbstractRobot class.

1. Familiarize yourself with the code.
2. Accept your fate and keep trying to understand the code.
3. ...
4. Rejoice that you've figured it out.
5. Adapt the code to the following new architecture and add new logic:
5.1. Make the AbstractRobot class abstract. Then move the attack and defend logic from the Robot class to the AbstractRobot class.
5.2. Edit the Robot class to utilize the AbstractRobot class.
5.3. Extend the BodyPart class with a new body part: BodyPart.CHEST.
5.4. Add the new body part to the implementation of the CanAttack and CanDefend interfaces (in the AbstractRobot class).

Requirements:

The AbstractRobot class must be abstract.
The AbstractRobot class must implement the CanAttack and CanDefend interfaces.
The Robot class must inherit the AbstractRobot class.
The logic defining the how a Robot behaves must be in the AbstractRobot class.
The BodyPart class must have and initialize a final static BodyPart variable called CHEST.
The new body part must be added to the logic for the attack and defend methods in the AbstractRobot class.

package com.codegym.task.task13.task1328; interface CanDefend { BodyPart defend(); }

Comments (6)

Popular
New
Old

You must be signed in to leave a comment

Rich

Level 23 , San Diego, United States

2 October 2020, 21:28

look at what the code is actually doing when it runs. what parts should be increasing the hit count and what should or should not reset it to 0

Guadalupe Gagnon

Level 37 , Tampa, United States

3 January 2019, 18:42

Abstract robot class, both line 16 and line 33 have: else if (hitCount == 4) if the last hit count was 3 and you increase it by two (as in the defense), you will be left with a hit count of 5 and none of your if statements will be true thereafter. This means that null will be returned as the body point from that point on. Change both of these to just else and not if else (logic).

+14

Picvajzmen

Level 20 , Belgrade, Serbia

3 July 2019, 11:35

Can you explain it more please? Or anyone else, because i still cannot verify it even with doing what you said... public BodyPart attack(){BodyPart attackedBodyPart = null; hitCount = hitCount + 1; if (hitCount == 1) { attackedBodyPart = BodyPart.ARM; } else if (hitCount == 2) { attackedBodyPart = BodyPart.HEAD; } else if (hitCount == 3) { hitCount = 0; attackedBodyPart = BodyPart.LEG; }else //hitCount = 0; attackedBodyPart = BodyPart.CHEST; return attackedBodyPart;} public BodyPart defend(){BodyPart defendedBodyPart = null; hitCount = hitCount + 2; if (hitCount == 1) { defendedBodyPart = BodyPart.HEAD; } else if (hitCount == 2) { defendedBodyPart = BodyPart.LEG; } else if (hitCount == 3) { hitCount = 0; defendedBodyPart = BodyPart.ARM; }else defendedBodyPart = BodyPart.CHEST; return defendedBodyPart;}

Picvajzmen

Level 20 , Belgrade, Serbia

4 July 2019, 06:33

solved it.. deleted the line that resets the hitCount to zero...

Brad Reed

Level 19 , Hartford, United States

16 July 2020, 21:46

I had a similar code, and doing that gives all kinds of errors that cannot be resolved unless i do else if.

TomL

Level 30 , Prague, Czech Republic

25 January 2021, 10:57

It helped me, thanks. But think hitCount = 0; should be inside else statement after CHEST..