Почему не корректно работает фильтр по прототипу?

Question

[kikosko]

Есть метод "kick" в классе "Team" и функция "typeFirstShuffle" суть которой заключается в перетасовке "target" по ее типу (типы: "archer", "swordsman", "mage") (сначала выводятся цели одинакового типа, а дальше если показатель "currentHealth" < 0 урон наносится другим живим целям), но метод "kick" работает не корректно. Когда происходит метод "battle" в классе "Game" урон наносит только "archer" по очереди по всем типам, дальше "swordsman" так же и тд. Так же, урон наносит только одна команда (то есть "archer" из "team1", по очереди бьёт все типы, потом "swordsman" и "mage"), а герои с другой команды не атакуют. Помогите реализовать идею, где:"arher1" сначала бьёт "arher2" (другие типы так само), потом наоборот, а если "currentHealth" цели с одинаковым типом ниже нуля, тогда искать случайно среди других типов
Вот мой код

Comments

[Bhudh]

(a.type === type) - (b.type === type)
Вы можете внятно объяснить, что Вы делаете в этом выражении?

Когда происходит метод "battle" в классе "Game" урон наносит только "archer" по очереди по всем типам

Так Вы сами поставили цикл на одного члена команды до полной победы над противником:

while(enemies.length && member.isAlive()) {
            member.kick( enemies[0] );
            if(!enemies[0].isAlive()) enemies.shift();
        }

А потом остальные члены команды бьют уже дохлых орков.

[kikosko]

Bhudh, Переделал таким образом, все равно не работает по логике сначала убийства цели того же типа, а потом поиска случайной цели

Team.prototype.selectRandomTarget = function (targetTeam) {
    if (!(targetTeam instanceof Team)) {
        throw new TypeMismatchError("targetTeam must be an instance of 'Team'");
    }
    var numberOfMembers = targetTeam.members.length;
    var target = null;
    while(target == null || !target.isAlive())
    {
        var randomIndex = Math.floor(Math.random() * numberOfMembers);
        target = targetTeam.members[randomIndex];
    }
    return target;
};
Unit.prototype.SelectSameType = function(targetTeam)
{ //Iterate through all the players and look for at least one of the same type that's alive
    for(let i = 0; i < targetTeam.length; i++)
    {
        let target = targetTeam[i];
        if(target.type === this.type && target.isAlive()) return target;
    }
    return null;
};
Team.prototype.kick = function(targetTeam) {
    console.log(`\nTeam ${this.name} is attacking`);

    for(var i=0; i < this.members.length; i++) {
        var singleMember = this.members[i];

        if(!this.isAlive() || !targetTeam.isAlive()) break;
        if(!singleMember.isAlive()) continue;

        //Check for the same type first
        var target = singleMember.SelectSameType(targetTeam)
        if(target === null) target = this.selectRandomTarget(targetTeam);

        singleMember.kick(target);
        if(target.isAlive()) target.kick(singleMember);
    }
};

[Bhudh]

var target = null;
    while(target == null || !target.isAlive())

Подумайте, что происходит при первой проверке условия?