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

[Edward Morra]

Попробуй подсчёт в массиве выполнить до пропуска инициализации

например?

[vovandolg]

например?

new sizestr = sizeof(frm_engine)+(-2+MAX_PLAYER_NAME);
noinit:string[sizestr];  

[Desulaid]

Попробуй подсчёт в массиве выполнить до пропуска инициализации

Это не поможет проблеме. Вы ведь никакое значение массиву не устанавливаете во время инициализации.

main()
{
    static const
        fmt_str[] = "%s - это была строка...";

    goto noinit;
    new
        str[sizeof fmt_str + (-2 + 9)];
    noinit:
    
    format(str, sizeof str, fmt_str, "кака-бяка");
    print(str);
}  

Как тут можно вообще ждать помощи, если формулировка проблемы не ясна совсем. Что значит "после подсчетов по этому методу, стала некорректно работать команда". Что значит "некорректно"? Чего вы ожидали и что получили в итоге укажите.

[Edward Morra]

Это не поможет проблеме. Вы ведь никакое значение массиву не устанавливаете во время инициализации.

main()
{
    static const
        fmt_str[] = "%s - это была строка...";

    goto noinit;
    new
        str[sizeof fmt_str + (-2 + 9)];
    noinit:
    
    format(str, sizeof str, fmt_str, "кака-бяка");
    print(str);
}  

Как тут можно вообще ждать помощи, если формулировка проблемы не ясна совсем. Что значит "после подсчетов по этому методу, стала некорректно работать команда". Что значит "некорректно"? Чего вы ожидали и что получили в итоге укажите.

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

[VVWVV]

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

Это баг.

Для решения проблемы необходимо переименовать метку.

Однако существует другой способ:

#define @new%0\32%1[%2]%3; goto _noinit_%1; new %1[%2]%3; _noinit_%1:  

и используй так:

@new str[10], strr[20], strrr[30];  

Синтаксис лучше придумать не смог.

Однако такой синтаксис работать не будет, ибо у вас названия идентичны, а это значит, что вам необходимо переименовать какой-то из массивов.

[Edward Morra]

Это баг.

Вот именно про этот баг я твердил.
Почему не наткнулся я на эту тему...
Дак, каким оптимальным вариантом решения проблемы будет являться?

[ziggi]

Вот именно про этот баг я твердил.
Почему не наткнулся я на эту тему...
Дак, каким оптимальным вариантом решения проблемы будет являться?

Самым лучшим решением будет забить на пропуск инициализации и писать код нормально. Ибо ты совершенно ничего не понимаешь в оптимизации (куча вызовов GetPlayerVehicleID, оптимизируешь инициализацию массива ), а такие "оптимизации" только вредят. Ну или просто используй разные названия переменных.

P.S. Нахватаются от всяких, а потом не понимают где проблемы... Любые микрооптимизации можно производить только при знании того, как оно работает и для чего это делается и только после общей оптимизации кода и алгоритмов (а это оптимизировать куда сложнее).

[Edward Morra]

Самым лучшим решением будет забить на пропуск инициализации и писать код нормально. Ибо ты совершенно ничего не понимаешь в оптимизации (куча вызовов GetPlayerVehicleID, оптимизируешь инициализацию массива ), а такие "оптимизации" только вредят. Ну или просто используй разные названия переменных.

P.S. Нахватаются от всяких, а потом не понимают где проблемы... Любые микрооптимизации можно производить только при знании того, как оно работает и для чего это делается и только после общей оптимизации кода и алгоритмов (а это оптимизировать куда сложнее).

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

[vovandolg]

оптимизируешь инициализацию массива

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

[ziggi]

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

Не знаю, лично я этим не пользуюсь и никому не советую. Повторюсь: выигрыш будет минимальным, по сравнению с грамотной оптимизацией алгоритмов.

Мог бы конкретней указать на мою некомпитентность в данном коде?

Как минимум, все вызовы GetPlayerVehicleID, GetPlayerState, IsPlayerInAnyVehicle можно заменить одним GetPlayerVehicleID. Также вызов PVar::GetInt можно сделать единожды.
Код не оформлял (только лапшу убрал), ибо лень:

CMD:engine(playerid,params[])
{
    if(!IsPlayerLogged{playerid}) return SendMe(playerid,COLOR_GREY,!" * [A] - Сначала нужно авторизоваться!");

    new vehicleid = GetPlayerVehicleID(playerid);
    if(vehicleid == 0) {
        return 1;
    }

    if(IsVehicleBike(vehicleid)) return false;

    if(Engine[vehicleid]) {
        printf("незаводим");
        static const frm_engine[]=" * %s заглушил двигатель.";
        new string[sizeof(frm_engine)+(-2+MAX_PLAYER_NAME)];
        format(string,sizeof(string),frm_engine,PlayerName(playerid));
        ProxDetector(playerid,5,string,COLOR_PURPLE);
        SetEngineOnOff(vehicleid,false);
        EngineTime[playerid] = 0;
        return 1;
    }

    if(Fuel[vehicleid] <= 0.0) {
        return SendMe(playerid,COLOR_GREY,!" * Бензобак пуст. Вызовите механиков!");
    }

    if(EngineTime[playerid]) {
        return SendMe(playerid,COLOR_GREY,!" * Нельзя так часто заводить двигатель!");
    }
    new Float:vHealth;
    GetVehicleHealth(vehicleid,vHealth);
    new driving = PVar::GetInt(playerid,"Driving");
    if(OrgCarInfo[vehicleid][orgcFrac] == 15 && (driving == 4 || driving == 6))
    {
        if(!SeatBelt{playerid} && !IsVehicleA(vehicleid))
        {
            SendMe(playerid,COLOR_GREY,!" * Перед началом движения вы обязаны пристегнуть ремень безопасности. Вы провалили экзамен!");
            PVar::SetInt(playerid,"Driving",0);
            acc_RemovePlayerFromVehicle(playerid);
            DisablePlayerRaceCheckpoint(playerid);
            SetEngineOnOff(vehicleid,false);
            AccountInfo[playerid][LoadCharacters{playerid}][acTestAS] = 0;
            licCP{playerid} = 0;
            licerror{playerid} = 0;
            lictesttime{playerid} = 0;
            return false;
        }
    }
    printf("заводим");
    EngineTime[playerid] = 6;
    FreezePlayer(playerid);
    GameTextForPlayer(playerid, "~w~€AЊYCKAEM ѓ‹…‚A¦E‡’...", 5000, 4);
    static const frm_str[]=" * %s пытается запустить двигатель.";
    new string[sizeof(frm_str)+(-2+MAX_PLAYER_NAME)];
    format(string,sizeof(string),frm_str,PlayerName(playerid));
    return ProxDetector(playerid,5,string,COLOR_PURPLE);
}  

И такой ужас лишь в одной команде, страшно представить что во всём остальном скрипте творится. Уверен, что IsVehicleBike, IsVehicleA реализованы массой проверок, в ProxDetector вычисляется расстояние для всех игроков.

P.S. Уродствами, вида SendMe и PVar::SetInt ты только путаешь людей, которые будут разбираться в твоём коде.