/ip - Проверка IP адреса

[checke7 Хороший 29]

Данная команда создана для проверки IP адреса игрока.

CMD:ip(playerid, params[])
{
	// ваша проверка на администратора
	if(isnull(params))
		return 0;
	if(sscanf(params, "u", params[0]))
		return SendClientMessage(playerid, 0xFF0000FF, "{FFC800}[Команда] {FFFFFF}Используй: {4080FF}/ip [ID]") & 0;
	if(params[0] == INVALID_PLAYER_ID) 
		return 0;
	
	new string[59+(-2+15)+1];
	GetPlayerIp(params[0], string, sizeof string);
	format(string, sizeof string, "{FFC800}[Информация] {FFFFFF}IP игрока на данный момент: %s", string);
	SendClientMessage(playerid, 0xFF00FFFF, string);
	return 1;
}

 

Отредактировано 5 июня, 2019 пользователем odosenok
Причина: поправил команду.

[Cawfee Максимальный 558]

Немножко подправил команду. @Rich_Hen, коротко об изменениях:

1. Спецификатор "u" в нативной функции sscanf возвращает либо ID введенного игрока, либо INVALID_PLAYER_ID (игрок не подключен). То есть сама библиотека вместо вас проверяет, подключен ли вообще такой игрок. Поэтому использовать IsPlayerConnected для params[0] не есть логично правильно.
2. Вы всегда должны иметь возможность отслеживать результат работы любой функции. Потому не нужно от балды лепить "return true" (1) или "return false" (0). Каждый литерал должен нести соответствующий смысл: если функция успешно выполнена, возвратите успех (1), в противном же случае сделайте возврат неудачи (0). Так, в случае, если не введено ничего в поле аргументов (первое условие) или игрок не подключен (второе условие), мы возвращаем неудачу, ибо функция не выполнилась как нужно.
3. В начале я добавил проверку "isnull". Это макрос, который проверяет, не является ли строка пустой. Таким образом, прежде, чем вызывать нативную функцию, мы используем обычную проверку на по типу "a != b?". Ибо время вызова нативной функции во много раз больше времени работы этого макроса (понятно, что это тысячные доли секунд, но так или иначе есть понятие оптимизация).
4. Использовались функции, которые необязательно должны быть во всех игровых модах (типа SendServerMessage). Не следует выкладывать на публику код, имеющий большое число зависимостей. Это же касается и константных выражений, присутствующих исключительно в вашем моде.

[HamanSamp 0]

Стандартный код, нахрена?

Убери этот код, от него глаза болят!

 

Pawn.CMD /
CMD:ip(playerid,params[])
{
	// Проверка на админку
	static
		id,
		ip[16],
		string[50+16];
	if(sscanf(params,"i",id))
		return SendClientMessage(playerid,-1,"CMD: /ip [id]");

	if(id == INVALID_PLAYER_ID)
		return SendClientMesage(playerid,-1,"Такого ID не существует!");

	GetPlayerIp(id,ip,16);

	format(string,50+16,"[Информация]: У игрока под ID: %i, IP адрес: %s",
		id,ip);
	SendClientMessage(playerid,-1,string);

	string[0] = EOS;
		
	return 1;
}

 

[Cawfee Максимальный 558]

@HamanSamp, и что вы этим добились? Если игрок под ID 100 не в сети, то команда все равно для него будет работать, но некорректно. Так что у меня большие сомнения, от чьего кода болят глаза.

Статические переменные здесь тоже бессмысленны. Команда явно не будет использоваться каждую минуту, что жаль тратить время на выделение данных.

[HamanSamp 0]

@odosenok Хах. Про static давно знаком? Если незнаком, можешь прочитать, на многих форумах их полно, разницы нету. Я про IsPlayerConnected помолчу, ведь лучше всегда работать через переменную, которая указывала бы мне авторизацию игрока. 

И от выше кода человека, глаза плавятся. 

 

 

[DEST Максимальный 1 032]

В самой ф-ии GetPlayerIp есть проверка на подключение игрока, см. https://wiki.sa-mp.com/wiki/GetPlayerIp_RU. 

[Cawfee Максимальный 558]

@HamanSamp, спецификатором, указывающим на ID полученного игрока, "i" не является. Я вам указыввю на конкретный косяк (хотя вариант @DEST гораздо практичнее (здесь признаю и свою недоработку)), а вы мне утверждаете обратное. В моих знаниях оператора static можете не сомневаться. Я уже аргументировал: не вижу смысла хранить переменные данной команды в сегменте данных, поскольку эта команда используется нечасто. То же самое, что в глобальные переменные вынести и использовать в участке кода, который будет использоваться от силы пару раз в сутки. Хотя по сути выгоды от этого никакой.

[DEST Максимальный 1 032]

"u" удобней использовать, так как не только ID можно вводить, но и ник, и часть ника и все это будет конвертировано в ID. 

По поводу переменной, хранящей состояние игрока (подключен или нет). Фактически тоже самое реализовано в самом сервере и проверяется с помощью ф-ии IsPlayerConnected, которая, в свою очередь, встроена во многие другие стандартные функции. 

[HamanSamp 0]

@DEST не думаю что это бы являлось практичной. Ведь, если игрок не авторизовался на сервере, его нет смысла проверять, неверно ли? @odosenok А насчёт static, команду используют ежечасно по несколько раз, есть смысл ставить static. 

И да, насчёт ассамблера, на форуме его найти можно? Глубоко изучить хочется, а то на разных форумах ничего не понятно.

[Cawfee Максимальный 558]

@HamanSamp, а вы думаете, что "u" в спецификаторе как-то по-другому работает? Точно также проверяет.

[hahdhhs 0]

@HamanSamp Внимательние

  открыть спойлер

CMD:ip(playerid,params[])
{
    //проверка на админа
    static
        id,
        ip[16],
        string[50+16];
    if(sscanf(params,"i",id))
        return SendClientMessage(playerid,-1,"CMD: /ip [id]");

    if(id == INVALID_PLAYER_ID)
        return SendClientMessage(playerid,-1,"Такого ID не существует!");

    GetPlayerIp(id,ip,16);

    format(string,50+16,"[Информация]: У игрока под ID: %i, IP адрес: %s",
        id,ip);
    SendClientMessage(playerid,-1,string);

    string[0] = EOS;

    return 1;
}

 

 

Отредактировано 10 июня, 2022 пользователем keyl