DemoRecord (совет) [Решено]

[Good Luck]

Всем привет.

Хочу хоть немного научиться разбираться в написание плагинов и вот сделал "Demo Recording".

Плагин вроде бы работает, ошибок никаких нету.

Опытные ребята, подскажите все ли я верно сделал? Просто если плагин работает - это ещё не значит, что всё хорошо.

Если что-то нужно добавить, то подскажите, пожалуйста.

Главный вопрос нужна ли проверка на коннект и игнорирование ботов и hltv?

Код: Выделить всё

#include <amxmodx>

#define DEMONAME "bbgg"

public plugin_init()
{
    register_plugin("Demo Recording", "0,1", "asd");
    
    register_clcmd("joinclass", "DemoTask");
    register_clcmd("menuselect", "DemoTask");
}

public DemoTask(id)
{
    set_task(10.0, "DemoRec", id);
}

public DemoRec(id)
{    
    client_cmd(id, "stop; record ^"%s^"", DEMONAME);    
} 

Добавлено спустя 2 минуты 30 секунд:
В консоле вот, что написало, после начала записи демо:

Код: Выделить всё


Not recording a demo.
recording to bbgg.dem.
Completed demo
recording to bbgg.dem.

[Subb98]

Код: Выделить всё

#define DEMONAME "bbgg" 

>>

Код: Выделить всё

new const DEMONAME[] = "bbgg"; 

Обсуждение тут: Вы должны зарегистрироваться, чтобы видеть ссылки.

Достаточно будет зарегистрировать joinclass:

Код: Выделить всё

register_clcmd("joinclass", "DemoTask"); 

Нужно добавить проверку на коннект:

Код: Выделить всё

public DemoRec(id)
{    
    if(is_user_connected(id))
    {
        client_cmd(id, "stop; record ^"%s^"", DEMONAME);
    }
} 

И удаление задачи при дисконнекте:

Код: Выделить всё

public client_disconnect(id)
{
    remove_task(id);
} 

[Good Luck]

И удаление задачи при дисконнекте:

Код: Выделить всё

public client_disconnect(id)
{
    remove_task(id);
} 

А если я не буду использовать set_task, а решу сразу демо писать, то значит и не нужно будет прописывать remove_task.
Верно?

[Subb98]

А если я не буду использовать set_task, а решу сразу демо писать, то значит и не нужно будет прописывать remove_task.
Верно?

Да.

[Good Luck]

А здесь не нужно добавлять return PLUGIN_HANDLED?

Код: Выделить всё

public DemoRec(id)
{    
    if(is_user_connected(id))
    {
        client_cmd(id, "stop; record ^"%s^"", DEMONAME);
    }
}  

Вот так:

Код: Выделить всё

public DemoRec(id)
{    
    if(is_user_connected(id))
    {
        client_cmd(id, "stop; record ^"%s^"", DEMONAME);
    }
   return PLUGIN_HANDLED
}  

[Subb98]

Good Luck, нет.

[Good Luck]

"Обсуждение тут: Холивар: #define, const и new const" - почитал! Большое спасибо!

Добавлено спустя 25 минут 9 секунд:
Тему открыл снова. Проблема такая.

Делаю, как сказал Вы должны зарегистрироваться, чтобы видеть ссылки.:

Достаточно будет зарегистрировать joinclass:

Код: Выделить всё

register_clcmd("joinclass", "DemoTask"); 

И демо не работает (не записывается).

Вставлю 2 команды:
[
code=php]register_clcmd("joinclass", "StartDemo");
register_clcmd("menuselect", "StartDemo");[/code]
И демо сразу записывается, но в консоле появляются такие записи:

Код: Выделить всё

Not recording a demo.
recording to bbgg.dem.
Completed demo
recording to bbgg.dem.

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

Быть может проще вызывать записать демо через set_task и putinserver, нежели по выбору команды?

[Subb98]

Good Luck, очевидно, что необходим set_task.

[Garey]

Можно в принципе в первом spawn'е игрока т.к. до этого момента в принципе бессмысленно записывать часть игры.

[Good Luck]

Good Luck, очевидно, что необходим set_task.

Сделал с таском, но что-то все равно не работает...

Код: Выделить всё

#include <amxmodx>

new const ND[] = "bbgg";

public plugin_init()
{
    register_plugin("Demo Recording", "0.1", "asd");
    
    register_clcmd("joinclass", "DemoTask");
}

public DemoTask(id)
{
    set_task(10.0, "DemoRec", id);
}

public DemoRec(id)
{    
    if(is_user_connected(id))
    {
        client_cmd(id, "stop; record ^"%s^"", ND);
    }
} 

public client_disconnected(id)
{
    remove_task(id);
} 

Или вы имели ввиду не обойтись через task, который через "putinserver"?