/do

Вид для печати

13.03.2016, 21:15
Иван Бубнов

/do

Предназначение команды думаю объяснять не нужно. Баян в моем исполнении.

PHP код:

CMD:do(playerid, params[]) { if (isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, world, interior, name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1 char]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, name, sizeof(name)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); strcat(string, params); strcat(string, " ("); strcat(string, name); strcat(string, ")"); new iter = GetPlayerPoolSize(); #if defined foreach foreach(new i : Player) { #else #if defined GetPlayerPoolSize for(new i = -1; ++i != iter;) #else for(new i = -1; ++i != MAX_PLAYERS;) #endif #endif { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world ) || (GetPlayerInterior(i) != interior)) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }

Автор: Иван Бубнов
Специально для Pro-Pawn.Ru

Копирование данной статьи на других ресурсах без разрешения автора запрещено!
Спасибо всем за советы по коду!
13.03.2016, 21:25
DeimoS

Вот эту проверку

PHP код:

if(0 == IsPlayerConnected(i)) continue;

тоже стоит внести в "#if defined foreach ", не? Точнее, в "#if !defined foreach "
13.03.2016, 21:27
Иван Бубнов

Цитата:

Сообщение от DeimoS

Вот эту проверку

PHP код:

if(0 == IsPlayerConnected(i)) continue;

тоже стоит внести в "#if defined foreach ", не? Точнее, в "#if !defined foreach "

Не додумался чет :good:
13.03.2016, 21:27
DeimoS

Как-то так, короч

PHP код:

CMD:do(playerid, params[]) { if(isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, string[144 + 1 + (-2 + MAX_PLAYER_NAME)]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, string, sizeof(string)); format(string, sizeof(string), "%s (%s)", params, string); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if(0 == IsPlayerConnected(i)) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if(GetPlayerVirtualWorld(i) != GetPlayerVirtualWorld(playerid) || GetPlayerInterior(i) != GetPlayerInterior(playerid)) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }
13.03.2016, 21:30
Иван Бубнов

Цитата:

Сообщение от DeimoS

Как-то так, короч

PHP код:

CMD:do(playerid, params[]) { if(isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, string[144 + 1 + (-2 + MAX_PLAYER_NAME)]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, string, sizeof(string)); format(string, sizeof(string), "%s (%s)", params, string); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if(0 == IsPlayerConnected(i)) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if(GetPlayerVirtualWorld(i) != GetPlayerVirtualWorld(playerid) || GetPlayerInterior(i) != GetPlayerInterior(playerid)) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }

я по другому сделал, шапку апдейт уже, но твой вариант проще
13.03.2016, 21:51
ziggi

Ты ещё кое-что упустил.

PHP код:

CMD:do(playerid, params[]) { if(isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, world, interior, string[144]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, string, sizeof(string)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); format(string, sizeof(string), "%s (%s)", params, string); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if(0 == IsPlayerConnected(i) || 0 != IsPlayerNPC(i)) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if(GetPlayerVirtualWorld(i) != world || GetPlayerInterior(i) != interior) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }

P.S. foreach не включает в результат ботов, а второй цикл включает, поэтому нужно добавлять условие на IsPlayerNPC.
И больше 144 ячеек выделять смысла нет, всё равно не отобразится.
13.03.2016, 21:54
Иван Бубнов

Цитата:

Сообщение от ziggi

Ты ещё кое-что упустил.

Обновил шапку
13.03.2016, 21:59
ziggi

Цитата:

Сообщение от Иван Бубнов

Обновил шапку

Я отредактировал пост, добавив информацию о памяти. Также ты забыл об этом:

PHP код:

world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid);

(лучше вызывать эти функции один раз)
13.03.2016, 22:10
Иван Бубнов

Цитата:

Сообщение от ziggi

Я отредактировал пост, добавив информацию о памяти. Также ты забыл об этом:

PHP код:

world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid);

(лучше вызывать эти функции один раз)

Учусь

- - - Активировано защита от Стяжкина - - -

http://xn--90ai1alafcp.xn--p1ai/uplo...a4d276c52d.jpg
13.03.2016, 22:14
Desulaid

