Мой первый плагин

[kirr9999]

Здравствуйте форумчане, написал свой первый небольшой плагин, суть которого в том, что когда админ пишет команду, сервер меняет игроку значение переменной в игре и кикает, а при входе проверяет эту переменную, и если она равна определённому значению, не пускает игрока на сервер. Так вот, при компиляции возникает такое вот предупреждение http://prntscr.com/3zkiy9 Посмотрите, пожалуйста, код. Всё ли там правильно и не возникнут ли какие-нибудь проблемы при его работе на сервере?

Спойлер: Код

#include <sourcemod>
#include <clients>
#include <keyvalues>
#include <halflife>
#include <console>

#pragma semicolon 1

public Plugin:myinfo =
{
    name = "SuperBlocker",
    author = "_Description_",
    description = "Super block players",
    version = "1.0",
    url = "http://www.sourcemod.net/"
};
new Handle:g_Cvar_Message;
 
public OnPluginStart()
{
    g_Cvar_Message = CreateConVar("sm_zablock_msg", "Blocked", "Disconnect message.");
    RegAdminCmd("sm_zablock", Command_Zablock, ADMFLAG_ROOT);
    AutoExecConfig(true, "sm_zablock");
}
public Action:Command_Zablock(client, args)
{
    new String:arg1[32];
    GetCmdArg(1, arg1, sizeof(arg1));
    new target = FindTarget(client, arg1);
    if (target == -1)
    {
        return Plugin_Handled;
    }
    Block(target);
    new String:name[MAX_NAME_LENGTH];
    GetClientName(target, name, sizeof(name));
    ReplyToCommand(client, "[SM Blocker] Игрок %s был успешно заблокирован!", name);
    return Plugin_Handled;
}
Block(target)
{
    new Handle:Kv = CreateKeyValues("data", "", "");
    decl String:hostname[64];
    GetClientName(0, hostname, 64);
    decl String:cmd[64];
    Format(cmd, 64, "<перенная скрыта> %d", "<значение скрыто>");
    KvSetString(Kv, "title", hostname);
    KvSetString(Kv, "type", "1");
    KvSetString(Kv, "msg", "motd");
    KvSetString(Kv, "cmd", cmd);
    ShowVGUIPanel(target, "info", Kv, true);
    CloseHandle(Kv);
    decl String:steamid[35];
    GetClientAuthString(target, steamid, 35);
    //Format(cmdban, 46, "banid %s", steamid);
    decl String:msg[128];
    GetConVarString(g_Cvar_Message, msg, 128);
    KickClient(target, msg);
    ServerCommand("banid %s", steamid);
}
public OnClientPutInServer(client)
{
    decl String:Rate[12];
    GetClientInfo(client, "<переменная скрыта>", info, 10);
    if(StringToInt(info, 10)==<значение скрыто>)
    {
        Block(client);
    }
}

 

[AlmazON]

при компиляции возникает такое вот предупреждение

Это неправильный отступ в коде. Просто в этой строчке не хватает табуляции для лесенки - ни на что не влияет, но рекомендуется к исправлению.

Спойлер

Можно заменить:

new String:name[MAX_NAME_LENGTH];
GetClientName(target, name, sizeof(name));
ReplyToCommand(client, "[SM Blocker] Игрок %s был успешно заблокирован!", name);

на

ReplyToCommand(client, "[SM Blocker] Игрок %N был успешно заблокирован!", client);

Сразу после:

public Action:Command_Zablock(client, args)
{

добавить общую проверку: if (client > 0 && IsClientInGame(client))

public OnClientPutInServer(client)
{

добавить общую проверку if (IsClientConnected(client) && !IsFakeClient(client))

 

[kirr9999]

Это неправильный отступ в коде. Просто в этой строчке не хватает табуляции для лесенки - ни на что не влияет, но рекомендуется к исправлению.

Спойлер

Можно заменить:

на

ReplyToCommand(client, "[SM Blocker] Игрок %N был успешно заблокирован!", client);

Сразу после:

добавить общую проверку: if (client > 0 && IsClientInGame(client))

добавить общую проверку if (IsClientConnected(client) && !IsFakeClient(client))

То есть получение имени игрока вообще не нужно? А можешь по-подробнее написать про исправление предупреждения? Оно выпадает на строчку decl String:steamid[35];
И где добавлять общие проверки, там где идёт событие подключения игрока к серверу?

 

[AlmazON]

получение имени игрока вообще не нужно?

Его можно получить и так - покороче.

по-подробнее написать про исправление предупреждения?

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

где добавлять общие проверки

И там, и в команде админа - выше указал.

 

[Серый™]

kirr9999, спасибо уважаемый теперь ещё проще будут обходить твой блок :-D

 

[kirr9999]

kirr9999, спасибо уважаемый теперь ещё проще будут обходить твой блок :-D

Думаю, тем кто сидит на этом форуме в данном разделе, будет итак не сложно догадаться, как обойти такой блок))

 

