Как лучше объявить переменную?

Nekro · 12 Фев 2021

Как лучше использовать переменны?

1.

C-подобный:

ConVar g_сEnable = null;

2.

C-подобный:

Handle g_hEnable = INVALID_HANDLE;

Есть ли вообще смысл от INVALID_HANDLE и null ?
3. Или лучше объявить на прямую тот же int/bool/float ?

Вот 3 примера, что лучше по произмодительсноти?

C-подобный:

ConVar
    g_cEnablePlugin;

public void OnPluginStart()
{
    g_cEnablePlugin = CreateConVar("sm_enable", "1", "Enable/Disalbe", _, true, 0.0, true, 1.0);
    HookEvent("round_end", OnEnd);
}

public void OnEnd(Handle event, char[] name, bool dontBroadcast)
{
    if(!g_cEnablePlugin.BoolValue)
        return;
}

C-подобный:

Handle
    g_hEnablePlugin;
    
bool
    bEnabled;

public void OnPluginStart()
{
    g_hEnablePlugin = CreateConVar("sm_enable", "1", "Enable/Disalbe", _, true, 0.0, true, 1.0);
    bEnabled = GetConVarBool(g_hEnablePlugin);
    HookEvent("round_end", OnEnd);
}

public void OnEnd(Handle event, char[] name, bool dontBroadcast)
{
    if(!bEnabled)
        return;
}

C-подобный:

bool
    bEnable;

public void OnPluginStart()
{
    ConVar cvar;
    cvar = CreateConVar("sm_enable", "1", "Enable/Disalbe", _, true, 0.0, true, 1.0);
    cvar.AddChangeHook(CVarChanged_Enable);
    bEnable = cvar.BoolValue;
    
    HookEvent("round_end", OnEnd);
}

public void CVarChanged_Enable(ConVar CVar, const char[] oldValue, const char[] newValue)
{
    bEnable = CVar.BoolValue;
}

public void OnEnd(Handle event, char[] name, bool dontBroadcast)
{
    if(!bEnable)
        return;
}

Rabb1t · 12 Фев 2021

Ну конвар это новый метод, хэндл старый формат для тех же кваров (и не только). По факту сейчас конвар это дочерний от хэндла (как и ArrayList, StringMap и тд).

Nekro · 12 Фев 2021

Rabb1t написал(а):
Ну конвар это новый метод, хэндл старый формат для тех же кваров (и не только). По факту сейчас конвар это дочерний от хэндла (как и ArrayList, StringMap и тд).

То есть лучше на прямую? Вроде чем больше хенделов тем хуже или как?

Rabb1t · 12 Фев 2021

Nekro написал(а):
То есть лучше на прямую?

Что значит напрямую? Использовать ConVar? Так это обёртка для Handle, но только для использования именно кваров. Просто методмап, использующий всё те же функции.

Nekro написал(а):
Вроде чем больше хенделов тем хуже или как?

Немного дичь, т.к. проблем не будет если ты лишний раз создать тот же квар.

Nekro · 12 Фев 2021

Rabb1t написал(а):
Что значит напрямую? Использовать ConVar? Так это обёртка для Handle, но только для использования именно кваров. Просто методмап, использующий всё те же функции.

Немного дичь, т.к. проблем не будет если ты лишний раз создать тот же квар.

То есть абсолютно без разницы каким методом действовать?
И нужно ли приравнивать к INVALID_HANDLE и null ?

Rabb1t · 12 Фев 2021

Nekro написал(а):
То есть абсолютно без разницы каким методом действовать?

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

Nekro написал(а):
И нужно ли приравнивать к INVALID_HANDLE и null ?

Опять же, на сколько мне известно, они по умолчанию имеют null. Про разницу INVALID_HANDLE и null, если честно сейчас не вспомню. По сути одно и тоже ВРОДЕ, просто первый формат исключительно для хэндлов (возможно ещё и дочерние классы), а null может ещё где-то использоваться.
В общем, тут я могу лишь ввести в заблуждение, поэтому пусть кто-то знающий наверняка даст ответ.

