Как лучше написать плагин?

Vladius · 6 Фев 2016

Такая ситуация. Я ввожу команду, и на следующем раунде игроки спавнятся с калашами. Как лучше реализовать:
1) Хукнуть эвенты при загрузке плагина и в каждом евенте проверять переменную bool, которую команда будет ставить на true.
2) В команде хукать эвенты, а в конце раунда разхукивать.
Так:

C-подобный:

#include <sourcemod>
#include <sdktools>

new bool:round=false;

public void OnPluginStart()
{
    HookEvent("player_spawn", Event_PlayerSpawned);
    HookEvent("round_end", Event_RoundEnd);
}
public void Event_PlayerSpawned(Handle:event, const String:name[], bool:dontBroadcast)
{
    if(round)
    {
        int ent = GetPlayerWeaponSlot(GetClientOfUserId(GetEventInt(event,"userid")), 0);
        if(ent != -1)
        {
            RemovePlayerItem(GetClientOfUserId(GetEventInt(event,"userid")), ent);
            RemoveEdict(ent);       
        }
        GivePlayerItem(GetClientOfUserId(GetEventInt(event,"userid")), "weapon_ak47");
    }
}
public void Event_RoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
        if(round){
            round=false;
        }
}
public Action:SpecialRound(client, args)
{
    if (args != 1)
    {
        ReplyToCommand(client, "[SM] Usage: sm_specround <weapon>");
        return Plugin_Handled;
    }
    GetCmdArg(1, weapon, sizeof(weapon));
    round=true;
   
    return Plugin_Handled;
   
}

Или так:

C-подобный:

#include <sourcemod>
#include <sdktools>

public void OnPluginStart()
{
   
    HookEvent("round_start", Event_RoundStart);
    HookEvent("player_spawn", Event_PlayerSpawned);
    HookEvent("round_end", Event_RoundEnd);
   
}
public void Event_RoundStart(Handle:event, const String:name[], bool:dontBroadcast)
{
    HookEvent("round_end", Event_RoundEnd);
}
public void Event_PlayerSpawned(Handle:event, const String:name[], bool:dontBroadcast)
{
   
        int ent = GetPlayerWeaponSlot(GetClientOfUserId(GetEventInt(event,"userid")), 0);
        if(ent != -1)
        {
            RemovePlayerItem(GetClientOfUserId(GetEventInt(event,"userid")), ent);
            RemoveEdict(ent);       
        }
        GivePlayerItem(GetClientOfUserId(GetEventInt(event,"userid")), weapon);
   
}
public void Event_RoundEnd(Handle:event, const String:name[], bool:dontBroadcast)
{
    UnhookEvent("round_start");
    UnhookEvent("player_spawn");
    UnhookEvent("round_end");
       
   
}
public Action:SpecialRound(client, args)
{
    if (args != 1)
    {
        ReplyToCommand(client, "[SM] Usage: sm_specround <weapon>");
        return Plugin_Handled;
    }
    GetCmdArg(1, weapon, sizeof(weapon));
    HookEvent("round_start", Event_RoundStart);
    HookEvent("player_spawn", Event_PlayerSpawned);
   
    return Plugin_Handled;
   
}

Надеюсь, что понятно объяснил.

AlmazON · 6 Фев 2016

Vladius написал(а):
2) В команде хукать эвенты, а в конце раунда разхукивать.

Если не часто исполняется, то это. Но в самих событиях ещё тот бардак.

DarklSide · 6 Фев 2016

Если (ан)хукать события, то отказатся в использовании команды в первом раунде.

По идее сами события не часто исполняются, поэтому лучше их микс.

При вслучае с булевой: использовать команду можно, только на этапе (после конца раунда и до начала следующего). Поэтому убрать событие "round_end" и попытаться проверять булевую при старте раунда, и (ан)хукать событие спавн игрока, а если csgo то лучше событие "round_prestart").
Еще добавить булевую - на состояние собития спавна (есть его хук или нет).

AlmazON · 6 Фев 2016

DarklSide написал(а):
попытаться проверять булевую при старте раунда, и (ан)хукать событие спавн игрока

В событии round_start? Бесполезная затея, событие спавна вызывается до начала раунда. Оффтоп

DarklSide написал(а):
csgo то лучше событие "round_prestart"

Вот тут сработает верно. Оффтоп
Вообще, вместо события спавна лучше использовать хук "игрок взял/поднял оружие", так как оно может лежать на карте/выдаваться другими плагинами, да и покупать никто не запретил... Оффтоп

