ZL_DLG v1.1 | Движок диалогов

[$continue$]

Открой инклуд - вот тебе сорурс.

хм, а почему бы не выложить пастбин?
Зачем мне качать инклуд, ради просмотра кода?

[Desulaid]

хм, а почему бы не выложить пастбин?
Зачем мне качать инклуд, ради просмотра кода?

а лучше код на стол и под спойлер

[Salvacore]

Баян же ?
Londlem подобное юзает вроде.

- - - Добавлено - - -

Update.

[L0ndl3m]

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

Сам паблик:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[]) 
{ 
    new 
        dlg_rel[10], 
        dlgid_str[6]; // макс. ID диалога не может быть больше чем 32767, поэтому 6 ячеек ( +1 для нуль-символа: '\0' ) 
 
    valstr(dlgid_str, dialogid, true); 
    dlg_rel = "dlg_", strcat(dlg_rel, dlgid_str); 
 
    return CallLocalFunction(dlg_rel, "iiis", playerid, response, listitem, (inputtext[0] == EOS) ? ("\1") : (inputtext)); 
}  

Насчёт valstr можно не беспокоиться ( он вызывает зависание сервера при максимальных и минимальных значениях ): значение никогда не достигнет cellmax или cellmin, из-за тех же лимитов для ID диалогов: 0-32767.

Сам макрос:

#define DLG(%1) \ 
    public dlg_%1(playerid, response, listitem, inputtext[]); \ 
    public dlg_%1(playerid, response, listitem, inputtext[])  

Как по мне, так макрос

const 
    DIALOG_REGISTER = 64; 
 
DLG(DIALOG_REGISTER) 
{ 
    // some code... 
}  

лучше выглядит чем

const 
    DIALOG_REGISTER = 64; 
 
DLG:DIALOG_REGISTER() 
{ 
    // some code... 
}  

Пустые скобки не очень смотрятся, лично для меня. Дело принципа.

[Zaur_Lumanov]

В новой версии подправлю. Если есть предложения, то я их готов учесть.

[Daniel_Cortez]

Пример:

DLG:590()
{
     if(response) return SendClientMessage(playerid,-1,"Вы нажали кнопку 'Выбрать' ");
     else return SendClientMessage(playerid,-1,"Вы вышли из диалогового окна.");
     return true;
}  

Параметры у диалоговой функции пропущены.
Из-за этого такое ощущение, будто параметры playerid и response берутся из ниоткуда, т.к. они нигде не объявлены.
Некоторые редакторы кода могут ругаться на такое.

const
    DIALOG_REGISTER = 64;

DLG(DIALOG_REGISTER)
{
    // some code...
}  

Не прокатит, вместо dlg_64 получится функция dlg_DIALOG_REGISTER.
Тут только объявлять ID диалогов через #define, причём сразу можно ставить крест на составлении более удобного списка ID диалогов с помощью enum.
+ те же самые проблемы с необъявленными параметрами.

Да и вообще, зачем пытаться выдумать что-то своё, когда можно воспользоваться уже накатанной лыжнёй, взяв пример с ZCMD?
Многие пользуются им (или совместимым с ним DC_CMD) и поэтому привыкли к их синтаксису.
Там командные функции объявляются следующим образом:

CMD:cmdname(playerid, params[])
{
    //...
}  

Если же пользователю не нужен параметр params, его можно пропустить:

CMD:test(playerid)
    return SendClientMessage(playerid, -1, "Sample text");  

playerid тоже можно пропустить (но нельзя пропустить его, не пропустив params):

CMD:shutdown()
    return SendRconCommand("exit");  

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    new
        dlg_rel[10],
        dlgid_str[6]; // макс. ID диалога не может быть больше чем 32767, поэтому 6 ячеек ( +1 для нуль-символа: '\0' )

    valstr(dlgid_str, dialogid, true);
    dlg_rel = "dlg_", strcat(dlg_rel, dlgid_str);

    return CallLocalFunction(dlg_rel, "iiis", playerid, response, listitem, (inputtext[0] == EOS) ? ("\1") : (inputtext));
}  

Для чего здесь две строковые переменные, когда можно использовать всего одну?

    static dlg_func_name[4+5+1] = "dlg_";
    valstr(dlg_func_name[4], dialogid, true);  

Данному каллбеку передаются следующие значения:
playerid (int) - ID игрока.
dialogid (int) - ID диалога.
response (bool) - Статус
listitem (int) - Выбранный элемент.
inputtext (string) - Текст, введённый в текстовое поле диалога.

Раз параметры те же, что в OnDialogResponse, зачем ещё раз расписывать их здесь?
Можно было просто сделать ссылку на статью в SA:MP wiki.

[wiki]OnDialogResponse[/wiki]

Важно: не прописывайте данную строчку до строчки "#include <a_samp>".

Очень просто лечится добавлением "#include <a_samp>" в самое начало вашего инклуда, после защиты от повторного подключения.

Кстати, про ту защиту...

#if defined _zl_dlg
    #endinput
#else
    #define _zl_dlg  

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

[seriu]

static dlg_func_name[4+5+1] = "dlg_";
    valstr(dlg_func_name[4], dialogid, true);  

Пытался вывести, через printf, выводило только "dlg_";

static dlg_func_name[4+5+1] = "dlg_";
    valstr(dlg_func_name[5], 233, true);
    printf("TEST %s",dlg_func_name);