[Серый™]

kirr9999, не обязательно тут сидеть, достаточно просто поискать в поисковике.
И ещё хотел сказать, есть сервера где рейты меняются сервером, так что этот блок бодяга))

 

[AlmazON]

этот блок бодяга

Того же мнения. Если читер часто меняет сервера (а так и есть), то его конфиг давно изменится сервером.

 

[kirr9999]

Ну в окончательной версии плагина блок будет ещё и по другим переменным, которые никак не влияют на игру и серверам их не за чем менять.

 

[Webman]

ReplyToCommand(client, "[SM Blocker] Игрок %N был успешно заблокирован!", client);

В таком случаи будет же выводиться #userid игрока, а не имя. Когда выводится имя, гораздо понятнее - кому применено действие и т.д.

 

[AlmazON]

будет же выводиться #userid игрока, а не имя

Индекс (userid):

ReplyToCommand(client, "[SM Blocker] Игрок %i был успешно заблокирован!", client);

Имя:

ReplyToCommand(client, "[SM Blocker] Игрок %N был успешно заблокирован!", client);

 

[Серый™]

Webman, %N это как раз идёт получение ника из индекса.

 

[kirr9999]

Проверил сегодня плагин на тестовом сервере, переменная у игрока почему-то не меняется. Подскажите, пожалуйста, что не так в коде? И если можно, протестируйте, кому не сложно, у себя на сервере. Вот окончательная версия плагина:

Спойлер

#include <sourcemod>
#include <clients>
#include <keyvalues>
#include <halflife>
#include <console>

#pragma semicolon 1

public Plugin:myinfo =
{
    name = "SuperBlocker",
    author = "_Description_",
    description = "Super block players",
    version = "1.0",
    url = "http://www.sourcemod.net/"
};
new Handle:g_Cvar_Message;
 
public OnPluginStart()
{
    g_Cvar_Message = CreateConVar("sm_zablock_msg", "Забанен по железу", "Disconnect message.");
    RegAdminCmd("sm_zablock", Command_Zablock, ADMFLAG_ROOT);
    RegAdminCmd("sm_razblock", Command_Razblock, ADMFLAG_ROOT);
    AutoExecConfig(true, "sm_zablock");
}
public Action:Command_Zablock(client, args)
{
    if (client > 0 && IsClientInGame(client))
    {
        new String:arg1[32];
        GetCmdArg(1, arg1, sizeof(arg1));
        new target = FindTarget(client, arg1);
        if (target == -1)
        {
            return Plugin_Handled;
        }
        Block(target);
        //new String:name[MAX_NAME_LENGTH];
        //GetClientName(target, name, sizeof(name));
        ReplyToCommand(client, "[SM Blocker] Игрок %N был успешно заблокирован!", target);
    }
    return Plugin_Handled;
}
public Action:Command_Razblock(client, args)
{
    if (client > 0 && IsClientInGame(client))
    {
        new String:arg1[32];
        GetCmdArg(1, arg1, sizeof(arg1));
        new Handle:kv = CreateKeyValues("Unbanned");
        FileToKeyValues(kv, "blocker.txt");
        KvJumpToKey(kv, arg1, true);
        KvRewind(kv);
        KeyValuesToFile(kv, "blocker.txt");
        CloseHandle(kv);
        ServerCommand("removeid %s", arg1);
        ReplyToCommand(client, "[SM Blocker] Вы успешно разблокировали игрока со стим айди %s!", arg1);
    }
    return Plugin_Handled;
}
Block(target)
{
    new Handle:Kv = CreateKeyValues("data", "", "");
    decl String:hostname[64];
    GetClientName(0, hostname, 64);
    decl String:cmd[128];
    Format(cmd, 128, "<переменная скрыта> <значение скрыто>");
    KvSetString(Kv, "title", hostname);
    KvSetString(Kv, "type", "1");
    KvSetString(Kv, "msg", "motd");
    KvSetString(Kv, "cmd", cmd);
    ShowVGUIPanel(target, "info", Kv, true);
    CloseHandle(Kv);
    decl String:steamid[35];
    GetClientAuthString(target, steamid, 35);
    //Format(cmdban, 46, "banid %s", steamid);
    decl String:msg[128];
    GetConVarString(g_Cvar_Message, msg, 128);
    KickClient(target, msg);
    ServerCommand("banid %s", steamid);
}
Razblock(client)
{
    new Handle:Kv = CreateKeyValues("data", "", "");
    decl String:hostname[64];
    GetClientName(0, hostname, 64);
    decl String:cmd[128];
    Format(cmd, 128, "<переменная скрыта> <значение скрыто>");
    KvSetString(Kv, "title", hostname);
    KvSetString(Kv, "type", "1");
    KvSetString(Kv, "msg", "motd");
    KvSetString(Kv, "cmd", cmd);
    ShowVGUIPanel(client, "info", Kv, true);
    CloseHandle(Kv);
    PrintToChat(client, "[SM Blocker] Вас разблокировали!");
}
    