Vladius · 6 Фев 2016

AlmazON написал(а):
В событии round_start? Бесполезная затея, событие спавна вызывается до начала раунда. Оффтоп
После - только для подключившихся игроков/новозаходящих в команду.

Вот тут сработает верно. Оффтоп
Так "грамотно" описал, что я всё равно не до конца понимаю затею.

Вообще, вместо события спавна лучше использовать хук "игрок взял/поднял оружие", так как оно может лежать на карте/выдаваться другими плагинами, да и покупать никто не запретил... Оффтоп
Если верно помню, именно вместо - вызов идёт и при спавне, когда выдаётся стандартное/купленное оружие. Полезнее будет.

Я для своего авп сервера делаю, так что на карте точно ничего не валяется.

DarklSide написал(а):
Если (ан)хукать события, то отказатся в использовании команды в первом раунде.

По идее сами события не часто исполняются, поэтому лучше их микс.

При вслучае с булевой: использовать команду можно, только на этапе (после конца раунда и до начала следующего). Поэтому убрать событие "round_end" и попытаться проверять булевую при старте раунда, и (ан)хукать событие спавн игрока, а если csgo то лучше событие "round_prestart").
Еще добавить булевую - на состояние собития спавна (есть его хук или нет).

Если дописать 1 код, то можно будет использовать и в середине раунда, я пока так и делаю. Можете пояснить, что такое микс?

DarklSide · 6 Фев 2016

Vladius написал(а):
Можете пояснить, что такое микс?

Их сочетание:

PHP:

#include <sdktools>
#pragma newdecls required
bool round = false;
bool spawn = false;
char weapon[20];
public void OnPluginStart()
{
   HookEvent("round_prestart", Event_RoundStart);
   RegAdminCmd("sm_specround", SpecialRound, ADMFLAG_GENERIC, "sm_specround ak47");
}
public void Event_RoundStart(Event event, const char[] name, bool dontBroadcast)
{
   if (round)
   {
     HookEvent("player_spawn", Event_PlayerSpawned);
     round = false;
     spawn = true;
   }
   else if (spawn)
   {
     UnhookEvent("player_spawn", Event_PlayerSpawned);
     spawn = round;
   }
}
public void Event_PlayerSpawned(Event event, const char[] name, bool dontBroadcast)
{
   int client = GetClientOfUserId(event.GetInt("userid"));
   int ent = GetPlayerWeaponSlot(client, 0);
   if (ent != -1)
   {
     RemovePlayerItem(client, ent);
     RemoveEdict(ent);
   }
   GivePlayerItem(client, weapon);
}
public Action SpecialRound(int client, int args)
{
   if (args != 1)ReplyToCommand(client, "[SM] Usage: sm_specround <weapon>");
   else
   {
     GetCmdArgString(weapon, sizeof(weapon));
     Format(weapon, sizeof(weapon), "weapon_%s", weapon);
     round = true;
   }
   return Plugin_Handled;
}
public Action CS_OnBuyCommand(int client, const char[] weapons)
{
   if (spawn)
   {
     printCl(client);
     return Plugin_Handled;
   }
   return Plugin_Continue;
}
void printCl(int client)
{
   PrintToChat(client, "Раунд на выданное оружие");
}
public Action CS_OnCSWeaponDrop(int client, int weaponIndex)
{
   if (spawn && !GetEntityClassname(weaponIndex, weapon, sizeof(weapon))) // && GetEntityClassname... - не разрешить дроп на выданное оружие.
   {
     printCl(client);
     return Plugin_Handled;
   }
   return Plugin_Continue;
}

AlmazON · 6 Фев 2016

DarklSide написал(а):
spawn = round;

Ничего не забыто? Нет логики.

DarklSide написал(а):
Format(weapon, sizeof(weapon), "weapon_%s", GetCmdArg(1, weapon, sizeof(weapon)));

Забавно даже) Пытаемся форматировать строку для оружия из количества записанных символов)

DarklSide написал(а):
CS_OnBuyCommand

На

Vladius написал(а):
авп сервера

Обычно нет зон покупок, как я понимаю - не требуется.

DarklSide написал(а):
GetEntityClassname

Сейчас никуда не ведёт... Оффтоп

Monomizer · 6 Фев 2016

AlmazON написал(а):
Ничего не забыто? Нет логики.
Забавно даже) Пытаемся форматировать строку для оружия из количества записанных символов)
На
Обычно нет зон покупок, как я понимаю - не требуется.
Сейчас никуда не ведёт... Оффтоп
Ну и советы...

