Исправляем краш при попадании спецификатора "%s" в SendClientMessage(ToAll)

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

29.02.2016, 00:22
DeimoS

Исправляем краш при попадании спецификатора "%s" в SendClientMessage(ToAll)
И вновь я Вас категорически приветствую! :pleasantry:
_{(что-то нечасто это бывает...)}

Ни для кого не секрет (а может для кого-то и секрет), что в SA-MP есть баг, при котором сервер крашило если в функции SendClientMessage и SendClientMessageToAll попадает спецификатор, который не подвергся форматированию.

Когда-то с помощью этого бага появилась возможность класть сервера просто вводя в чат сообщение, в котором содержится любой спецификатор. Куй, естественно, реагировал на это со скоростью черепахи и смекалистые умельцы сделали защиту со стороны сервера путём замены всех "%" на "#".

Когда Куй наконец решил обратить на нас, обычных смертных, внимание, он добавил "защиту" со стороны клиента, просто вставив идентичный код в обработчик текста, введённого игроком в чат, а не сделав защиту напрямую в SendClientMessage, поэтому и по сей день в SA-MP можно положить любой сервер, где данные, введённые Вами в диалоговое окно, выводятся в чат через SendClientMessage _{(сообщения в личку, например, или же на RP серверах сообщения на радио, если они реализованы через диалог)} и не стоит подобной защиты.
Так же, если Вы сами в SendClientMessage укажете спецификатор "%s", сервер неминуемо крашнет :с

В интернете гуляют реализации, но во всех, попавшихся мне, защита реализована идентично той, что стоит в чате _{(то бишь, знаки процента заменяются на знак решётки)}, что лично мне не совсем нравится. Поэтому я решил написать свою функцию, которая, по возможности, сможет оставить все знаки процента в сообщении игрока, при этом не доставляя никаких проблем серверу.
И написал :spiteful:

Защита реализована, как я уже сказал выше, в виде функции.

Код функции без описания

PHP код:

stock FindSpecifiersInString(message[], array_size = sizeof(message)) { new message_length = strlen(message); for(new i; i < message_length; i++) { if(message[i] == '%') { if(message_length < array_size) strins(message, "%", i++, message_length++); else message[i] = '#'; } } }

Функция имеет один обязательный параметр и один совсем "ненужный" параметр.
1. message[]
  
  - собственно, массив(!) с текстом, в котором мы будем обрабатывать символы
  -[/INDENT][/INDENT]
