Форматирование данных

Приветствую! Хотелось уточнить, как лучше обрабатывать данные перед и в момент форматирования. Рационально ли привязывать данные к отдельной переменной, массиву или константе(наблюдал подобное в гейммодах и стандартных скриптах сампа от самп тим). Приведу пример для понимания:

Код:

---

stock MyFunc(playerid)
{
        new MFCaption[] = "Caption";
        new MFInfo[] = "Your nick: %s";
        new MFButton1[] = "Button1";
        new MFButton2[] = "Button2";
        new MFEdit[sizeof(MFInfo)+(MAX_PLAYER_NAME-2)];
        GetPlayerName(playerid, MFEdit, sizeof(MFEdit));
        format(MFEdit, sizeof(MFEdit), MFInfo, MFEdit);
        ShowPlayerDialog(playerid, 0, 0, MFCaption, MFEdit, MFButton1, MFButton2);
        return 1;
}

---

P.S.: Код накидал с помощью телефона за несколько минут, он не оптимизирован и необходим чисто для примера. Хотелось бы узнать мнение Daniel_Cortez и DeimoS.

Цитата:

---

Сообщение от Wolfgang

Приветствую! Хотелось уточнить, как лучше обрабатывать данные перед и в момент форматирования. Рационально ли привязывать данные к отдельной переменной, массиву или константе(наблюдал подобное в гейммодах и стандартных скриптах сампа от самп тим). Приведу пример для понимания:

Код:

---

stock MyFunc(playerid)
{
        new MFCaption[] = "Caption";
        new MFInfo[] = "Your nick: %s";
        new MFButton1[] = "Button1";
        new MFButton2[] = "Button2";
        new MFEdit[sizeof(MFInfo)+(MAX_PLAYER_NAME-2)];
        GetPlayerName(playerid, MFEdit, sizeof(MFEdit));
        format(MFEdit, sizeof(MFEdit), MFInfo, MFEdit);
        ShowPlayerDialog(playerid, 0, 0, MFCaption, MFEdit, MFButton1, MFButton2);
        return 1;
}

---

P.S.: Код накидал с помощью телефона за несколько минут, он не оптимизирован и необходим чисто для примера. Хотелось бы узнать мнение Daniel_Cortez и DeimoS.

---

Обычный текст можно можно без записи в массив/константу. Форматированный желательно в переменную(click)

Если же текста довольно много, рекомендуется создать массив(имя фракций тому пример).

https://i.imgur.com/BMqhIl6.png
Создай отдельно массив для ника и текста, а то в массив MFEdit изначально записывается имя игрока, потом же ты пытаешься записать в тот же массив иные данные. Если попытаешься отпринтить, то должно вывести нечто подобное: получаемыйник + Your Nick: получаемый ник

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

Гораздо удобнее и эффективнее начать использовать тот же Sublime Text 3, в котором, при выделении текста, показывается количество символов в нём. Собственно, после этого просто просматриваешь строку и прямо на ходу отнимаешь от того значения, которое выдал редактор, все спецификаторы и спец-символы, получая длину строки без спецификаторов. А уже дальше так же составляешь формулу, вписывая и прибавляя значение для каждого спецификатора. Что-то по типу такого:

https://i.imgur.com/zppgFRc.png
Так как в конце строки стоит спецификатор - его можно не выделять, чтоб сразу не учитывать его при подсчёте. В итоге, отнимаем оставшиеся 3 спецификатора (51-6 = 45) и записываем это значение в размер массива.

Далее начинаем составлять формулу:
1) ID игрока не может быть длиннее 999, так что можно указать цифру "3", но лучше для этого значения создать макрос/константу, а-ля

const MAX_PLAYERS_LEN = 3;

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

2) Для ника игрока есть макрос MAX_PLAYER_NAME

3) Для уровня игрока можно указать значение "11", так как 11 - это то количество символов, которое может быть в минимально возможном значении, записанном в 4 байта: -2147483648.

4) Для скина можно так же указать "3", так как максимальный ID скина - 311. Но и тут лучше обзавестись макросом/константой, как и в случае с ID игрока.

5) Осталось ко всей этой формуле прибавить единицу, дабы в строке было место для хранения нуль-символа.

В итоге получится такой код:

new string[45+MAX_PLAYERS_LEN+11+MAX_SKIN_LEN+1];
format(string, sizeof(string), "Ник игрока с ID %d - %s. Уровень игрока: %d. Скин: %d", playerid, player_name, player_level, player_skin);
SendClientMessage(playerid, -1, string);

То бишь, на выходе получается та же самая формула (при составлении которой ты делаешь то же самое, что и в варианте Daniel Cortez), только, при этом, у тебя не будет лишней строки в коде. Ну и в комментариях к той статье (на которые я оставлял ссылку выше) я описывал ситуацию, когда, чем строка длиннее, тем сложнее будет изменять формулу для ситуаций с использованием варианта от Daniel Cortez.