Оффтоп

DarklSide · 6 Фев 2016

AlmazON написал(а):
Нет логики.

Есть.

AlmazON написал(а):
Забавно даже) Пытаемся форматировать строку для оружия из количества записанных символов)

Забавно.
P.S.: если сначала не идет получение any.

AlmazON написал(а):
На awp Обычно нет зон покупок, как я понимаю - не требуется.

AlmazON написал(а):
Сейчас никуда не ведёт...

Некоторое мульти (без использования pickup'a и получения тайма времени).

AlmazON · 6 Фев 2016

@Monomizer, Оффтоп

DarklSide написал(а):
Есть

DarklSide написал(а):
{UnhookEvent("player_spawn", Event_PlayerSpawned);
spawn = round;
}}

Наконец заметил фигурную скобочку... Ты бы ещё 2 стиля написания впихнул)

DarklSide написал(а):
Некоторое мульти (без использования pickup'a и получения тайма времени)

Ну какая нам разница, удалось получить classname оружия или нет? Вот если бы его после сравнить... Тогда получим фильтр:

DarklSide написал(а):
не разрешить дроп на выданное оружие

DarklSide · 6 Фев 2016

PHP:

public void Event_RoundStart(Event event, const char[] name, bool dontBroadcast)
{
   if (round)
   {
     HookEvent("player_spawn", Event_PlayerSpawned);
     round = false;
   }
   else if (spawn){UnhookEvent("player_spawn", Event_PlayerSpawned);
   spawn = round;
}}

P.S.: при вставке сообщения удалил блок, после парсинга TextFX'ом.

AlmazON · 6 Фев 2016

Оффтоп

DarklSide написал(а):
Format(weapon, sizeof(weapon), "weapon_%s", GetCmdArg(1, weapon, sizeof(weapon)));

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

DarklSide · 6 Фев 2016

PHP:

public Action SpecialRound(int client, int args)
{
    if (args != 1)ReplyToCommand(client, "[SM] Usage: sm_specround <weapon>");
    else
    {
        GetCmdArgString(weapon, sizeof(weapon));
        Format(weapon, sizeof(weapon), "weapon_%s", weapon);
        spawn = true;
        round = true;
    }
    return Plugin_Handled;
}

В этом случае лучше получать все строку, раз она задана.

Vladius · 8 Фев 2016

Честно говоря, я так до конца и не уяснил ответ на свой вопрос.

inklesspen · 10 Фев 2016

Возможно это не то, что вам нужно, но времени у меня щас не особо много, а помочь хочу:)

Vladius · 19 Фев 2016

inklesspen написал(а):
Возможно это не то, что вам нужно, но времени у меня щас не особо много, а помочь хочу:)

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

Hejter · 22 Фев 2016

Vladius написал(а):
Спасибо, но дело в том, что у меня уже есть рабочий плагин, просто хотел оптимизировать. Сейчас у меня уже другая проблема. После последнего обновления на карте awp_lego_2 если был установлен раунд на калашх, то в следующем раунде у выживших игроков калаши не пропадут, как раньше. То есть нужно самому удалять у них калаши. А как это сделать я не знаю.

Получаешь слот оружия GetPlayerWeaponSlot и делаешь цикл по всем слотам оружия и удаляешь.
Цикл: for (int i = 0; i < 5; ++i)

Vladius · 24 Фев 2016

Hejter написал(а):
Получаешь слот оружия GetPlayerWeaponSlot и делаешь цикл по всем слотам оружия и удаляешь.
Цикл: for (int i = 0; i < 5; ++i)

Так не получится, потому что умершие игроки останутся без оружия. Я пока жду фикса, но вот уже 4 обновления ничего не решают.

Hejter · 25 Фев 2016

Vladius написал(а):
Так не получится, потому что умершие игроки останутся без оружия. Я пока жду фикса, но вот уже 4 обновления ничего не решают.

А зачем тебе умершим оружие? что за бред.

Vladius · 3 Мар 2016

Hejter написал(а):
А зачем тебе умершим оружие? что за бред.

Короче, у игроков оружия просто не будет от таких действий.

Как лучше написать плагин?

Участник

Не путать с самим yand3xmail

Участник

Не путать с самим yand3xmail

Участник

Участник

Не путать с самим yand3xmail

Держу JDW в бане.

Участник

Не путать с самим yand3xmail

Участник

Не путать с самим yand3xmail

Участник

Участник

Не пишу модули под LSD :с

Вложения

Участник

xor ebx, ebx

Участник

xor ebx, ebx

Участник