2. array_size
  
  Данный параметр нужен для определения размера массива, который Вы передаёте в функцию в качестве параметра. Устанавливать его не нужно, ибо сервер всё сделает сам за Вас (при неверной установке значения в данном массиве функция будет работать неверно!)
  
  Открыть/закрыть
  Если текст, передающийся в функцию, находится в переменной с неизвестным числом ячеек (как в случае с диалогами и параметром inputtext), функция в любом случае будет заменять все "%" на "%".
  
  Дабы такого не случилось, перед вызовом функции, запишите inputtext в любой массив. Например:
  
  PHP код:
  
  if(dialogid == 1111) { new inputtext_buff[145];//Объявляем массив // 144+1 - именно столько (144) функция SendClientMessage(ToAll) сможет отправить символов за раз (1 - нуль-символ) strcat(inputtext_buff, inputtext, 129);// Записываем содержимое inputtext в массив //128+1 - именно столько (128) символов игрок может ввести в диалоговое окно (1 - нуль-символ) FindSpecifiersInString(inputtext_buff);//Вызываем функцию format(inputtext_buff, sizeof(inputtext_buff), "Ты написал: %s.", inputtext_buff);//Отображаем и, для экономии памяти, записываем текст в тот же массив (в данном случае такое приемлемо) SendClientMessage(playerid, -1, inputtext_buff);// Отображаем //В случае необходимости операцию можно повторять до бесконечности, а-ля /*Хотя в данном случае лучше в format указать другой массив и тут уже использовать ранее записанные данные в inputtext_buff но это ведь просто пример того, что функцию можно использовать несколько раз подряд для разного текста :) strcat(inputtext_buff, inputtext, 129); FindSpecifiersInString(inputtext_buff); format(inputtext_buff, sizeof(inputtext_buff), "Он написал: %s.", inputtext_buff); SendClientMessage(playerid, -1, inputtext_buff);*/ return 1; }
  
  И теперь функция будет обрабатывать все "%" как и положено
  
  * Если указать "0", функция не будет пытаться сохранить "%" в первозданном виде, а сразу начнёт заменять все "%" на "#"
В ней учтено, что, в случае, если в SendClientMessage попадёт, например, "% s", или "% 000 s", или "%02d" - функция распознает данный текст как идентификатор и в SendClientMessage(ToAll) попадёт уже исправленная версия.

Пример использования:

PHP код:

main() { new string[13] = "% %% %s %%%"; FindSpecifiersInString(string, 0);//Все "%" заменит на "#" printf("%s", string); string = "% %% %s %%%";//Так как размер массива всего 13, а символов в строке 11, функция сможет сохранить только 2 первых символа, а всё остальное заменит на # FindSpecifiersInString(string);//Продублирует все "%". В случае нехватки места начнёт заменять их на "#" printf("%s", string); }

Код функции с описанием

PHP код:

stock FindSpecifiersInString(message[], array_size = sizeof(message))//Во втором параметре узнаём размер массива, переданный в функцию { new message_length = strlen(message);//Узнаём размер строки, переданной в функцию for(new i; i < message_length; i++)//Запускаем цикл на кол-во итераций, равное числу символов в строке { if(message[i] == '%')//Ищем символ "%" { if(message_length < array_size) strins(message, "%", i++, message_length++); //Если нашли, проверяем наличие места в массиве и если свободные ячейки есть - добавляем новый символ, делая соответствующие //Изменения в переменных else message[i] = '#';//Если места нет, начинаем заменять "%" на "#" } } }

P.S. Если попытаться в функции указать массив с неизвестным кол-вом ячеек (как в случае с параметром "inputtext" в OnDialogResponse), компилятор выплюнет предупреждение:

PHP код:

warning 224: indeterminate array size in "sizeof" expression (symbol "")

Я пока не придумал как это решить, но на работоспособность сей warning влиять не будет.
Если Вы знаете как обойти данную проблему, прошу отписаться по этому поводу ниже :)

Автор темы - DeimoS
Автор кода - DeimoS

Отдельная благодарность Daniel_Cortez
за ответы на идиотские вопросы с моей стороны :3

Специально для Pro-Pawn.ru
Копирование материала без указания авторства запрещено.

Ставь лайк, подписывайся на канал.
Ссылка на вебмани, а так же список всей музыки, использованной в видео, в описании.
Новое видео выложу после 1000 лайков :3
29.02.2016, 00:46
$continue$

"Я нуб в павно - но и ты не компилятор (с) G-I"
Что, за этакий стиль, переменную для счетчика цикла объявлять в одном for а использовать в другом for?
В чем профит?

Хотел бы я посмотреть как тебе отстреливают ногу за goto - СИшники.

А, да, не охото вчитываться в код, с sizeof проблема в функции?
Если да, то:

PHP код:

stock FindSpecifiersInString(message[], bool:mode = false, array_size = sizeof(message)) { new message_length = strlen(message); for(new i, j; i < message_length; i++) { FSIS_For_Start: if(message[i] == '%') { for(j = i+1; j <= message_length; j++) { switch(message[j]) { case '0'..'9', ' ', '.', '#': continue; default: { if(mode == true && array_size != 0 && message_length < array_size) { strins(message, "%", i, message_length); message_length++; i = j; } else message[i] -= 0x2; i++; goto FSIS_For_Start; } } } } } }
29.02.2016, 00:53
DeimoS

Цитата:

Сообщение от $continue$

Что, за этакий стиль, переменную для счетчика цикла объявлять в одном for а использовать в другом for?
В чем профит?

Может для того, чтоб при каждой новой итерации в первом цикле не нужно было её инициализировать при старте второго, не? Хотя не удивлюсь, если ты прописываешь инициализацию переменных внутри циклов и считаешь это нормой.

Цитата:

Сообщение от $continue$

Хотел бы я посмотреть как тебе отстреливают ногу за goto - СИшники.

В данном случае без него я не нашёл способа реализовать всё правильно. Да и тут goto не переносит тебя на 1000 строк в совершенно другой участок мода, так что это не так смертельно.

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

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

Цитата:

Сообщение от $continue$

"Я нуб в павно - но и ты не компилятор (с) G-I"
Что, за этакий стиль, переменную для счетчика цикла объявлять в одном for а использовать в другом for?
В чем профит?

Хотел бы я посмотреть как тебе отстреливают ногу за goto - СИшники.

А, да, не охото вчитываться в код, с sizeof проблема в функции?
Если да, то:

PHP код:

stock FindSpecifiersInString(message[], bool:mode = false, array_size = sizeof(message), message_length = strlen(message)) { for(new i, j; i < message_length; i++) { FSIS_For_Start: if(message[i] == '%') { for(j = i+1; j <= message_length; j++) { switch(message[j]) { case '0'..'9', ' ', '.', '#': continue; default: { if(mode == true && array_size != 0 && message_length < array_size) { strins(message, "%", i, message_length); message_length++; i = j; } else message[i] -= 0x2; i++; goto FSIS_For_Start; } } } } } }

strlen прямо в параметрах функции? Окей, странный мальчик с третьей рукой из спины. Посмеёмся вместе когда компилятор крашнет :3

И да, гений программирования, куда бы ты sizeof не запихал, компилятор выругается в любом случае, ибо проблема именно в том, что в функцию передаётся динамический массив (как в случае с inputtext). Возможно есть какая-то pragma или что-то ещё, но мне пока лень копать мат.часть и выискивать решение.
Сап
29.02.2016, 01:15
$continue$

1) Вынести за пределы for переменную, не?
2) Не заметил, что используется strlen - отредактировал код, раньше чем ты ответил :3
На счёт размера - действительно, может можно как то с помощью emit?
Убогость - Pawn, ничего не поделать.
29.02.2016, 01:32
DeimoS

