Help mountain mercenary javascript

Level Help
1 
// Gather coins to summon soldiers and have them attack the enemy.

while(true) {
    // Move to the nearest coin.
    // Use move instead of moveXY so you can command constantly.
    var item = hero.findNearestItem();
    hero.move(item.pos);
    
    // If you have funds for a soldier, summon one.
    if (hero.gold > hero.costOf("soldier")) {
        hero.summon("soldier");
    }
    var enemy = hero.findNearest(hero.findEnemies());
    if (enemy) {**strong text**
        
        var soldiers = hero.findFriends();
        // Loop over all your soldiers and order them to attack.
        for (soldier in soldiers) {
            // Use the 'attack' command to make your soldiers attack.
            hero.command(soldier, "attack", enemy);
            soldierIndex += 1;
        }
    }
}

@Falcons118 or @Deadpool198 or @milton.jinich

2

try putting a if after the loop since you may not have any alive troops.

3

No, the soldiers never attack. Ever.

4

Try putting this in a if. Than your code is identical to mine(but in JS)

5

I think it’s the for loop.

6

well, soldierIndex isn’t defined.

7

put soldierIndex = 0 out of the loop.

8

@Dragonlouis
instead of

 for (soldier in soldiers)

change that too

for (var i=0; i<soldiers.length;i++){
var friend=soldiers[i];
var enemy= friend.findNearestEnemy();
//now if friend.type=="soldier" and there is an enemy
command friend to attack the enemy
}
9

also put soldierIndex = 0; in there

10

image
image1627×900 455 KB

11

well you dont really need the soldierIndex

12

for var soldier in soldiers

13

no do what I said in the post above

this should work

14

because I dont think you are looping over all of your soldiers

15

if you want to do soldierIndex += 1 you have to add it.