Цитата Сообщение от codestation Посмотреть сообщение
А еще ты игнорируешь, грубо говоря, общий стиль написания команд. Я про отсутствии params у viewreq.
Ну вообще вполне логично не прописывать "params", если команда не должна принимать дополнительные параметры. Это, как минимум, улучшает читаемость, ибо достаточно глянуть на шапку команды, чтоб понять, принимает команда параметры или нет.

Ну и это исключает возможные опечатки с тем, что ты случайно пропишешь в команде "params" (например, в format на автомате пропишешь не "giveplayerid", а "params[0]"), вместо какой-то другой переменной/массива. В случае, если пихать "params" всюду, то компилятор не сообщит об ошибке.
Это, к слову, относится ко всем случаям с необязательными переменными, которые так любят пихать новички ради "экономии" времени (не говоря уже о том, что это лишь раздувает потребление сегмента данных). Например, очень часто встречал ошибку, когда у человека не показывает какое-то форматируемое сообщение, которое он начинал записывать в локальный массив, а уже в SendClientMessage прописывал глобальный массив.
В общем, лучше не ленитесь и, по возможности, максимально пользуйтесь локальными переменными. Даже если в каждом блоке условия вы начнёте указывать свой массив, а-ля "string", а не один общий - это будет лучше, чем потом вы схватите какой-то баг, который отловите только уже когда вы/кто-то из игроков его заметит и вы отыщите истинную причину.