Help with Mad Maxer Code

Level Help
1 
while (enemyIndex < enemies.length) {
        var target = enemies[enemyIndex];
        enemyIndex += 1;

        // Is this enemy farther than the farthest we've seen so far?
        var distance = hero.distanceTo(target);
        if (distance > maxDistance) {
            maxDistance = distance;
            farthest = target;
        }
    }

    if (farthest) {
        // Take out the farthest enemy!
        // Keep attacking the enemy while its health is greater than 0.
      var enemy= hero.findNearestEnemy();
       
        while (enemy.health>0) {
            hero.attack(enemy);
        }
    }
}

I donā€™t understand what I am doing wrong
My hero just keeps on attacking the decoys and not the ogres

2

Hi @Falcons118, welcome to the forum.
You seem to be missing some code at the top. Please could you post all of it. And if you put ```javascript at the start of your code it recognises the javascript comments, meaning the whole code isnā€™t red.
Thanks

3

Sorry about thatā€¦ This is my first time posting something on this website

4 
while(true) {
    var farthest = null;
    var maxDistance = 0;
    var enemyIndex = 0;
    var enemies = hero.findEnemies();

    // Look at all the enemies to figure out which one is farthest away.
    while (enemyIndex < enemies.length) {
        var target = enemies[enemyIndex];
        enemyIndex += 1;

        var distance = hero.distanceTo(target);
        if (distance > maxDistance) {
            maxDistance = distance;
            farthest = target;
        }
    }

    if (farthest) {
        // Take out the farthest enemy!
        // Keep attacking the enemy while its health is greater than 0.
      var enemy= hero.findNearestEnemy();
       
        while (enemy.health>0) {
            hero.attack(enemy);
        }
    }
}
5

@Deadpool198 is that better

6

@Deadpool198 can you help me now.

7

Inside the if farthest, what variable have you used to attack?

8

@Deadpool198 Isnā€™t the while loop supposed to be in there

9

You went through all the bother of determining ā€˜farthestā€™, but now you arenā€™t going to use it?

10

@dedreous I donā€™t know how to use it. And when I click on the hints(in CodeCombat )thereā€™s nothing useful that will help me in that level

11

Ok, letā€™s fix that thenā€¦I hope Danny wonā€™t mind my stepping in here :wink:
Give me a few to take a closer look and Iā€™ll repost my suggestions.

In your ā€˜if (farthest)ā€™ code block, you are now re-defining ā€˜enemyā€™. You donā€™t need to do this, as ā€˜farthestā€™ is already defined as the enemy you want to take out first.

Attack farthest, not enemy.

12

Additionally, the Hints are an excellent resource, but not always revealing. It is always a good idea to have a look at them, but then do not be afraid to ask for help if they were not enough.

It is a very narrow line between giving away too little and giving away to much. The goal here is to help folks learn, after all :slightly_smiling_face:

13

@dedreous Thanks for the help

14

never mind it doesnā€™t work

15

My hero just says ā€œbut its deadā€

16

Can you tell me what I need to change now @dedreous

17

Repost your code, as it is nowā€¦Iā€™ll try to see if I can spot it.

18 

while(true) {
    var farthest = null;
    var maxDistance = 0;
    var enemyIndex = 0;
    var enemies = hero.findEnemies();
    var enemy = hero.findNearestEnemy();

    while (enemyIndex < enemies.length) {
        var target = enemies[enemyIndex];
        enemyIndex += 1;

        var distance = hero.distanceTo(target);
        if (distance > maxDistance) {
            maxDistance = distance;
            farthest = target;
        }
    }

    if (farthest) {
        // Take out the farthest enemy!
        // Keep attacking the enemy while its health is greater than 0.
        while(enemy.health>0) {
            hero.attack(farthest);
        }
    }
}
19

I changed the attack enemy to attack farthest

20

@dedreous isnā€™t that what you told me to change