Правильно ли я сделал

Andrey33:

NazarPunk, Он же делает подсчет сколько игроков играет и устанавливает уровень на количество игроков, разве не так?

Но не сбрасывает значение переменной.

И по хорошему нужно не перебирать всех каждые две секунды, а использовать событие Игрок покидает игру. А нужных юнитов в группе хранить.

Во-первых, события следует ставить Юнит вошёл в игровую область и Игрок покинул игру. Во-вторых, переменную User можно напрямую задать, что это группа игроков, соответствующих условиям. В-третьих, ставишь действие Если/То/Иначе, где в если (Переключающий юнит) равно Герой равно Да, То установи уровень способностей, Иначе (пусто).

Andrey33:

Переименовать в CountPlayingHuman

Andrey33:

В том, что переменная глобальная и по такому названию через месяц можно и забыть зачем она нужна.

Сейчас сообразил, что там два триггера нужно. Первый почти такой же, как на последнем скрине, только там не надо Если/То/Иначе, перенеси условие из Если в условие триггера. Потом зря убрал выбор всех героев в группу, но тут моя вина, выбери всех героев в группу и поставь им действия из То. Только у тебя на всех героев это распространится, в том числе и вражеских, чтобы вражеские бонуса не получали, задай в сопоставлении кроме Герой ещё (Владелец юнита) не равно Игрок_за_мобов.
А во втором триггере делаешь событие, что Игрок оставил игру и действие Триггер - Выполнить Первый триггер, игнорируя условия.

Andrey33:

Вот так?

Нет. В переменную User поставь условия.
Самое первое действие.
Чтобы не было утечки в самый низ триггера (не в действия с группой, а отдельно под него) поставь Custom Script:

call DestroyGroup( GetLastCreatedGroup() )

Но в таком случае будет некритичное количество утечек, потому можно и без него.