Цитата:

Сообщение от $continue$

1) Вынести за пределы for переменную, не?
2) Не заметил, что используется strlen - отредактировал код, раньше чем ты ответил :3
На счёт размера - действительно, может можно как то с помощью emit?
Убогость - Pawn, ничего не поделать.

1) И что изменится? Переменные относятся конкретно к циклам и логично искать инициализацию именно в циклах, не?
2) Не поверишь, но именно так выглядела функция изначально. Но решил перенести параметр array_size в переменную, дабы молодые умы не решили, что его тоже нужно указывать при вызове функции и не получилась из-за этого куча багов :3
29.02.2016, 02:10
vovandolg

Так а в чём соль, знак будет показываться и без краша или это правильная рабочая заменка?
А то как нубу не очень дошло что да зачем)
29.02.2016, 05:57
SliM

почему нельзя сделать так?

PHP код:

new message_length, array_size = strlen(message); message_length = strlen(message);

____________________________UPD
Сам догнал почему:)
____________________________

Цитата:

Поэтому я решил написать свою функцию, которая, по возможности, сможет оставить все знаки вопроса в сообщении игрока

Это опечатка? или я через строку читал, что за знаки вопроса?

Цитата:

(как в случае с параметром "inputtext" в OnRequestClass)

все таки в OnRequestClass не в OnDialogResponse?
29.02.2016, 06:22
DeimoS

Цитата:

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

Это опечатка? или я через строку читал, что за знаки вопроса?

все таки в OnRequestClass не в OnDialogResponse?

Спасибо, исправил. 3 часа ночи было и немного начал "плыть" ;)

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

Цитата:

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

Так а в чём соль, знак будет показываться и без краша или это правильная рабочая заменка?
А то как нубу не очень дошло что да зачем)

Есть функция SendClientMessage в которой ещё с мезозойских времён имеется вот такой баг:

http://i.imgur.com/3EMEck8.png

То бишь, если в функцию передать спецификатор (спецификатор - это "%s", "%d" и т.п.), который не был форматирован до этого в том же format, то сервер из-за этого упадёт (точнее, падает он, вроде, только от спецификатора "%s". Но про это я в теме говорил).
Куй исправил тему с OnPlayerText и OnPlayerCommandText, обрабатывая введённый игроком текст и заменяя все "%" на "#", но с диалогами он этого не сделал (а это самый распространённый пример. Авось ещё где-то можно подобный баг провернуть).

