Что будет эффективней?

[Salvacore]

Не понял...

Чисто профессиональный юмор для близко знакомых с моим кодом.

[Daniel_Cortez]

Итак, подсчитаем:
1 вариант:
Часто начал замечать такую дебильную тенденцию (в частности Про-Павн) максимально утруждать свой код и сваливать всю свою лень на препроцессор.

CMD:freeze(playerid, params[])
{
    if(PInfo[playerid][pAdmin] < 1) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Вам недоступна данная команда.");
    extract params -> new player:getid; else return Message(playerid, COLOR_GREEN,"(Используйте): {669966}/freeze [ID Player]");//-4 байта из стека
    if(0 == IsPlayerConnected(getid)) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Игрока нет на сервере.");
    static const
        fmt_str1[] = "Администратор %s(%d) Вас заморозил.", // + 36*4=144 байта
        fmt_str2[] = "Вы заморозили игрока %s(%d)."; // + 28*4=112 байта
    const
        fmt1_size = sizeof(fmt_str1) - 2 + MAX_PLAYER_NAME - 2 + 3, // 36 - 2 + 24 - 2 + 3 = 59;
        fmt2_size = sizeof(fmt_str2) - 2 + MAX_PLAYER_NAME - 2 + 3; // 28 - 2 + 24 - 2 + 3 = 51;
    #if fmt1_size > fmt2_size
    const size = fmt1_size;
    #else
    const size = fmt2_size;
    #endif
    new msg[size];// + 59 * 4 = 236;
    format(msg, sizeof(msg), fmt_str1, Nick(playerid)), Message(getid, COLOR_RED, msg);
    format(msg, sizeof(msg), fmt_str2, Nick(getid)), Message(playerid, COLOR_RED, msg);
    return TogglePlayerControllable(getid, false);
} //Итого 144 + 112 + 144 = 462  

Чёрд, да если верить таким отзывам, из-за меня на Pro-Pawn одни проблемы.

Что ж, расставлю все точки над "i".
Ещё в прошлом году я придумал такой трюк с подсчётом размера строки, чтобы, как многие из вас уже догадались, уменьшить используемый серверов объём стекового пространства.

SimplePawn

Я думал, этой программой уже никто не пользуется.

Да, у топикстартера в 1-м варианте код выглядит раздутым, и учитывая скудные возможности препроцессора в компиляторе Pawn, вряд ли с этим можно что-то сделать.
Но в то же время такой код куда более универсален. Если нужно изменить содержимое форматной строки, придётся... всего лишь изменить это содержимое - остальную работу компилятор сделает сам, не нужно ничего пересчитывать.
IMO, именно таким и должен быть качественный код.

Если же полагаться на сторонние программы вроде SimplePawn, каждый раз придётся выискивать её, чтобы пересчитать всё заново.
По сути всё сводится к "хочешь поменять текст? живо ищи калькулятор! или SimplePawn!" -_-
Пожалуй, это и оказалось основной причиной прекращения дальнейшей разработки той программы.

[Rasta]

Daniel_Cortez, Да ема е. Я уже почти все команды переписал...

Да, у топикстартера в 1-м варианте код выглядит раздутым

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

[Rasta]

Никто не ответит? Просто остановился на половине, а дальше не знаю как именно писать. Буду благодарен.

[Salvacore]

:D
Тут твой выбор, есть 2 мнения, выбирай что ближе.(1)

[underwoker]

Я в свое время предпочел такой вариант. Как предложил ^_^:

CMD:freeze(playerid, params[])
{
    new strcmd[59];//Вместо потерянной памяти из сегмента данных используем стек, при этом теряем 0 байтов
    if(PInfo[playerid][pAdmin] < 1) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Вам недоступна данная команда.");
    else if(sscanf(params, "i", params[0])) return Message(playerid, COLOR_GREEN,"(Используйте): {669966}/freeze [ID Player]");
    else if(!IsPlayerConnected(params[0])) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Игрока нет на сервере.");
    format(strcmd, sizeof(strcmd), "Администратор %s(%d) Вас заморозил.", Nick(playerid));
    Message(params[0], COLOR_RED, strcmd);
    format(strcmd, sizeof(strcmd), "Вы заморозили игрока %s(%d).", Nick(params[0]));
    Message(playerid, COLOR_RED, strcmd);
    return TogglePlayerControllable(params[0], false);
}  

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

[Salvacore]

Я в свое время предпочел такой вариант. Как предложил ^_^:

CMD:freeze(playerid, params[])
{
    new strcmd[59];//Вместо потерянной памяти из сегмента данных используем стек, при этом теряем 0 байтов
    if(PInfo[playerid][pAdmin] < 1) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Вам недоступна данная команда.");
    else if(sscanf(params, "i", params[0])) return Message(playerid, COLOR_GREEN,"(Используйте): {669966}/freeze [ID Player]");
    else if(!IsPlayerConnected(params[0])) return Message(playerid, COLOR_RED,"(Ошибка): {FF6666}Игрока нет на сервере.");
    format(strcmd, sizeof(strcmd), "Администратор %s(%d) Вас заморозил.", Nick(playerid));
    Message(params[0], COLOR_RED, strcmd);
    format(strcmd, sizeof(strcmd), "Вы заморозили игрока %s(%d).", Nick(params[0]));
    Message(playerid, COLOR_RED, strcmd);
    return TogglePlayerControllable(params[0], false);
}  

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

Раз не погнался, два не погнался, и тут вдруг твой мод оказывается RLS'ом.

[Rasta]

Раз не погнался, два не погнался, и тут вдруг твой мод оказывается RLS'ом.

Подобного слогана придерживался.

Я бы предпочёл 1-й способ, так как от начала с ним работаю.
Суть не в этом.
Если 1-й способ будет считаться медленнее 2-ого, но качественным и универсальным, то считается ли он эффективным?
Или же всё в порядке, оба одинаковы и выбирать уже на свой вкус?
Я так точного ответа и не услышал. Можете просто мне ответить, что считается лучшим вариантом?

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

Так как, после 3-ого поста, я переписал практически половину своего кода.
Не очень-то увлекает меня переписывать код по-несколько раз. Просто, хочу точно закрепить у себя в голове что лучше и, в дальнейшем, придерживаться этих способов, естественно, до появления новых "продвинутых" :)

[Rasta]

Вопрос не решён.

[Salvacore]

А ты затести, какой вариант команды быстрее.