DeimoS конечно времени потратил на пост)
Статья на данную тему есть здесь: https://pro-pawn.ru/showthread.php?1...BE%D0%BA%D0%B8

Цитата:

---

Сообщение от punkochel

DeimoS конечно времени потратил на пост)
Статья на данную тему есть здесь: https://pro-pawn.ru/showthread.php?1...BE%D0%BA%D0%B8

---

Эмм, я, если что, на эту статью и ссылаюсь, а так же объясняю почему я считаю, что использовать такой метод не стоит. Стоило сначала прочитать что я писал, прежде чем отвечать :)

А что думаешь насчёт подобного хранения данных(текста)?:

Код:

---

new MFCaption[] = "Caption";
new MFButton1[] = "Button1";
new MFButton2[] = "Button2";
ShowPlayerDialog(playerid, 0, 0, MFCaption, MFEdit, MFButton1, MFButton2);

---

Конечно можно всё хранить в массиве, который взаимодействует с Enum-ератором, но тогда должно быть несколько диалогов для продуктивности подобного массива со связкой или как вариант можно использовать дефайны/константы в этом варианте кода. Подобную конструкцию с привязыванием данных к переменным(видимо для лучшей читабельности и редактирования кода) наблюдал в официальных скриптах от самп тим, по этому интересно узнать мнение и об этом.

Цитата:

---

Сообщение от Wolfgang

А что думаешь насчёт подобного хранения данных(текста)?:

Код:

---

new MFCaption[] = "Caption";
new MFButton1[] = "Button1";
new MFButton2[] = "Button2";
ShowPlayerDialog(playerid, 0, 0, MFCaption, MFEdit, MFButton1, MFButton2);

---

Конечно можно всё хранить в массиве, который взаимодействует с Enum-ератором, но тогда должно быть несколько диалогов для продуктивности подобного массива со связкой или как вариант можно использовать дефайны/константы в этом варианте кода. Подобную конструкцию с привязыванием данных к переменным(видимо для лучшей читабельности и редактирования кода) наблюдал в официальных скриптах от самп тим, по этому интересно узнать мнение и об этом.

---

Зачем выделять в стеке 3 массива, когда можно передать их как аргументы нативной функции?

Цитата:

---

Сообщение от SteveStage

Зачем выделять в стеке 3 массива, когда можно передать их как аргументы нативной функции?

---

Память под эти самые аргументы из не откуда берётся?

Цитата:

---

Сообщение от Wolfgang

А что думаешь насчёт подобного хранения данных(текста)?:

Код:

---

new MFCaption[] = "Caption";
new MFButton1[] = "Button1";
new MFButton2[] = "Button2";
ShowPlayerDialog(playerid, 0, 0, MFCaption, MFEdit, MFButton1, MFButton2);

---

Конечно можно всё хранить в массиве, который взаимодействует с Enum-ератором, но тогда должно быть несколько диалогов для продуктивности подобного массива со связкой или как вариант можно использовать дефайны/константы в этом варианте кода. Подобную конструкцию с привязыванием данных к переменным(видимо для лучшей читабельности и редактирования кода) наблюдал в официальных скриптах от самп тим, по этому интересно узнать мнение и об этом.

---

А зачем оно надо? Твой текст, который ты собираешься записывать в локальные переменные, всё равно будет хранится в глобальном сегменте данных. То бишь, ты, по сути, сначала записываешь текст в сегмент данных (при старте сервера), а при вызове этого кода будешь копировать текст в стек. Если и делать подобное, то только так:

static const MFCaption[] = "Caption";
static const MFButton1[] = "Button1";
static const MFButton2[] = "Button2";
ShowPlayerDialog(playerid, 0, DIALOG_STYLE_MSGBOX, MFCaption, MFEdit, MFButton1, MFButton2);

Создавая не локальные переменные, а глобальные. Избавившись тем самым от лишнего копирования данных из глобального сегмента данных в стек.

Хотя лично я в этом не вижу ничего удобного. Наоборот это только захламляет код лишними строками и усложнит его редактирование, если ты вдруг решишь между ShowPlayerDialog и переменными вставить какой-либо ещё код.
Уж лучше просто переносами и табуляцией отделять параметры друг от друга:

ShowPlayerDialog(playerid, 0, DIALOG_STYLE_MSGBOX, "Caption", 
    "{FFFFFF}\
        Строка 1\n\
        Строка 2\n\
        \n\
        Строка 3\n\
    ", 
    "Button_1", "Button_2");

С нормальным текстовым редактором, который поддерживает адекватную подсветку синтаксиса, такой вариант оформления кода будет хорошо читаем и его будет удобно редактировать в будущем.

Цитата:

---

Сообщение от execution

Память под эти самые аргументы из не откуда берётся?

---

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