Неправильно. Прототип функции вот такой: native get_players(players[32], &num ,const flags[]="", const team[]="");
&num - амперсанд (это симол &) означает, что в этот аргумент будет установлено значение самой функцией. Т.е. в твоем случае делать i = 1 глупо, т.к. i заполняется после вызова самой функции, и означает кол-во игроков, которые были найдены (их идентификаторы находятся в массиве players). После вызова функции проверяй значение i, и если оно равно 1, то это (player[0]) последний живой игрок.