Эта функция является более функциональным аналогом тех вариаций, что гуляют в интернете (опять же, я об этом писал в теме: про способы, которые заменяют все "%" на "#"). Хотя что те варианты, что мой - все рабочие, хотя логичнее использовать мой, ибо тогда обрабатываться будут не все диалоги, а только нужные.

P.S. Если я неправильно понял вопроса и ты спрашивал о том, как работает сей баг, который исправляет функция, вставь этот код:

PHP код:

public OnPlayerSpawn(playerid) { ShowPlayerDialog(playerid, 1112, DIALOG_STYLE_INPUT, "Crash", "Crash", "Да", "Нет (Да)"); return 1; } public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[]) { if(dialogid == 1112) { new inputtext_buff[145]; strcat(inputtext_buff, inputtext, 128); //FindSpecifiersInString(inputtext_buff, true); format(inputtext_buff, sizeof(inputtext_buff), "Написал: %s.", inputtext_buff); return SendClientMessage(playerid, -1, inputtext_buff); } return 1; }

А после попробуй ввести в это диалоговое окно что-то подобное

PHP код:

%s % s %d %b

И смотри что будет :)
А потом убери комментирование строки с вызовом функции + вставь код самой функции из урока и посмотри что выйдет после

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

Да или просто в OnPlayerConnect добавь

PHP код:

SendClientMessage(playerid, -1, "%s");

И при первом вошедшем игроке на сервер, его просто крашнет
29.02.2016, 08:33
Daniel_Cortez

1. sizeof нельзя использовать на массивах-параметрах функции.
2. При вставке ещё одного символа "%" теряется символ в конце строки. Этого побочного эффекта в описании не указано.
29.02.2016, 08:45
DeimoS

Цитата:

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

1. sizeof нельзя использовать на массивах-параметрах функции.
2. При вставке ещё одного символа "%" теряется символ в конце строки. Этого побочного эффекта в описании не указано.

1) При этом всё работает :pardon: Правда, как уже писал в теме, компилятор выплёвывает warning в случае, если попытаться параметром передать массив с неизвестным числом ячеек, но тогда sizeof просто ровна нулю, на чём и построена проверка
2) Так вставка происходит только если в массиве, переданном в функцию, имеется место, не? Именно для этого я и спрашивал у тебя насчёт sizeof и именно об этой проверке я написал в предыдущем пункте. Если длина текущей строки+1 новый символ не равно или больше размеру массива, полученного через sizeof - добавляем новый символ. Иначе заменяем "%" на "#" :download:
29.02.2016, 09:43
Daniel_Cortez

Цитата:

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

1) При этом всё работает :pardon: Правда, как уже писал в теме, компилятор выплёвывает warning в случае, если попытаться параметром передать массив с неизвестным числом ячеек, но тогда sizeof просто ровна нулю, на чём и построена проверка

И этого легко избежать, я даже вчера показывал в XMPP, как именно.

Цитата:

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

2) Так вставка происходит только если в массиве, переданном в функцию, имеется место, не?

Цитата:

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

Именно для этого я и спрашивал у тебя насчёт sizeof и именно об этой проверке я написал в предыдущем пункте. Если длина текущей строки+1 новый символ не равно или больше размеру массива, полученного через sizeof - добавляем новый символ.

sizeof возвращает 0, когда не может определить размер массива, поэтому в array_size всегда будет 0.
Как результат, вставка никогда не будет производиться. Вместо этого символ "%" всегда будет заменяться на "#".

Цитата:

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

Иначе заменяем "%" на "#" :download:

ИМХО, _только_ такой вариант был бы куда проще и без нежелательных эффектов.

Цитата:

PHP код:

message[i] -= 0x2;

Попахивает понтокодом.
29.02.2016, 10:26
DeimoS

Цитата:

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

И этого легко избежать, я даже вчера показывал в XMPP, как именно.

Хмм, что-то не могу найти :с