А может скреплять строки, а не форматировать?
13.03.2016, 22:19
Иван Бубнов

Цитата:

Сообщение от Anton Styazhkin

А может скреплять строки, а не форматировать?

Ты это имел ввиду?

string = params, strcat(string, " ("), strcat(string, string), strcat(string, ")");

PS: Правда на счет string = params не уверен.

Тогда придется еще ник записывать в другой массив
13.03.2016, 22:40
ziggi

Цитата:

Сообщение от Иван Бубнов

Учусь

А что на счёт размера string? Уже третий раз пишу, что выделять больше, чем 144 ячейки смысла нет.
13.03.2016, 22:41
$continue$

MAX_CHATBUBBLE_LENGTH + 1 (нулевой символ туда не входит)

Цитата:

Сообщение от ziggi

А что на счёт размера string? Уже третий раз пишу, что выделять больше, чем 144 ячейки смысла нет.
13.03.2016, 22:49
DeimoS

Код из шапки не скомпилируется, если подключён foreach :3
13.03.2016, 22:52
Иван Бубнов

Цитата:

Сообщение от DeimoS

Код из шапки не скомпилируется, если подключён foreach :3

Теперь скомпилируется
13.03.2016, 22:55
Desulaid

Во, а что если так?

PHP код:

CMD:do(playerid, params[]) { if (isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, world, interior, name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, name, sizeof(name)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); strcat(string, params); strcat(string, " ("); strcat(string, name); strcat(string, ")"); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world) || (GetPlayerInterior(i) != interior)) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }

Не компилировал, результат не знаю :Р
13.03.2016, 22:58
Иван Бубнов

Цитата:

Сообщение от Anton Styazhkin

Во, а что если так?

PHP код:

CMD:do(playerid, params[]) { #if defined sscanf if (sscanf(params, "s[124]", params)) #else if (isnull(params)) #endif { return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); } new Float:PosX, Float:PosY, Float:PosZ, world, interior, name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, name, sizeof(name)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); strcat(string, params); strcat(string, " ("); strcat(string, name); strcat(string, ")"); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world ) || (GetPlayerInterior(i) != interior) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }

Не компилировал, результат не знаю :Р

Зачем sscanf?
13.03.2016, 22:58
$continue$

Открыть/закрыть

if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue;
if((GetPlayerVirtualWorld(i) != world )
|| (GetPlayerInterior(i) != interior)) continue;

Цитата:

Сообщение от Anton Styazhkin

Во, а что если так?
13.03.2016, 23:00
Desulaid

Цитата:

Сообщение от Иван Бубнов

Зачем sscanf?

хз, обновил пост
13.03.2016, 23:07
Иван Бубнов

Цитата:

Сообщение от Anton Styazhkin

хз, обновил пост

Тогда будет правильнее так, наверное

PHP код:

CMD:do(playerid, params[]) { #if defined sscanf if (sscanf(params, "s[124]", params)) #else if (isnull(params)) #endif { return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); } new Float:PosX, Float:PosY, Float:PosZ, world, interior, name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, name, sizeof(name)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); #if defined sscanf strcat(string, params[0]); #else strcat(string, params); #endif strcat(string, " ("); strcat(string, name); strcat(string, ")"); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world ) || (GetPlayerInterior(i) != interior)) continue; SendClientMessage(i, 0xE600FFFF, string); } #if defined sscanf SetPlayerChatBubble(playerid, params[0], 0xE600FFFF, 30.0, 10000); #else SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); #endif return 1; }
13.03.2016, 23:10
Desulaid

Цитата:

Сообщение от Иван Бубнов

Тогда будет правильнее так, наверное

PHP код:

CMD:do(playerid, params[]) { #if defined sscanf if (sscanf(params, "s[124]", params)) #else if (isnull(params)) #endif { return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); } new Float:PosX, Float:PosY, Float:PosZ, world, interior, name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, name, sizeof(name)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); #if defined sscanf strcat(string, params[0]); #else strcat(string, params); #endif strcat(string, " ("); strcat(string, name); strcat(string, ")"); #if defined foreach foreach(new i : Player) { #else for(new i = GetPlayerPoolSize(); i != -1; i--) { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world ) || (GetPlayerInterior(i) != interior)) continue; SendClientMessage(i, 0xE600FFFF, string); } #if defined sscanf SetPlayerChatBubble(playerid, params[0], 0xE600FFFF, 30.0, 10000); #else SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); #endif return 1; }

Не, только хуже и ты так вроде только первый символ строки покажешь
13.03.2016, 23:11
Иван Бубнов

Цитата:

Сообщение от Anton Styazhkin

Не, только хуже и ты так вроде только первый символ строки покажешь

ай лол, тогда sscanf там не уместен.
14.03.2016, 19:52
Иван Бубнов

Обновил
19.03.2016, 16:00
Иван Бубнов

Обновил
19.03.2016, 17:40
L0ndl3m

Можете реализовать команду без использования функции strcat?
19.03.2016, 17:47
Иван Бубнов

Цитата:

Сообщение от Londlem

Можете реализовать команду без использования функции strcat?

PHP код:

CMD:do(playerid, params[]) { if (isnull(params)) return SendClientMessage(playerid, -1, !"Введите: /do <текст>"); new Float:PosX, Float:PosY, Float:PosZ, world, interior, string[MAX_CHATBUBBLE_LENGTH+1]; GetPlayerPos(playerid, PosX, PosY, PosZ); GetPlayerName(playerid, string, sizeof(string)); world = GetPlayerVirtualWorld(playerid); interior = GetPlayerInterior(playerid); format(string, sizeof(string), "%s (%s)", params, string); #if defined foreach foreach(new i : Player) { #else for(new i = -1; ++i != GetPlayerPoolSize();) { if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue; #endif if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue; if((GetPlayerVirtualWorld(i) != world ) || (GetPlayerInterior(i) != interior)) continue; SendClientMessage(i, 0xE600FFFF, string); } SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000); return 1; }
19.03.2016, 18:00
VVWVV

1. Используй упакованные строки, ибо strcat позволяет это сделать.

PHP код:

string[MAX_CHATBUBBLE_LENGTH+1 char];

PHP код:

strcat(string, params);     strcat(string, !" (");     strcat(string, name);     strcat(string, !")");

2. Использовать функцию GetPlayerPoolSize при каждой итерации - плохо.

PHP код:

for(new i = -1; ++i != GetPlayerPoolSize();)

3. Не стоит забывать о тех, кто ещё сидит на версии 0.3e.

PHP код:

#if defined foreach     foreach (new i: Player) #else     #if defined GetPlayerPoolSize         for (new i = GetPlayerPoolSize(); --i != -1;)     #else         for (new i = -1; ++i != MAX_PLAYERS;)     #endif         if (IsPlayerConnected(i) == 0 || IsPlayerNPC(i) != 0) continue;         else #endif     {}

UPD:

PHP код:

CMD:do(playerid, params[]) {     if (isnull(params))         return SendClientMessage(playerid, -1, !"Введите: /do <текст>");     new         Float:PosX, Float:PosY, Float:PosZ, world, interior,         name[MAX_PLAYER_NAME], string[MAX_CHATBUBBLE_LENGTH+1 char];     GetPlayerPos(playerid, PosX, PosY, PosZ);     GetPlayerName(playerid, name, sizeof(name));     world = GetPlayerVirtualWorld(playerid);     interior = GetPlayerInterior(playerid);     strcat(string, params);     strcat(string, !" (");     strcat(string, name);     strcat(string, !")"); #if defined foreach     foreach(new i: Player) #else     #if defined GetPlayerPoolSize     for(new i = GetPlayerPoolSize() + 1; --i != -1;)     #else     for(new i = -1; ++i != MAX_PLAYERS;)     #endif         if ((0 == IsPlayerConnected(i)) || (0 != IsPlayerNPC(i))) continue;     else #endif     {         if(0 == IsPlayerInRangeOfPoint(i, 15.0, PosX, PosY, PosZ)) continue;         if((GetPlayerVirtualWorld(i) != world )         || (GetPlayerInterior(i) != interior)) continue;         SendClientMessage(i, 0xE600FFFF, string);     }     SetPlayerChatBubble(playerid, params, 0xE600FFFF, 30.0, 10000);     return 1; }
19.03.2016, 18:49
NewGreen

Цитата:

Сообщение от VVWVV

3. Не стоит забывать о тех, кто ещё сидит на версии 0.3e.

PHP код:

#if defined foreach foreach (new i: Player) #else #if defined GetPlayerPoolSize for (new i = GetPlayerPoolSize(); --i != -1;) #else for (new i = -1; ++i != MAX_PLAYERS;) #endif if (IsPlayerConnected(i) == 0 || IsPlayerNPC(i) != 0) continue; else #endif {}

Использовать for в такой извращенной форме плохо, когда есть уже готовые решения:

PHP код:

for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)

а сейчас что мы имеем:

GetPlayerPoolSize() - согласно wiki -Gets the highest playerid currently in use on the server. - т.е. получает самый старший playerid который используется в настоящее время на сервере.

Допустим старший ID у нас 10:

Запускаем код:

PHP код:

for (new i = 10/*GetPlayerPoolSize()*/; --i != -1;) printf("i = %d", i);

Получаем:

Открыть/закрыть

Код:

i = 9 i = 8 i = 7 i = 6 i = 5 i = 4 i = 3 i = 2 i = 1 i = 0

т.е. ID номер 10 исключается из цикла.
19.03.2016, 18:51
Иван Бубнов

Обновлено

- - - Добавлено - - -

Цитата:

Сообщение от NewGreen

Использовать for в такой извращенной форме плохо, когда есть уже готовые решения:

PHP код:

for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)

Обоснуй
19.03.2016, 18:53
NewGreen

Цитата:

Сообщение от Иван Бубнов

Обновлено

- - - Добавлено - - -

Обоснуй

см. спойлер в моем сообщении выше
19.03.2016, 18:56
Иван Бубнов

Стыд и позор. /dell
19.03.2016, 18:56
VVWVV

Цитата:

Сообщение от NewGreen

см. спойлер в моем сообщении выше

Сам заметил, при тестировании. Спасибо. Код обновил (добавил свою версию).

PHP код:

for(new i = GetPlayerPoolSize() + 1; --i != -1;)

Цитата:

Сообщение от NewGreen

for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)

Зачем ещё одна переменная?

Цитата:

Сообщение от Иван Бубнов

Стыд и позор. /dell

Извиняюсь, но я был не прав.
19.03.2016, 19:02
Иван Бубнов

Цитата:

Сообщение от VVWVV

Сам заметил, при тестировании. Спасибо. Код обновил (добавил свою версию).

Извиняюсь, но я был не прав.

Не тебе это) Я просто кое что лишнее написал грину :hi:
19.03.2016, 19:25
NewGreen

Цитата:

Сообщение от Иван Бубнов

Не тебе это) Я просто кое что лишнее написал грину :hi:

Ну раз это было мне, то обоснуйте, как вы выразились ранее

- - - Добавлено - - -

Цитата:

Сообщение от VVWVV

Зачем ещё одна переменная?

Так отсчет ID начинается с нуля, хоть это может быть и не столь важно
19.03.2016, 19:37
VVWVV

Цитата:

Сообщение от NewGreen

Ну раз это было мне, то обоснуйте, как вы выразились ранее

- - - Добавлено - - -

Так отсчет ID начинается с нуля, хоть это может быть и не столь важно

Зачем нам тратить память под ненужную переменную?
19.03.2016, 20:13
NewGreen

Цитата:

Сообщение от VVWVV

Зачем нам тратить память под ненужную переменную?

Для вас принципиально важно ?, я написал это, так как, это одна классических конструкций цикла for, в том же wiki она используется.
19.03.2016, 20:21
VVWVV

Цитата:

Сообщение от NewGreen

Для вас принципиально важно ?, я написал это, так как, это одна классических конструкций цикла for, в том же wiki она используется.

Для вас, может быть, нет, а для меня - да.
Впрочем, это ещё и оптимизация скрипта.
19.03.2016, 20:28
Иван Бубнов

Закрыто.

Текущее время: 18:50. Часовой пояс GMT +4.