GameChat Filter [Удалено]

[Grey83]

Почистил и оптимизировал немного код
Должно корректно перехватывать сообщения на старом движке
Заодно перевёл инклюд на новый синтаксис (ну и добавил функцию инвертирования регистра :cool:)

P.S. Да, мне снова было нечем заняться и страдал фигнёй переписывая AnyChatFilter v1.5.0.1 (кстати, нумерация такого вида обычно только в версиях SM, в плагинах чаще вида x.x или x.x.x)
--- Добавлено позже ---
Если кому интересно, то вот плагин, на котором я тестировал конвертацию регистра:

 

[Kruzya]

@Grey83, нумерация из четырёх цифр более удобная, если для себя обозначить, что означает какая цифра.

 

[Grey83]

@Крузяра, ну меня только на 3 хватает: 1 - глобальные изменения в коде, 2 - серьёзные нововведения (ну или смена синтаксиса), 3 - мелкие изменения и фиксы

 

[rejchev]

Почистил и оптимизировал немного код

Ну манипуляции с фигурными скобками(а точнее их отсутствие) больше снижает читабельность кода, нежели нечто большее(На текущее время написано множество статей/книг по такому поводу) =>
Код не чистит оптимизации не дает.
Новый синт. из этой же истории

if(!iArgs) for(int i; i < 3; i++) LoadFiles(g_szPaths[i], view_as<REGEX_FILES>(i));

При таком раскладе, всегда будет -1 или же будет индекс, но разделения не произойдет.
Тем более наличие "случайного дублирования" символа приведет к ложному форматированию. Не доглядел..

public void whiteCommands(ConVar cv, const char[] oldV, const char[] newV)
{
    char szBuffer[PLATFORM_MAX_PATH];
    cv.GetString(SZF(szBuffer));    TrimString(szBuffer);

    if(FindSlashPos(szBuffer) != -1)
        ExplodeString(szBuffer, "|", SZF(g_szWhiteCom), sizeof(g_szWhiteCom[]));
    else strcopy(g_szWhiteCom[0], sizeof(g_szWhiteCom[]), szBuffer);
}

stock int FindSlashPos(const char[] str)
{
    int i;
    while(str[i]) if(str[i] == '/') return i;
    return -1;
}

кстати, нумерация такого вида обычно только в версиях SM, в плагинах чаще вида x.x или x.x.x

На самом деле никогда не задумывался на этот счет, какой-либо конкретной привязки нет, что дает автору право выбора...

 

[Grey83]

Ну манипуляции с фигурными скобками(а точнее их отсутствие) больше снижает читабельность кода

мне больше всего ухудшает читаемость наличие ненужных скобок и расположение открывающей в конце строки :dntknw:

При таком раскладе, всегда будет -1 или же будет индекс, но разделения не произойдет.

почему ты так решил?

 

[rejchev]

почему ты так решил?

Хм, а ведь почему... До выхлопа то дело никогда не дойдет.
Цикл while будет бесконечно проверять первый символ буфера...

 

[Rezder]

Почему он наказывает за слово например апдаун т.е если в слове есть другое слово, таких же будет много слов и просто так муты будет выдавать.
Можно сделать чтобы он чисто за слово наказывал если оно точь в точь совпадает?

 

[rejchev]

Почему он наказывает за слово например апдаун т.е если в слове есть другое слово, таких же будет много слов и просто так муты будет выдавать.
Можно сделать чтобы он чисто за слово наказывал если оно точь в точь совпадает?

В след. обнове...

 

[Kruzya]

@Skeriiz, там хуже будет.
SM просто вырубит плагин. Это бесконечный цикл без результата.

stock int FindSlashPos(const char[] str)
{
    int i = -1;
    while(str[++i]) if(str[i] == '/') return i;
    return -1;
}

Вот так будет с результатом, если уж экономить на строчках.

 

[asd& [BLR]]

Css v34 SM 1.9.0.6190.
Плагин работает чудесно до первого наказания. После первого мута начинает просто выдавать предупреждения и по концовке их обнуляет, но никакого мута не дает.
В логах ошибок никаких нету.
После перезагрузки плагина опять таки выдает 1 мут и больше не хочет.

Также неполноценно работает с VIP чатом (проверял на випке от wS). Выдает предупреждения, но в чате не блокирует само сообщение.

 

[Kruzya]

Также неполноценно работает с VIP чатом (проверял на випке от wS).

Проблемы реализации самого вип чата. Препроцессор не используется, видимо.

 

[rejchev]

Пользователь Skeriiz обновил ресурс [CHAT] AnyChatFilter новой записью:

Фикс[#0.4]

• Исправлена проблема с выдачей наказания: [CHAT] AnyChatFilter

Узнать больше об этом обновлении...

 

[Grey83]

@Крузяра, ага, увеличение счётчика забыл добавить.

Исправил исходник в том сообщении
--- Добавлено позже ---
@Skeriiz, в функцию RestrictPlayer() нет необходимости передавать значение квара sm_acf_command в качестве одного из параметров, т.к. можно и так это значение в функции получить:

void RestrictPlayer(int client)
{
    char sUID[24], buffer[128];
    FormatEx(SZF(sUID), "#%i", GetClientUserId(client));
    strcopy(SZF(buffer), g_szCommand);
    ReplaceString(SZF(buffer), "{UID}", sUID, true);
    ServerCommand(buffer);
    g_iWarnCount[client] = 0;
}

 

[rejchev]

@Skeriiz, в функцию RestrictPlayer() нет необходимости передавать значение квара sm_acf_command в качестве одного из параметров, т.к. можно и так это значение в функции получить:

void RestrictPlayer(int client)
{
    char sUID[24], buffer[128];
    FormatEx(SZF(sUID), "#%i", GetClientUserId(client));
    strcopy(SZF(buffer), g_szCommand);
    ReplaceString(SZF(buffer), "{UID}", sUID, true);
    ServerCommand(buffer);
    g_iWarnCount[client] = 0;
}

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

 

[Kruzya]

А как по мне, если оставить передачу аргументом, то можно оставить гибкость функции. Нет привязки к конкретной глобальной переменной. В любой другой момент можно передать другую строку, и оно просто будет работать.

 

[Grey83]

@Крузяра, вот только другая переменная должна будет являться строковой и содержать (ну или не содержать) в себе вот эти символы "{UID}"
А также иметь отношение к счётчику предупреждений (g_iWarnCount[client] = 0;)
--- Добавлено позже ---
Да и вообще создавать отдельную функцию не было никакой необходимости, т.к. на неё ссылаются всего 1 раз в коде плагина
Обычно какие-либо действия выводить в отдельную функцию имеет смысл тогда, когда одна и та же последовательность действий используется несколько раз в коде.

 

[Kruzya]

@Grey83, необязательно.
Иногда вынести в отдельную функцию полезно, если планируется её использовать в других плагинах. Чтобы просто скопировать, и вставить в другом месте.

 

[Grey83]

@Крузяра, а что мешает просто скопировать и вставить не вынося в отдельную функцию?

 

[rejchev]

@Крузяра, а что мешает просто скопировать и вставить не вынося в отдельную функцию?

смысловое обособление, не иначе.

Бессмысленная дилемма..

 

[Kruzya]

@Grey83, как раз таки привязка к внешним дополнительным глобальным переменным.
Нет привязки - нет проблем при банальном копировании, ибо ничего не меняешь, а на компиляции не ловишь непонятных ошибок вроде "undefined variable".