Цитата:

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

sizeof возвращает 0, когда не может определить размер массива, поэтому в array_size всегда будет 0.
Как результат, вставка никогда не будет производиться. Вместо этого символ "%" всегда будет заменяться на "#".

Хмм, до меня, кажется, дошло о чём ты говоришь. Проверил (чего почему-то не сделал вчера) и действительно, если вызывать sizeof прямо внутри функции, а не в параметрах, sizeof отказывается работать :с Вернул код к изначальному состоянию.
Но, как и писал ранее, сей манёвр всё равно не избавляет от предупреждения со стороны компилятора при попытке засунуть в функцию inputtext

Цитата:

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

ИМХО, _только_ такой вариант был бы куда проще и без нежелательных эффектов.

Ну я в теме уже говорил для чего и почему я решил написать эту функцию :)

Цитата:

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

Попахивает понтокодом.

Этот кусок кода я просо скопировал с одной темы из интернета, где как раз и идёт замещение всех символов на "#"
29.02.2016, 12:42
vovandolg

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

Цитата:

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

Есть функция SendClientMessage в которой ещё с мезозойских времён имеется вот такой баг:

http://i.imgur.com/3EMEck8.png

То бишь, если в функцию передать спецификатор (спецификатор - это "%s", "%d" и т.п.), который не был форматирован до этого в том же format, то сервер из-за этого упадёт (точнее, падает он, вроде, только от спецификатора "%s". Но про это я в теме говорил).
Куй исправил тему с OnPlayerText и OnPlayerCommandText, обрабатывая введённый игроком текст и заменяя все "%" на "#", но с диалогами он этого не сделал (а это самый распространённый пример. Авось ещё где-то можно подобный баг провернуть).

Эта функция является более функциональным аналогом тех вариаций, что гуляют в интернете (опять же, я об этом писал в теме: про способы, которые заменяют все "%" на "#"). Хотя что те варианты, что мой - все рабочие, хотя логичнее использовать мой, ибо тогда обрабатываться будут не все диалоги, а только нужные.

P.S. Если я неправильно понял вопроса и ты спрашивал о том, как работает сей баг, который исправляет функция, вставь этот код:

PHP код:

public OnPlayerSpawn(playerid) { ShowPlayerDialog(playerid, 1112, DIALOG_STYLE_INPUT, "Crash", "Crash", "Да", "Нет (Да)"); return 1; } public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[]) { if(dialogid == 1112) { new inputtext_buff[145]; strcat(inputtext_buff, inputtext, 128); //FindSpecifiersInString(inputtext_buff, true); format(inputtext_buff, sizeof(inputtext_buff), "Написал: %s.", inputtext_buff); return SendClientMessage(playerid, -1, inputtext_buff); } return 1; }

А после попробуй ввести в это диалоговое окно что-то подобное

PHP код:

%s % s %d %b

И смотри что будет :)
А потом убери комментирование строки с вызовом функции + вставь код самой функции из урока и посмотри что выйдет после

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

Да или просто в OnPlayerConnect добавь

PHP код:

SendClientMessage(playerid, -1, "%s");

И при первом вошедшем игроке на сервер, его просто крашнет

Так есть же коротко строчные коды которые предотвращают это... я там в рлснике видел примеры с while
Или то дичь полная что в рлс?
29.02.2016, 13:32
DeimoS

Цитата:

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

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

Так есть же коротко строчные коды которые предотвращают это... я там в рлснике видел примеры с while
Или то дичь полная что в рлс?

Неужели я так плохо всё объяснил в шапке темы?
Все подобные фиксы 100% заменяют все "%" на "#", без возможности оставить их в виде "%".
Большинство из этих фиксов так же совершают ненужные действия тем, что обрабатывают все без разбору диалоги, которые были показаны игроку: будь то действительно диалог с inputtext, который нужно обработать, или же какой-нибудь DIALOG_STYLE_MSGBOX (ибо его содержимое будет передано как раз в inputtext) с текстов в 2000 символов - в обоих случаях этот цикл прогонит содержимое сквозь себя, что во втором случае вовсе не нужно (выход - либо перехватывать ShowPlayerDialog, записывать тип показанного игроку диалога и сверять его, либо прямо по ID ориентироваться, либо использовать мою функцию, расставив форматирование строки только в тех диалогах, где это нужно)

Я же хотел сделать функцию, которая:
Во-первых, позволит контролировать то, какие диалоги нужно исправлять, а какие не нужно (+ к оптимизации)
Во-вторых, позволит сохранить сообщение в первозданном виде, если мне это требуется.
Вот и всё
29.02.2016, 13:49
Daniel_Cortez

Цитата:

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

Хмм, что-то не могу найти :с

Код HTML:

(23:45:16) DeimoS: И тогда strlen не подойдёт (23:45:19) Daniel_Cortez: или сделать в функции опциональный параметр size (23:45:41) Daniel_Cortez: SomeFunction(string[], size = sizeof(string)) (23:46:11) Daniel_Cortez: куй этого явно не знал, когда делал функции типа GetPlayerName (23:46:37) Daniel_Cortez: иначе мог бы сэкономить кучу времени и нервов скриптерам

Цитата:

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

Хмм, до меня, кажется, дошло о чём ты говоришь. Проверил (чего почему-то не сделал вчера) и действительно, если вызывать sizeof прямо внутри функции, а не в параметрах, sizeof отказывается работать :с Вернул код к изначальному состоянию.
Но, как и писал ранее, сей манёвр всё равно не избавляет от предупреждения со стороны компилятора при попытке засунуть в функцию inputtext

Он и не должен. Размер передаваемой строки можно узнать только через отдельный параметр size, который желательно сделать опциональным.

Цитата:

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

Ну я в теме уже говорил для чего и почему я решил написать эту функцию :)