rejchev · 12 Фев 2021

ConVar, Database и прочие подобные наследуемые типы данных - попытка разработчиков прийти к ООП. Отсюда и INVALID_HANDLE == null
Все переменные, при объявлении, инициализируются со стандартным, для типа данных, значением.

Nekro · 12 Фев 2021

Rabb1t написал(а):
На сколько мне известно - да. Просто один старый формат, другой новый. Новый - методмат, использующий старые ф-и. В общем, тут как удобнее и кому как нравится.

Опять же, на сколько мне известно, они по умолчанию имеют null. Про разницу INVALID_HANDLE и null, если честно сейчас не вспомню. По сути одно и тоже ВРОДЕ, просто первый формат исключительно для хэндлов (возможно ещё и дочерние классы), а null может ещё где-то использоваться.
В общем, тут я могу лишь ввести в заблуждение, поэтому пусть кто-то знающий наверняка даст ответ.

Т.е. через

C-подобный:

ConVar cvar;
cvar = CreateConVar("sm_jumpeffect_enable", "1", "Включить/выключить эффект волн при прыжке.", _, true, _, true, 1.0);
cvar.AddChangeHook(CVarChanged_Enable);
bEnable = cvar.BoolValue;

Или через Cvar или Handle на ситуацию вообще никак не влияет?

Rabb1t · 12 Фев 2021

Nekro написал(а):
Т.е. через

C-подобный:

ConVar cvar; cvar = CreateConVar("sm_jumpeffect_enable", "1", "Включить/выключить эффект волн при прыжке.", _, true, _, true, 1.0); cvar.AddChangeHook(CVarChanged_Enable); bEnable = cvar.BoolValue;

Или через Cvar или Handle на ситуацию вообще никак не влияет?

Если использовать Handle, то тебе не будет доступны медоты, проперти и тд из методмапов, например: ConVar.AddChangeHook().
По факту этот метод вызывает эту ф-ю.

Nekro · 12 Фев 2021

Rabb1t написал(а):
Если использовать Handle, то тебе не будет доступны медоты, проперти и тд из методмапов, например: ConVar.AddChangeHook().
По факту этот метод вызывает эту ф-ю.

Обновил шапку. Посмотри пожалуйста.

Grey83 · 12 Фев 2021

При объявлении любого типа переменной она автоматически приравнивается к нулю (для INVALID_HANDLE прописан дефайн равный 0, кстати), если явно не указано иное.

Ты интересуешься о том как лучше работать с кварами: хранить значение в отдельной переменной и изменять её значение при изменении значения соответствующего квара или же каждый раз запрашивать текущее значение квара?

Сообщения автоматически склеены: 12 Фев 2021

Nekro написал(а):
Обновил шапку

Во втором примере ты не отслеживаешь изменения значения квара. Квары не имеют смысла в таком случае.

Nekro · 12 Фев 2021

Grey83 написал(а):
При объявлении любого типа переменной она автоматически приравнивается к нулю (для INVALID_HANDLE прописан дефайн равный 0, кстати), если явно не указано иное.

Ты интересуешься о том как лучше работать с кварами: хранить значение в отдельной переменной и изменять её значение при изменении значения соответствующего квара или же каждый раз запрашивать текущее значение квара?

Сообщения автоматически склеены: 12 Фев 2021

Во втором примере ты не отслеживаешь изменения значения квара. Квары не имеют смысла в таком случае.

Да, хочу понять какой из методов мне использовать.

К стати зачем тогда все постоянно пишут Handle g_hEnable = INVALID_HANDLE; ? Или так было в старом синтаксисе? Вроде встречаю и в новом это.

Grey83 · 12 Фев 2021

ИМХО, если значение квара будет запрашиваться редко, то можно использовать и 1-й вариант (там выигрыш всё равно будет мизерным), а если часто, то лучше второй.
Исследования не проводил, но мне кажется, что получать значение квара выйдет дороже, чем значение переменной.

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