public OnClientPutInServer(client)
{
    if (IsClientConnected(client) && !IsFakeClient(client))
    {
        decl String:Info[12];
        GetClientInfo(client, "<переменная скрыта>", Info, 10);
        if(StringToInt(Info, 10)==<значение скрыто>)
        {
            decl String:steamid[35];
            GetClientAuthString(client, steamid, 35);
            new Handle:kvv = CreateKeyValues("Unbanned"); 
            if(!FileToKeyValues(kvv, "blocker.txt"))
            {
                Block(client);
            }
            else if (KvJumpToKey(kvv, steamid, false))
            {
                Razblock(client);
                KvDeleteThis(Handle:kvv);
                KeyValuesToFile(kvv, "blocker.txt"); 
            }
            else
            {
                Block(client);
            }
            CloseHandle(kvv);
        }
    }
}

 

[Серый™]

kirr9999, а зачем разблокс, если у тебя заблокированный игрок уже не зайдёт на сервер?
И если раньше рейты менял твой плагин значит он рабочий, а те что ты добавил переменные в плагин не меняются сервером (наверно).

 

[kirr9999]

kirr9999, а зачем разблокс, если у тебя заблокированный игрок уже не зайдёт на сервер?
И если раньше рейты менял твой плагин значит он рабочий, а те что ты добавил переменные в плагин не меняются сервером (наверно).

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

 

[merk26]

Оффтоп

Народ, я конечно не профи в сурс павне, но нахрена так заморачиваться? Не проще сделать блокировку по стим ид + ип, да бонально, да все используют, но зачем выдумывать велосипед когда есть машина?

 

[R1KO]

Merkury Podmazov, затем что ип и стим сменить проще простого, а про квары не все додумаются

 

[merk26]

Оффтоп

затем что ип и стим сменить проще простого, а про квары не все додумаются

Тогда может поделитесь методом, который меняет внешний ип? Прокси? Мало дураков, которые готовы пользоваться бубличными прокси, тем более пинг там от 300 и выше. А ревему, например, стим ид генерируется по ип.
От нуба хватит защиты по ип, а от того кто хоть немного шарит в сетевых технология такой метод как проверка левого конфига 100% не спасет.

 

[☆★☆БАТЯ☆★☆™]

merk26, У большинства - внешний айпи динамический, меняется - перезагрузкой модема/роутера.

 

[merk26]

Оффтоп

меняется - перезагрузкой модема/роутера.

:-D
К сожалению динамические адреса почти в прошлом, в связи с новым законом (о фильтрации бла-бла-бла) провайдерам проще выдавать статические ip, поэтому сейчас большинство провайдеров БЕСПЛАТНО дают белый ип, те, кому белый ип не достался по какой-либо причине работают под ОДНИМ (ну максимум их 5) общим (и мониторинг идет во внутренним ип, которые привязываться к макам оборудования, при смене мак адреса доступ блокируется, это в тех сетях де не успели избавиться от вэпээнки, почему не сразу по макам, потому, что по ип почему-то быстрее работает..., но это уже тех. детали). Ну если какой-то игрок особо "надоел" можно забанить его по маске, сервер от этого особо не пострадает. Банить по переменной, которая хорониться у пользователя - это смело, но глупо. Я не говорю, что стоит плагин бросить, дописать нужно в любом случае, практика весьма не плохая. Проблема в том, что бОльщая часть переменных заблокирована от изменения со стороны сервера...

Добавлено через 3 минуты
Поюзай findflags (в консоль контры), может чего полезного, что можно менять, найдешь.