В таком случае кроме плюсов реализации напиши и про потерю последних символов в строке. Ты ж скриптер, а не маркетолог.

Цитата:

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

Этот кусок кода я просо скопировал с одной темы из интернета, где как раз и идёт замещение всех символов на "#"

Дежавю какое-то...

Цитата:

Код не мой, исправлять не буду!
29.02.2016, 16:53
DeimoS

Цитата:

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

Код HTML:

(23:45:16) DeimoS: И тогда strlen не подойдёт (23:45:19) Daniel_Cortez: или сделать в функции опциональный параметр size (23:45:41) Daniel_Cortez: SomeFunction(string[], size = sizeof(string)) (23:46:11) Daniel_Cortez: куй этого явно не знал, когда делал функции типа GetPlayerName (23:46:37) Daniel_Cortez: иначе мог бы сэкономить кучу времени и нервов скриптерам

Ааа, так ты об этом. Я-то думал, что ты о предупреждении, которое выдаёт компилятор

Цитата:

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

Он и не должен. Размер передаваемой строки можно узнать только через отдельный параметр size, который желательно сделать опциональным.

Таки уже

Цитата:

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

Дежавю какое-то...

:D
Вот новая, полностью переработанная версия. Что-то я вчера намудрил

PHP код:

stock FindSpecifiersInString(message[], array_size = sizeof(message)) { new message_length = strlen(message); for(new i; i < message_length; i++) { if(message[i] == '%') { if(message_length < array_size) strins(message, "%", i++, message_length++); else message[i] = '#'; } } }

Тщательно её не тестировал, поэтому пока добавлять в шапку не буду. Чуть позже вечером или завтра займусь тестами, если никто раньше этого не ткнёт меня носом в мой кривой код :3
29.02.2016, 18:19
seriu

Цитата:

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

Попахивает понтокодом.

