Hey, need help with Library Tactician

Level Help
1

Just keep getting an infinite loop. No idea what to do

# Hushbaum has been ambushed by ogres!
# She is busy healing her soldiers, you should command them to fight!
# The ogres will send more troops if they think they can get to Hushbaum or your archers, so keep them inside the circle!

# Soldiers spread out in a circle and defend.
def commandSoldier(soldier, soldierIndex, numSoldiers):
    angle = Math.PI * 2 * soldierIndex / numSoldiers
    defendPos = {"x": 41, "y": 40}
    defendPos.x += 10 * Math.cos(angle)
    defendPos.y += 10 * Math.sin(angle)
    hero.command(soldier, "defend", defendPos);

# Find the strongest target (most health)
# This function returns something! When you call the function, you will get some value back.
def findStrongestTarget():
    mostHealth = 0
    bestTarget = None
    enemies = hero.findEnemies()
    # Figure out which enemy has the most health, and set bestTarget to be that enemy.
    for enemy in enemies:
        if enemy.health > mostHealth:
            
            mostHealth = enemy.health
            bestTarget = enemy
        
    # Only focus archers' fire if there is a big ogre.
    if bestTarget and bestTarget.health > 15:
        return bestTarget
    else:
        return None


# If the strongestTarget has more than 15 health, attack that target. Otherwise, attack the nearest target.
def commandArcher(archer):
    nearest = archer.findNearestEnemy()
    if archerTarget:
        hero.command(archer, "attack", archerTarget)
    elif nearest:
        hero.command(archer, "attack", nearest)

archerTarget = None

while True:
    # If archerTarget is defeated or doesn't exist, find a new one.
    if not archerTarget or archerTarget.health <= 0:
        # Set archerTarget to be the target that is returned by findStrongestTarget()
        archerTarget = findStrongestTarget()
    
    friends = hero.findFriends()
    soldiers = hero.findByType("soldier")
    # Create a variable containing your archers.
    archers = hero.findByType("archer")
    for i in range(len(soldiers)):
        soldier = soldiers[i]
        commandSoldier(soldier, i, len(soldiers));
    # use commandArcher() to command your archers
    for archer in archers:
        commandArcher(archer)
        pass
2

Just some better-formatting tips:

length = len(soldiers) # no point in constantly requesting the length
for i in range(length):
    commandSoldier(soldiers[i], i, length) # no point in defining a new variable
# next tip
for archer in archers:
    commandArcher(archer) # no 'pass' needed
#  next tip 
friends = hero.findFriends() # delete this, you don't ever use it
# next tip
for enemy in hero.findEnemies(): # no point in defining a new variable
    if enemy.health > mostHealth:
        mostHealth = enemy.health
        bestTarget = enemy
# next tip
if bestTarget and bestTarget.health > 15:
   return bestTarget
return None # no point in putting an 'else' since this won't ever execute if the above is true
# next tip
if archerTarget:
    hero.command(archer, "attack", archerTarget)
else:
    nearest = archer.findNearestEnemy() # no point in defining it when you aren't going to use it
    if nearest:
        hero.command(archer, "attack", nearest)

And I can’t see any reason for it giving an infinite loop. (Except you using Python)

3

Do you have a GitHub account?