Online Helper 2.1.3

[Nordic Warrior]

sbelov020 добавил(а) новый ресурс:

Online Helper - Позволяет при отсутствии игроков менять и удерживать одну популярную карту для набора онлайна

Нажмите на эту ссылку, чтобы узнать больше

 

[steelzzz]

Что за х..
Зачем делать цикл на игроков, если есть get_playersnum() ?

 

[voed]

Зачем делать цикл на игроков, если есть get_playersnum() ?

get_players, если точнее, get_playersnum тут не подойдет

 

[Nordic Warrior]

steelzzz, voed, https://dev-cs.ru/threads/4081/
Обе эти функции не подходят.

 

[Sonyx]

sbelov020, прекрасно подходит get_players, просто вы так и не поняли как его использовать)

28 Окт 2018

так же

public check_players_1(id)
{
set_task(5.0, "check_players_1", 2)
}

решается одной функцией
https://dev-cs.ru/amxx/amxmodx/change_task/view

28 Окт 2018

Да и вообще весь функционал можно производить на 1 таске, сделав вилку на кол-ве игроков

 

[SergeyShorokhov]

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

 

[zhorzh78]

Если я правильно понял задумку...

#include <amxmodx>
#include <amxmisc>

#define PLUGIN "Online Helper"
#define VERSION "0.1"
#define AUTHOR "Idea by Nordic Warrior"

new const szTargetMap[] = "de_dust2_2x2";    // Карта по умолчанию
new cpTimelimit, g_iRestoreTimeLimit, bool:isTargetMap;

public plugin_init()
{
    register_plugin(PLUGIN, VERSION, AUTHOR);
    
    if(!is_map_valid(szTargetMap))
    {
        pause("ad");
        log_amx("The map name is not valid");
        return;
    }

    register_logevent("Rlog_Round_Draw", 4, "1=Round_Draw");
    register_logevent("Rlog_Game_Commencing", 2, "1=Game_Commencing");

    cpTimelimit = get_cvar_pointer("mp_timelimit");
    g_iRestoreTimeLimit = get_pcvar_num(cpTimelimit);

    new mapname[MAX_NAME_LENGTH];
    get_mapname(mapname, charsmax(mapname));

    if(!strcmp(mapname, szTargetMap))
        isTargetMap = true;
}

public Rlog_Game_Commencing()
{
    if(isTargetMap)
        set_pcvar_num(cpTimelimit, g_iRestoreTimeLimit);
}

public Rlog_Round_Draw()
{
    if(get_playersnum_ex(GetPlayers_ExcludeHLTV))
        return;

    if(isTargetMap)
    {
        set_pcvar_num(cpTimelimit, 0);
        return;
    }
    
    engine_changelevel(szTargetMap);
}

 

[Nordic Warrior]

решается одной функцией

опа. сегодня я выучил еще одну функцию благодарю

Да и вообще весь функционал можно производить на 1 таске, сделав вилку на кол-ве игроков

долго ломал над этим голову, так и не придумал ничего

прекрасно подходит get_players, просто вы так и не поняли как его использовать)

проверять кол-во собраных ид?..

cpTimelimit = get_cvar_pointer("mp_timelimit"); g_iRestoreTimeLimit = get_pcvar_num(cpTimelimit);

я вот никак не пойму, кто мне объяснит? зачем делать указатель на квар и потом уже менять квар/получать его значение, когда можно просто напрямую через get_cvar_num и set_cvar_num ?

 

[fantom]

я вот никак не пойму, кто мне объяснит? зачем делать указатель на квар и потом уже менять квар/получать его значение, когда можно просто напрямую через get_cvar_num и set_cvar_num

С указателем быстрее. Если задача получить квар единажды при старте карты то нет ничего плохого не использовать указатель. Но если получение квара происходит например в спавне или каждую секунду по таску то нужно уже использовать указатель

 

[Nordic Warrior]

fantom, вот как. понял, спасибо. переделаю.

if(!is_map_valid(szTargetMap)) { pause("ad"); log_amx("The map name is not valid"); return; }

а это обязательно делать? я сначала хотел тоже сделать проверку на мапу, но потом подумал, а толку? сервер ведь все равно не упадет от этого

engine_changelevel(szTargetMap);

чем это лучше простого changelevel?

 

[Sonyx]

проверять кол-во собраных ид?..

именно

 

[zhorzh78]

сервер ведь все равно не упадет от этого

карту не сменит

чем это лучше простого changelevel?

Тем, что он есть в АМХ 1.8.3 (1.9), а change_level - нет)

зачем делать указатель на квар и потом уже менять квар/получать его значение

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

 

[Nordic Warrior]

карту не сменит

ну так и так не сменит ведь.

 

[Nordic Warrior]

Ресурс Online Helper был обновлен:

Обновлено до версии 0.2

Нажмите на эту ссылку, чтобы узнать подробности обновления

 

[neygomon]

change_task(1, 5.0)

зачем???
у тебя и так цикл "вечный"(флаг "b") с периодом в 5 сек.

 

[Nordic Warrior]

neygomon, затем, что первый таск, если ты не заметил идет в init, 15 секунд. Сделано, чтоб между сменой карты хотя бы один игрок успел приконнектиться, и плагин бы не начинал с нуля игроков

 

[Denzer]

1.

#define MAP "35hp_2"    // Карта по умолчанию

Как по мне лучше это вынести в константу, а не в макрос.
2.

set_task(15.0, "check_players_1", 1,.flags="b")

Айди таска лучше выводить в макросы, дабы код был более читабельный.
Пример:

#define CHECK_PLAYERS 5555
set_task(15.0, "check_players_1", CHECK_PLAYERS, .flags= "b")

3.

public check_players_1(id)

public check_players_2(id)

Зачем тут id?
4.

if(Count == 0)

Такое можно писать так:

if(!Count)

Лишь маленькие поправки по самому коду, логику плагина не смотрел.

 

[fantom]

if(Count == 0)

Такое можно писать так:

if(!Count)

Я все же предпочел бы оригинальный вариант. Он более наглядно показывает суть проверки. Легче для понимания.

 

[Nordic Warrior]

лучше это вынести в константу

А можно пояснить, почему? Если это не несет никакой технической выгоды, то мне больше нравится дефайном.

Айди таска лучше выводить в макросы, дабы код был более читабельный.

это я знаю, не хотелось в таком крохотном плагине загромождать дефайнами.

Зачем тут id?

тьфу, не подумал. исправлено)

Такое можно писать так

думаю разницы нет?

 

[neygomon]

sbelov020,

plugin_init()
{
    set_task(15.0, "start_check");
}

public start_check()
{
    check_players_1();
    set_task(5.0, "check_players_1", 1,.flags="b")
}