В теме 11 сообщений

Данная команда создана для проверки 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;
}

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Немножко подправил команду. @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). Не следует выкладывать на публику код, имеющий большое число зависимостей. Это же касается и константных выражений, присутствующих исключительно в вашем моде.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

 

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;
}

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

 

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

Заметка от Cawfee , создано

Ваш вопрос не соответствует содержимому данной темы, что приравнивается к оффтопу. Пожалуйста, оставьте свой вопрос в соответствующем разделе ("Флейм"), предварительно воспользовавшись поиском по форуму.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

@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;
}


 

 

Отредактировано пользователем keyl

Заметка от keyl , создано

3.2. Код больше 10 строк необходимо брать в теги "Код" и "Спойлер". Как это сделать рассказано в разделе "FAQ".

Устное предупреждение.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

Вы должны быть пользователем, чтобы оставить комментарий

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!


Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.


Войти

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу

  • Похожий контент

    • Mister Odin
      От Mister Odin
       
      new
       g_teleport_list[39][E_TELEPORT_LIST] = {  {"Спавн пгт.Батырево", 1802.3612, 2508.4824, 15.8887},  {"Спавн г.Арзамас", -113.5887, 977.7222, 12.0346},     {"Спавн г.Южный", 2744.2770, -2446.7246, 21.8988},     {"Центральный банк", 1852.020385,2040.791381,15.892713},  {"Рублёвка", -837.7783, 888.6161, 12.7232},  {"Казино", 1332.4252, 2358.7014, 17.6641},  {"Битва за контейнеры", 614.5167, 1722.3566, 12.0709},  {"Автосалон низкого класса", 2493.9616, -722.9544, 12.3315},  {"Автосалон среднего класса", 1407.7449, 455.4801, 13.1630},  {"Автосалон высокого класса", -14.0637, 2614.5664, 10.9892},   {"Мотосалон Harley Davidson", 785.720153, 750.465087, 12.000024},  {"Автошкола", 1909.174438,2227.679687,15.708162},  {"Военкомат", 1916.778564,2302.411376,15.574637},  {"Правительство области", -139.140975,593.789611,12.145712},  {"Отдел полиции №1 (ГИБДД)", 1906.805786,-2234.382812,11.257631},  {"Отдел полиции №2 (УМВД)", 2581.309326,-2416.139892,21.960090},  {"Отдел ФСБ", 1823.904663,2095.636718,15.848405},  {"Городская больница г.Арзамас", -285.797210,581.851562,12.120290},   {"СМИ", -317.708038,821.221679,13.051450},  {"Воинская часть", 1703.197265,1678.025878,15.279437},  {"Арзамасская ОПГ", 438.705322,1046.505126,12.002637},  {"Батыревская ОПГ", 1941.203857,2161.006591,15.705187},  {"Лыткаринская ОПГ", -2346.002441,75.257041,21.002962},  {"Шахта", 2381.5727, 1726.4451, -2.1506},  {"Завод", -1062.6697, 2204.1894, 38.0964},  {"Инкосация", 1864.1374, 2011.7136, 15.8546},  {"Транспортная Компания", 2362.1115, 1972.5627, 15.5530},  {"База механиков", 1840.918090,-118.025146,15.695312},  {"Курьерская служба доставки", 2764.509765,-2396.882568,21.890625},  {"Аренда автобуса(Южный)", 2771.041992,-2454.780761,21.845964},  {"Аренда автобуса(Арзамас)", -127.487670,943.652648,12.142824},   {"Аренда автобуса(Батырево)", 1798.986328,2529.591552,15.664262},  {"Гоночная трасса", -1568.7784, 1611.8305, 36.3971},  {"Дом на горе", -789.5397, -456.7479, 741.1422},  {"СТО и тюнинг-ателье", 1856.1375, -122.3266, 15.6888},  {"Аренда транспорта пгт.Батырево", 1763.641723,2255.673339,15.865348},  {"Аренда транспорта г.Арзамас", -36.923530,1360.673583,12.002090},  {"Аренда транспорта г.Южный", 1966.599975,-2603.943603,10.820312},  {"Тайный интерьер в Лыткарино", -2424.5818, 2846.3291, 40.7908} };