item_pickup или SDKHook_WeaponCanUse

[Черная вдова]

Что лучше использовать что бы заблокировать подбор оружия ?
item_pickup или SDKHook_WeaponCanUse
Мне надо что бы оно оставалось на полу, через item_pickup я так понимаю нагрузка меньше будет
У меня получилось только приходится его периодически дропать, а не просто оставлять на земле, а можно как то лучше ?

 

[R1KO]

Лучше сдк хукс. Может и чуть затратнее, зато у тебя будет и индекс оружия и имя и всё что только захочешь, к тому же чтобы запретить достаточно просто Plugin_Handled вернуть. И разница в нагрузке не настолько существенна чтобы переживать по этому поводу. Это же не хук трансмит

 

[Черная вдова]

Ок, а слот можно узнать в какой переместиться ?

 

[Monomizer]

Парни, сорри что влезаю, но темы отдельной не стоит...
Кароче в 1.6. есть профилирование для "таких" вопросов, а sourcemod подобие такого есть, именно в файле?
видел вроде бы была команда sm profile etc...
Руки ещё не доходили

 

[NiGaByte]

Почему не срабатывает? Хукаю в конце раунда, UnХукаю тут, либо в начале раунда, и нечего не получается.

public Action OnPlayerSpawn(Handle:event, const String:name[], bool:dontBroadcast)
{
    for( int iClient = 1; iClient <= MaxClients; iClient++ )
    {
        SDKUnhook(iClient, SDKHook_WeaponCanUse, UnBlockWeaponCanUse);
    }
}

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Continue;
}

Как быть?

 

[Someone]

OnPlayerSpawn

Там ведь не событие возрождения, начало раунда, да ?

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
return Plugin_Continue;
}

А что ты от кода ожидаешь ?

 

[NiGaByte]

Там ведь не событие возрождения, начало раунда, да ?


А что ты от кода ожидаешь ?

я хукаю SDKHook_WeaponCanUse в событии OnRoundEnd и делаю return Plugin_Handled;
Далее, мне нужно чтобы SDKHook_WeaponCanUse УнХукнулся, и делаю то что написал выше, и без разницы когда, в начале раунда или при спавне.
--- Добавлено позже ---
[SIZE=4]@Someone[/SIZE][SIZE=2] [/SIZE],

public OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    for( int iClient = 1; iClient <= MaxClients; iClient++ )
    {
        SDKHook(iClient, SDKHook_WeaponCanUse, BlockWeaponCanUse);
    }
}

public Action:BlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Handled;
}

public OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
   for( int iClient = 1; iClient <= MaxClients; iClient++ )
   {
        SDKUnhook(iClient, SDKHook_WeaponCanUse, UnBlockWeaponCanUse);
   }
}

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Continue;
}

Вот так делаю, хотя это немного нагибает мою логику, я сначала хукаю, а потом унХукаю, и при этом возвращаю Continue;
Как правильно сделать? Ну не кодил я раньше е-маё... Знания есть, опыта 5 дней. Подскажите, будьте так добры.

 

[Someone]

я хукаю SDKHook_WeaponCanUse в событии OnRoundEnd и делаю return Plugin_Handled;
Далее, мне нужно чтобы SDKHook_WeaponCanUse УнХукнулся, и делаю то что написал выше, и без разницы когда, в начале раунда или при спавне.
--- Добавлено позже ---
@Someone,

public OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    for( int iClient = 1; iClient <= MaxClients; iClient++ )
    {
        SDKHook(iClient, SDKHook_WeaponCanUse, BlockWeaponCanUse);
    }
}

public Action:BlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Handled;
}

public OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
   for( int iClient = 1; iClient <= MaxClients; iClient++ )
   {
        SDKUnhook(iClient, SDKHook_WeaponCanUse, UnBlockWeaponCanUse);
   }
}

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Continue;
}

Вот так делаю, хотя это немного нагибает мою логику, я сначала хукаю, а потом унХукаю, и при этом возвращаю Continue;
Как правильно сделать? Ну не кодил я раньше е-маё... Знания есть, опыта 5 дней. Подскажите, будьте так добры.

#include <sdkhooks>

#pragma semicolon 1
#pragma newdecls required

bool g_bBlock;    // true блокирует

public void OnPluginStart()
{
    HookEvent("round_start",    Event_RoundStart,      EventHookMode_PostNoCopy);
    HookEvent("round_end",         Event_RoundEnd,      EventHookMode_PostNoCopy);
}

public void OnClientPutInServer(int iClient)
{
    SDKHook(iClient, SDKHook_WeaponCanUse, WeaponCanUse); // Разхукает сам
}

public void Event_RoundStart(Event hEvent, const char[] name, bool bDontBroadcast)
{
    g_bBlock = false;
}

public void Event_RoundEnd(Event hEvent, const char[] name, bool bDontBroadcast)
{
    g_bBlock = true;
}

public Action WeaponCanUse(int iClient, int iWeapon)
{
    switch(g_bBlock)
    {
        case 0: return Plugin_Continue;
        case 1: return Plugin_Handled;
    }
}

 

