Подсчёт размера форматируемой строки

[DeimoS]

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

Я изначально рассказал всем логику составления этого макроса. И про "неожиданное" я тебе уже говорил.

А на остальное уже не буду отвечать, ибо пора прекращать эту полемику, которая начинает перерастать в недопонимания и стрелометание. Как я уже сказал выше, я свою точку зрения по поводу этого вопроса озвучил. Ты озвучил свою. У каждого своя голова на плечах, так что пусть каждый сам и решает - готов ли он взять на себя ТАКУЮ "ответственность" и помнить СТОЛЬКО новой информации.
А за то, о чём ты просил не распространяться, прости :( Запамятовал

[Redsan]

Я правильно считаю?

new query_string[85-12+MAX_PLAYER_NAME+32+64];
format(query_string, sizeof(query_string), "INSERT INTO `accounts` (`player_name`, `password`, `email`) VALUES ('%s', '%s', '%s')", pInfo[playerid][pName], password, email);
// максимальная длина password - 32
// максимальная длина email - 64  

Или объясните на этом примере как высчитывались значения?

stock SaveAccount(playerid)
{
    new query_string[(21)+(16+11)+(20+MAX_PLAYER_NAME)+(16+32)] = "UPDATE `accounts` SET";

    format(query_string, sizeof(query_string), "%s `player_name` = '%s',", query_string, pInfo[playerid][pName]);
    format(query_string, sizeof(query_string), "%s `password` = '%s'", query_string, pInfo[playerid][pPassword]);

    format(query_string, sizeof(query_string), "%s WHERE `id` = '%d'", query_string, pInfo[playerid][pID]);
    mysql_function_query(mysql_connect_id, query_string, false, "", "");
    return 1;
}  

[m1n1vv]

UPDATE `accounts` SET //(21)
%s `player_name` = '%s', //(20+MAX_PLAYER_NAME)
%s `password` = '%s' //(16+32)
%s WHERE `id` = '%d' //(16+11)  

Например (16+11). Первое число это "длина" текста в кавычках, а второе - что выведут %s и %d.

Тот код можно оформить и так:

stock SaveAccount(playerid)
{
    static const str[] = "UPDATE `accounts` SET `player_name` = '%s', `password` = '%s' WHERE `id` = '%d'";
    new string[sizeof str+MAX_PLAYERS_NAME+32+11-2*3];
    format(string, sizeof string, str, 
        pInfo[playerid][pName], 
        pInfo[playerid][pPassword],
        pInfo[playerid][pID]
    );
    mysql_function_query(mysql_connect_id, string, false, "", "");
    return 1;
}  

[Daniel_Cortez]

Я правильно считаю?

new query_string[85-12+MAX_PLAYER_NAME+32+64];
format(query_string, sizeof(query_string), "INSERT INTO `accounts` (`player_name`, `password`, `email`) VALUES ('%s', '%s', '%s')", pInfo[playerid][pName], password, email);
// максимальная длина password - 32
// максимальная длина email - 64  

const MAX_PLAYER_PASSWORD = 32;
const MAX_PLAYER_EMAIL = 64;
enum pInfo
{
    pName[MAX_PLAYER_NAME + 1],
    pPassword[MAX_PLAYER_PASSWORD + 1],
    pEmail[MAX_PLAYER_EMAIL + 1],
    // ...
};  

new fmt_str[] = "INSERT INTO `accounts` (`player_name`, `password`, `email`) VALUES ('%s', '%s', '%s')";
new query[sizeof(fmt_str) + (-2+MAX_PLAYER_NAME) + (-2+MAX_PLAYER_PASSWORD) + (-2+MAX_PLAYER_EMAIL)];
format(query, sizeof(query), fmt_str, pInfo[playerid][pName], pInfo[playerid][pPassword], pInfo[playerid][pEmail]);
// А вообще здесь следовало бы использовать mysql_format. Только не спрашивайте, почему.  

[m1n1vv]

const MAX_PLAYER_PASSWORD = 32;
const MAX_PLAYER_EMAIL = 64;  

enum MAX_LENGHT
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
}  

Удобней в чтении кода

[Daniel_Cortez]

enum MAX_LENGHT
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
}  

Удобней в чтении кода

Удобнее в плане большего расхода пространства на экране или засорения глобального пространства имён лишним идентификатором MAX_LENGHT (не говоря уже об опечатке в названии)?
Вообще "enum" - это сокращение от "enumeration", что переводится как "перечисление". Оператор enum предназначен для перечисления констант, значения которых зависят друг от друга (я бы написал "следуют" друг за другом, но в Pawn для элементов перечисления можно сделать не только инкремент, но и декремент или даже битовый сдвиг).
В вашем же примере значения MAX_PLAYER_PASSWORD и MAX_PLAYER_EMAIL никак не зависят друг от друга, поэтому непонятно, в чём вообще смысл об их объявления в enum.
Да, это допустимый стиль с точки зрения корректности синтаксиса, но, ИМХО, чтобы считать его "удобным", нужно обладать довольно извращённым чувством удобства.
С таким же успехом можно объявлять константы так:

enum {MAX_PLAYER_PASSWORD = 32};
enum {MAX_PLAYER_EMAIL = 64};  

Но почему-то так никто не делает. По очевидным причинам.

[Nexius_Tailer]

Да, это допустимый стиль с точки зрения корректности синтаксиса, но, ИМХО, чтобы считать его "удобным", нужно обладать довольно извращённым чувством удобства.
С таким же успехом можно объявлять константы так:

enum {MAX_PLAYER_PASSWORD = 32}; 
enum {MAX_PLAYER_EMAIL = 64};  

Но почему-то так никто не делает. По очевидным причинам.

Почему это извращённым чувством удобства? К примеру имеем мы кучу дефайнов, большинство которых начинаются с "MAX_". Максимальным может быть не только размер, но и количество чего-либо. Поэтому довольно удобное и наглядное разграничивание.
Да и вообще, если уж говорить о том, чем кто пользуется, то это не const, а:

#define  

Тем не менее другие варианты тоже существуют и имеют место быть. Кому как удобнее?

[Daniel_Cortez]

Почему это извращённым чувством удобства? К примеру имеем мы кучу дефайнов, большинство которых начинаются с "MAX_". Максимальным может быть не только размер, но и количество чего-либо.

Потому что прочитайте полностью предыдущий пост.
MAX_PLAYER_PASSWORD и MAX_PLAYER_EMAIL как-то зависят друг от друга? Нет. Обе константы обозначают максимальные длины строк - и на этом всё. Можно спокойно изменить MAX_PLAYER_PASSWORD - на значении MAX_PLAYER_EMAIL это никак не отразится.

Поэтому довольно удобное и наглядное разграничивание.

const MAX_HOUSES = 300;

const MAX_PLAYER_PASSWORD = 32;
const MAX_PLAYER_EMAIL = 64;

new const Float:fishing_spots[][3] =
{
    // ...
};  

Пустые строки. Довольно древний лайфхак - и тоже весьма наглядное разграничивание.
Если же взять тот пример с enum - всё равно ж придётся ставить пустые строки до и после перечисления.

Попробую предугадать ответ:

А я не буду ставить пустые строки - с enum разграничение и так уже есть.

Хорошо, допустим:

const MAX_HOUSES = 300;
enum
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
};
new const Float:fishing_spots[][3] =  

Будь там полностью пустые строки, разделение было бы очевидным, но в таком виде границы перечисления становятся более "размытыми".

Да и вообще, если уж говорить о том, чем кто пользуется, то это не const, а:

#define  

Так я ничего и не говорил против #define, речь была конкретно о бессмысленности enum для не зависящих друг от друга констант.
На самом деле в модах куда чаще объявляют константы с помощью #define (вернее, это уже не константы, а макросы), но единственное их преимущество перед объявлением через const - возможность использования с оператором стрингизации:

#define MAX_AD_MESSAGE 80
// ...
CMD:ad(playerid, params[])
{
    // ...
    if (strlen(params) > 80)
        return SendClientMessage(playerid, -1, "ОШИБКА: Текст сообщения должен быть не длиннее " #MAX_AD_MESSAGE " символов.");
    // ...
}  

Если объявить MAX_AD_MESSAGE с помощью const, такой трюк не прокатит.

Тем не менее другие варианты тоже существуют и имеют место быть. Кому как удобнее?

Целью моего поста было лишь донести нецелесообразность использования конкретных вариантов, а не навязать использование только одного.
Стиль кодинга следует с самого начала хорошо продумывать - иначе вместо кода можно получить кучу едва работающих, прибитых друг к другу гвоздями костылей и заплаток, которую невозможно структурировать, не переписав всё с нуля. И пока найдёшь в этой куче то, что нужно (допустим, константу, название которой забыл), вспомнишь столько непристойных слов, что на целый словарь хватит.

[Nexius_Tailer]

Хорошо, допустим:

const MAX_HOUSES = 300;
enum
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
};
new const Float:fishing_spots[][3] =  

Будь там полностью пустые строки, разделение было бы очевидным, но в таком виде границы перечисления становятся более "размытыми".

Да нет, всё отлично

enum MAX_COUNT
{
    MAX_HOUSES = 300,
    MAX_BUSINESS = 250,
    MAX_FLATS = 100
}

enum MAX_LENGHT
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
}  

[Desulaid]

Да нет, всё отлично

enum MAX_COUNT
{
    MAX_HOUSES = 300,
    MAX_BUSINESS = 250,
    MAX_FLATS = 100
}

enum MAX_LENGHT
{
    MAX_PLAYER_PASSWORD = 32,
    MAX_PLAYER_EMAIL = 64
}  

Я тоже люблю мороженое с рыбой в одном рожке, ведь это все еда.

Не вижу смысла в использовании оператора enum для такого извращения, да и код выглядит как-то не очень красиво. И вообще КТО ТАК ДЕЛАЕТ?!