Понтокодом попахивает тут http://pro-pawn.ru/showthread.php?79..._stringhash%29
а тут всего лишь символ 0x25 (%) заменяется на 0x23(#)
29.02.2016, 18:49
Daniel_Cortez

Цитата:

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

тут всего лишь символ 0x25 (%) заменяется на 0x23(#)

... в то время, как вместо понтов типа 0x25 или 0x23 можно написать '%' или '#'. Или вы тоже вместо 0 всегда пишете 0x0?
И кстати, сможете сказать, какой символ скрывается под кодом 0xBA? Чур в таблицу ASCII-символов не подглядывать.
Кроме того, можно было обойтись присваиванием значения '#' вместо вычитания. Что по-вашему сложнее: взять значение из ячейки, совершить над ним операцию и записать в ячейку или просто записать в ячейку уже известное нам значение (константу)? (Правильный ответ очевиден: просто записать - ОСТОРОЖНО, СПОЙЛЕРЫ.)

Цитата:

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

Понтокодом попахивает тут http://pro-pawn.ru/showthread.php?79..._stringhash%29

Там чёрным по белому написано, что y_stringhash и таблица используются для достижения производительности и перекрытия потенциальной возможности перегрузить сервер. Потому смысл в этом коде есть, хоть его и трудно уловить для новичков.
Считаете иначе - докажите, что это понтокод, если не хотите быть пустословом.
29.02.2016, 19:04
DeimoS

И пока тут назревает маленькая война, я всё же обновил тему, указав в ней финальный (я надеюсь) вариант сей функции. Из функции убран бесполезный аргумент + убрана попытка выискивать конкретные спецификаторы, ибо SendClientMessage всё равно не дружит с "%", если его указать в виде самостоятельного знака
01.03.2016, 15:22
seriu

Цитата:

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

в то время, как вместо понтов типа 0x25 или 0x23 можно написать '%' или '#'. Или вы тоже вместо 0 всегда пишете 0x0?
И кстати, сможете сказать, какой символ скрывается под кодом 0xBA? Чур в таблицу ASCII-символов не подглядывать.
Кроме того, можно было обойтись присваиванием значения '#' вместо вычитания. Что по-вашему сложнее: взять значение из ячейки, совершить над ним операцию и записать в ячейку или просто записать в ячейку уже известное нам значение (константу)? (Правильный ответ очевиден: просто записать - ОСТОРОЖНО, СПОЙЛЕРЫ.)

В чем проблема? не вычитай заменяй сразу на 0x23
При чем тут символ 0x0 ?
0xBA - в душе не чаю, что это за символ, мне это даже не интересно.

Цитата:

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

Там чёрным по белому написано, что y_stringhash и таблица используются для достижения производительности и перекрытия потенциальной возможности перегрузить сервер. Потому смысл в этом коде есть, хоть его и трудно уловить для новичков.
Считаете иначе - докажите, что это понтокод, если не хотите быть пустословом.

Можно было просто сделать /veh id color1 color2
Или у тебя все команды такие и ты во всех командах перебираешь кучу строк что бы проверить на совпадение имен?
по мойму гораздо проще к этому имени приравнять ид и по ид сравнивать 1 запрос.
01.03.2016, 16:23
$continue$

При том, что 0x0 = 0. В чем профит писать так в Pawn?
Не более чем выпендрежный код.

Цитата:

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

При чем тут символ 0x0 ?
01.03.2016, 18:16
DeimoS

А в чём профит комментировать сообщения вот так?

Цитата:

Сообщение от $continue$

При том, что 0x0 = 0. В чем профит писать так в Pawn?
Не более чем выпендрежный код.
02.03.2016, 11:39
Daniel_Cortez

Цитата:

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

В чем проблема? не вычитай заменяй сразу на 0x23

*facepalm*

Цитата:

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

0xBA - в душе не чаю, что это за символ, мне это даже не интересно.

Т.е. 0xBA вы не знаете, а всякие 0x23 и 0x25 вы заучили наизусть и для вас использовать их - норма?
И всё же, в чём профит юзать 0x23, если можно использовать то же самое значение в понятном виде '#' и не запоминать код символа?

Цитата:

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

Можно было просто сделать /veh id color1 color2

Можно было, но фишка моей реализации, в отличие от многих других, в том, что она может создавать транспорт по его названию, причём куда более эффективным образом, чем просто перебором всех названий.

Цитата:

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

Или у тебя все команды такие и ты во всех командах перебираешь кучу строк что бы проверить на совпадение имен?

Если присмотреться, там нет никакого перебора строк.

Цитата:

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

по мойму гораздо проще к этому имени приравнять ид и по ид сравнивать 1 запрос.

Не знаю, какие запросы вы туда хотите впихнуть, в той реализации и без того эффективный алгоритм поиска.