Вопрос Проблема Почему не обновляет переменную с балансом игрока?

[dereksjq]

Спойлер: Команда

mp.events.addCommand("setmoney", (player, target, amount) => {
    if (amount == null) {
        player.outputChatBox("/setmoney [ID] [Money]")
        return;
    }

    if (GetPlayerByID(target) == null) {
        player.outputChatBox("Игрока нет на сервере!")
        return
    }

    GetPlayerByID(target).setVariable("cash", amount)
    player.outputChatBox(`Вы успешно установили $${amount} игроку !{#FF0000}${GetPlayerByID(target).name}`)
    GetPlayerByID(target).outputChatBox(`Администратор !{#FF0000}${player.name} !{#FFF}установил вам новый баланс. Ваш баланс: !{#FF0000}${GetPlayerByID(target).getVariable("cash")}`)
})

Спойлер: Функция получение игрока, как объект по его ИД

function GetPlayerByID(idPlayer) { return mp.players.at(parseInt(idPlayer)); }

Пробовал так же через data.cash - так же, только вместо "0" - "null0".
В вики про рейдж вообще setMoney, но оно у меня вызывает ошибку, мол нет такой функции.

UPD: возможно, что что-то не так с переменной "amount". Но только что?

 

[dereksjq]

up

 

[youngBeaver]

Попробуй сначала получать в переменную таргет сущность, только потом через setvariable привязывать данные

UPD: Открыл вики, addCommand принимает вторым аргументом полный набор всех аргументов. Следственно, когда ты ищешь игрока по target, то ты получаешь помимо ид еще слитно количество денег. Из чего делаем вывод, что тебе нужно 4 параметра (player, fullText, targetID, amount)

 

[XDeveluxe]

Во-первых:
Зачем каждый раз выискивать игрока из пула всех игроков?

Почему бы не получить его 1 раз, не записать его в отдельную переменную и не работать далее с ней?

let targetPlayer = GetPlayerByID(target);

Далее взаимодействуешь с targetPlayer.

Во-вторых:
Вполне вероятно, что у тебя проблема не с переменной "cash", а с переменной "amount", потому что при выводе мы видим, что {amount} равняется 0. Этот же 0 и записался в "cash".
В JS можно удобно проверять переменную сразу на все ненужные варианты по типу "пустой, undefined, null и т.д" с помощью:

if (!amount)
{
    player.outputChatBox("/setmoney [ID] [Money]")
    return;
}

 

[frytech]

Ну и код конечно, полный пиздец
Комментаторы выше безусловно правы, только это проблему не решит
У тебя amount либо undefined либо 0, вот и все, т.е. даже не передается значение

 

[youngBeaver]

Зачем каждый раз выискивать игрока из пула всех игроков?
Посмотреть вложение 10351

Почему бы не получить его 1 раз, не записать его в отдельную переменную и не работать далее с ней?

let targetPlayer = GetPlayerByID(target);

Далее взаимодействуешь с targetPlayer.

Я обновил свой комментарий, там в другом вероятнее всего ошибка. А так да, легче занять одну строчку и проинициализировать переменную таргетом, чем заниматься такими приколами

 

[dereksjq]

Спасибо всем за критику, возьму на заметку.
Проблема была в fullText, как описал wholinc.

 

[youngBeaver]

Спасибо всем за критику, возьму на заметку.
Проблема была в fullText, как описал wholinc.

удачной разработки