warning 224: indeterminate array size in "sizeof" expression (symbol "")

[TheMallard]

source\themallards\utils.inc(23) : warning 224: indeterminate array size in "sizeof" expression (symbol "")

Собственно, ошибка заключается в функции логгирования.

stock Log(const filename[], const message[])
{
    static const fmt_str[] =
        "\r\n[%02d/%02d/%02d %02d:%02d:%02d] %s %s";
    new File:filehandle,
        string[sizeof(fmt_str) + sizeof(message)], // строка 23
        year,
        month,
        day,
        hour,
        minute,
        second;
    if(!(filehandle = fopen(filename, io_append)))
        return -1;
    getdate(year, month, day);
    gettime(hour, minute, second);
    format(string, sizeof(string), fmt_str,
        year, month, day, hour, minute, second, message);
    for(new idx = 0, length = strlen(string); idx < length; idx++)
        fputchar(filehandle, string[idx], false);
    return fclose(filehandle);
}  

[L0ndl3m]

stock Log(const filename[], const message[])
{
    static const fmt_str[] =
        "\r\n[%02d/%02d/%02d %02d:%02d:%02d] %s";
    new File:filehandle,
        string[sizeof(fmt_str) + (-4 * 6) + (-2 + 127)],
        // 127 - максимальная длина ввода текста
        year,
        month,
        day,
        hour,
        minute,
        second;

    if(!(filehandle = fopen(filename, io_append)))
        return -1;
    getdate(year, month, day);
    gettime(hour, minute, second);
    format(string, sizeof(string), fmt_str,
        year, month, day, hour, minute, second, message);
    for(new idx = 0, length = strlen(string); idx < length; idx++)
        fputchar(filehandle, string[idx], false);
    return fclose(filehandle);
}  

[VVWVV]

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

Можно было сделать так:

stock Log(const filename[], const message[128])
{
    static const fmt_str[] = "\r\n[%02d/%02d/%02d %02d:%02d:%02d] %s";
    new File:filehandle,
        string[sizeof fmt_str + (-4 * 6) + (-2 + sizeof message)],
        year,
        month,
        day,
        hour,
        minute,
        second;

    if(!(filehandle = fopen(filename, io_append)))
        return -1;
    getdate(year, month, day);
    gettime(hour, minute, second);
    format(string, sizeof(string), fmt_str,
        year, month, day, hour, minute, second, message);
    for(new idx = 0, length = strlen(string); idx < length; idx++)
        fputchar(filehandle, string[idx], false);
    return fclose(filehandle);
}  

Но если вам непринципиален размер строки, т.е. нет никаких ограничений, то можно сделать так:

stock Log(const filename[], const message[], size = sizeof message)
{
    static const fmt_str[] = "\r\n[%02d/%02d/%02d %02d:%02d:%02d] %s";
    new File:filehandle,
        string[sizeof fmt_str + (-4 * 6) + (-2 + size)],
        year,
        month,
        day,
        hour,
        minute,
        second;

    if(!(filehandle = fopen(filename, io_append)))
        return -1;
    getdate(year, month, day);
    gettime(hour, minute, second);
    format(string, sizeof(string), fmt_str,
        year, month, day, hour, minute, second, message);
    for(new idx = 0, length = strlen(string); idx < length; idx++)
        fputchar(filehandle, string[idx], false);
    return fclose(filehandle);
}  

[L0ndl3m]

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

Можно было сделать так:

// ...  

В таком случае следует выделять 128 символов, т.к. 127 я выделял по причине того, что sizeof автоматически добавляет единицу к конечному размеру строки.

[Daniel_Cortez]

Но если вам непринципиален размер строки, т.е. нет никаких ограничений, то можно сделать так:

stock Log(const filename[], const message[], size = sizeof message)
{
    static const fmt_str[] = "\r\n[%02d/%02d/%02d %02d:%02d:%02d] %s";
    new File:filehandle,
        string[sizeof fmt_str + (-4 * 6) + (-2 + size)],
        year,
        month,
        day,
        hour,
        minute,
        second;

    if(!(filehandle = fopen(filename, io_append)))
        return -1;
    getdate(year, month, day);
    gettime(hour, minute, second);
    format(string, sizeof(string), fmt_str,
        year, month, day, hour, minute, second, message);
    for(new idx = 0, length = strlen(string); idx < length; idx++)
        fputchar(filehandle, string[idx], false);
    return fclose(filehandle);
}  

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

[VVWVV]

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

Спасибо.