Пример:

int iColor;
...
    cvar = CreateConVar("sm_*_color", "f0f", "Default * color. Set by HEX (RGB or RRGGBB, values 0 - F or 00 - FF, resp.). Wrong color code = magenta", FCVAR_PRINTABLEONLY);
...
{
    char clr[8];
    cvar.GetString(clr, sizeof(clr));
    clr[7] = 0;

    int i;
    while(clr[i])
    {
        if(!(clr[i] >= '0' && clr[i] <= '9') && !(clr[i] >= 'A' && clr[i] <= 'F') && !(clr[i] >= 'a' && clr[i] <= 'f'))
        {
            iColor = 0xFF00FF;
            return;
        }
        i++;
    }

    clr[6] = 0;
    if(i == 3)
    {
        clr[4] = clr[5] = clr[2];
        clr[2] = clr[3] = clr[1];
        clr[1] = clr[0];
        i = 6;
    }

    if(i != 6) iColor = 0xFF00FF;
    else StringToIntEx(clr, iColor, 16);
}

Тогда только 3-й вариант.

Сообщения автоматически склеены: 12 Фев 2021

Nekro написал(а):
Кстати зачем тогда все постоянно пишут Handle g_hEnable = INVALID_HANDLE; ?

Привычка или карго-культ, наверное.
В SM1.6.3 точно было приравнивание к нулю этого типа, в более ранних версиях - хз, но и там могло так быть.

babka68 · 12 Фев 2021

Nekro написал(а):
К стати зачем тогда все постоянно пишут Handle g_hEnable = INVALID_HANDLE; ? Или так было в старом синтаксисе? Вроде встречаю и в новом это.

В старом INVALID_HANDLE
а в новом будет null вроде так

Nekro · 12 Фев 2021

Grey83 написал(а):
значение квара выйдет дороже, чем значение переменной.

В смысле g_cEnablePlugin и bEnable ?

Сообщения автоматически склеены: 12 Фев 2021

babka68 написал(а):
В старом INVALID_HANDLE
а в новом будет null вроде так

А какой смысл, если они изначально и так 0?

Grey83 · 12 Фев 2021

@Nekro, да

Сообщения автоматически склеены: 12 Фев 2021

@babka68, ну если тип указан через пробел, а не двоеточие, то это уже новый синтаксис.

babka68 · 12 Фев 2021

Вот тут нашёл в вики SourcePawn Transitional Syntax - AlliedModders Wiki

null - новое общее ключевое слово для замены INVALID_HANDLE.

Dragokas · 3 Мар 2021

Nekro написал(а):
А какой смысл, если они изначально и так 0?

Вероятно, привычка от C++, где переменным при инициализации не присваивается значение автоматически.
А еще на старом синтаксисе было такое ключевое слово "decl", которое если использовать с массивами / строками без обнуления, то внутри окажется мусор.

Madness aka null138 · 3 Мар 2021

Dragokas написал(а):
Вероятно, привычка от C++, где переменным при инициализации не присваивается значение автоматически.
А еще на старом синтаксисе было такое ключевое слово "decl", которое если использовать с массивами / строками без обнуления, то внутри окажется мусор.

или же просто повторяют за теми, которые делают по привычке, не зная для чего оно.

Как лучше объявить переменную?

Nekro

Терра инкогнита

Rabb1t

Амбассадор

Nekro

Терра инкогнита

Rabb1t

Амбассадор

Nekro

Терра инкогнита

Rabb1t

Амбассадор

rejchev

менеджер клоунов

Nekro

Терра инкогнита

Rabb1t

Амбассадор

Nekro

Терра инкогнита

Grey83

не пишу плагины с весны 2022

Nekro

Терра инкогнита

Grey83

не пишу плагины с весны 2022

babka68

Участник

Nekro

Терра инкогнита

Grey83

не пишу плагины с весны 2022

babka68

Участник

Dragokas

Добрая душа

Madness aka null138

Участник

Похожие темы