[NiGaByte]

public OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    for( int iClient = 1; iClient <= MaxClients; iClient++ )
    {
        SDKHook(iClient, SDKHook_WeaponCanUse, BlockWeaponCanUse);
    }
}

public Action:BlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Handled;
}

public OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
   for( int iClient = 1; iClient <= MaxClients; iClient++ )
   {
        SDKUnhook(iClient, SDKHook_WeaponCanUse, UnBlockWeaponCanUse);
   }
}

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Continue;
}

Вот так делаю, хотя это немного нагибает мою логику, я сначала хукаю, а потом унХукаю, и при этом возвращаю Continue;
Как правильно сделать? Ну не кодил я раньше

#include <sdkhooks>

#pragma semicolon 1
#pragma newdecls required

bool g_bBlock;    // true блокирует

public void OnPluginStart()
{
    HookEvent("round_start",    Event_RoundStart,      EventHookMode_PostNoCopy);
    HookEvent("round_end",         Event_RoundEnd,      EventHookMode_PostNoCopy);
}

public void OnClientPutInServer(int iClient)
{
    SDKHook(iClient, SDKHook_WeaponCanUse, WeaponCanUse); // Разхукает сам
}

public void Event_RoundStart(Event hEvent, const char[] name, bool bDontBroadcast)
{
    g_bBlock = false;
}

public void Event_RoundEnd(Event hEvent, const char[] name, bool bDontBroadcast)
{
    g_bBlock = true;
}

public Action WeaponCanUse(int iClient, int iWeapon)
{
    switch(g_bBlock)
    {
        case 0: return Plugin_Continue;
        case 1: return Plugin_Handled;
    }
}

А свитчем не очень жестко? МБ выгодней If?

 

[Someone]

public OnRoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    for( int iClient = 1; iClient <= MaxClients; iClient++ )
    {
        SDKHook(iClient, SDKHook_WeaponCanUse, BlockWeaponCanUse);
    }
}

public Action:BlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Handled;
}

public OnRoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
   for( int iClient = 1; iClient <= MaxClients; iClient++ )
   {
        SDKUnhook(iClient, SDKHook_WeaponCanUse, UnBlockWeaponCanUse);
   }
}

public Action:UnBlockWeaponCanUse(iClient, weapon)
{
    return Plugin_Continue;
}

Вот так делаю, хотя это немного нагибает мою логику, я сначала хукаю, а потом унХукаю, и при этом возвращаю Continue;
Как правильно сделать? Ну не кодил я раньше

А свитчем не очень жестко? МБ выгодней If?

switch быстрее, но в этой ситуации не знаю.

 

[NiGaByte]

А ну я типа вкурил.
--- Добавлено позже ---
только меня напрягает глобальная переменная, чёт в падлу хранить её там.
--- Добавлено позже ---
@Someone, он так ругаеться что WeaponCanUse(iClient, weapon) должен вернуть значение...
--- Добавлено позже ---
warning 209: function "WeaponCanUse" should return a value

 

[Someone]

только меня напрягает глобальная переменная, чёт в падлу хранить её там.

Это в разы лучше, чем хукать и анхукать.

@Someone, он так ругаеться что WeaponCanUse(iClient, weapon) должен вернуть значение...

Ну тогда:

public Action WeaponCanUse(int iClient, int iWeapon)
{
    if(g_bBlock)    return Plugin_Handled
    return Plugin_Continue;
}

 

[NiGaByte]

@Someone, он так ругаеться что WeaponCanUse(iClient, weapon) должен вернуть значение...

Ну тогда:

а я могу не чего не передавать?
ну типа

public Action:WeaponCanUse()
{
    switch(g_bBlock)
    {
        case 0: return Plugin_Continue;
        case 1: return Plugin_Handled;
    }
}

 

[Someone]

он так ругаеться что WeaponCanUse(iClient, weapon) должен вернуть значение...

Если сделал, как я написал, то не будет.

 

[NiGaByte]

@Someone, он так ругаеться что WeaponCanUse(iClient, weapon) должен вернуть значение...

а я могу не чего не передавать?
ну типа

public Action:WeaponCanUse()
{
    switch(g_bBlock)
    {
        case 0: return Plugin_Continue;
        case 1: return Plugin_Handled;
    }
}

Не не могу.
--- Добавлено позже ---
@Someone, а какое событие первее, round_start или player_spawn?

 

[R1KO]

@NiGaByte,

round_start

но ксго как всегда выделилась и в ней вроде идет так:

1. round_prestart
2. player_spawn
3. round_start

 

[Danyas]

Это в разы лучше, чем хукать и анхукать.


Ну тогда:

public Action WeaponCanUse(int iClient, int iWeapon)
{
    if(g_bBlock)    return Plugin_Handled
    return Plugin_Continue;
}

public Action WeaponCanUse(int iClient, int iWeapon)
{
    return g_bBlock ? Plugin_Handled : Plugin_Continue;
}

Тогда так :D