[knox]

Название:
auto_new_line

Описание:
Проставляет \n по возможности на ближайшем пробеле.

Аргументы:

• string - Массив со строкой в которой будет проставляться переходы
  
• lenght_line - Количество символов в одной строке, по умолчанию 70
  
• string_size - Размер строки, по умолчанию количество равно sizeof string

Код:

Код [Показать]

stock auto_new_line(string[], const lenght_line = 70, const string_size = sizeof string)
{
    new
        point = 0,
        len = strlen(string),
        check = 0;

    while(point < len - lenght_line)
    {
        for(new j = point; j < point + lenght_line; ++j)
        {
            if(string[j] == '\n')
            {
                check = 1;
                point = j + 1;
                break;
            }
        }
        if(check == 0)
        {
            for(new j = point + lenght_line; j >= 0; --j)
            {
                if(string[j] == ' ')
                {
                    string[j] = '\n';
                    check = 1;
                    point = j;
                    break;
                }
            }
        }
        if(check == 0)
        {
            strins(string, "\n", point + lenght_line, string_size);
            point = point + lenght_line + 1;
        }
        check = 0;
    }
    return 1;
}

Пример:

Пример [Показать]

public OnPlayerSpawn(playerid)
{
    new string[450] = "\
        Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod \
        tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, \
        quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. \
        Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu \
        fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in \
        culpa qui officia deserunt mollit anim id est laborum.\
    ";
    auto_new_line(string, 70);
    ShowPlayerDialog(playerid, 0, DIALOG_STYLE_MSGBOX, "Lorem ipsum", string, "Ok", "");
    return 0;
}

Скрин. [Показать]

https://pasteboard.co/I7vZkLz.png

Автор: knox

Сообщение отредактировал Barty Knox: 28 марта 2019 - 16:32

[NAVIGANTE]

knoX - лучший

[Romzes]

Полезно.

[knox]

NAVIGANTE (28 марта 2019 - 18:38) писал:

knoX - лучший

Да я такой.

Удивлен что не нашлось ни одного человека который не написал: "цикл в цикле гавнокодер"

[Sharkwd]

Barty Knox (29 марта 2019 - 21:38) писал:

Да я такой.

Удивлен что не нашлось ни одного человека который не написал: "цикл в цикле гавнокодер"

цикл в цикле гавнокодер

А так плюс, пригодится.

[Pa4enka]

Barty Knox (29 марта 2019 - 21:38) писал:

Удивлен что не нашлось ни одного человека который не написал: "цикл в цикле гавнокодер"

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

Сообщение отредактировал Pa4enka: 30 марта 2019 - 00:39

[knox]

Pa4enka (30 марта 2019 - 00:33) писал:

К тому же, на параллельном портале принципе описали недостатки этого алгоритма, которые вы даже не учли. А потом ищите похвалы на иных форумах. Забавная ситуация.

Я учел, но как там было сказано, возможно улучшить код, я пока размышляю об этом.

[knox]

Я тут решил попробовать по новому сделать, и вот до чего додумался:

stock InsertLineBreak(string[], const lenght_line = 70, const string_size = sizeof string)
{
	new
		line_start_pos = 0,
		line_end_pos = lenght_line,
		shift_pos = 0,
		count = 0;

	while(lenght_line < string_size)
	{
		if((line_end_pos - shift_pos) < 0)
		{
			strins(string, "\n", line_end_pos, string_size);

			line_start_pos = line_end_pos + 1;
			line_end_pos = line_start_pos + lenght_line;

			if(line_end_pos >= string_size)
				break;

			shift_pos = 0;
			count++;
		}
		else if(string[line_end_pos - shift_pos] == '\0')
		{
			break;
		}
		else if(string[line_end_pos - shift_pos] == '\n')
		{
			line_start_pos = line_end_pos - shift_pos + 1;
			line_end_pos = line_start_pos + lenght_line;
			
			if(line_end_pos >= string_size)
				break;

			shift_pos = 0;
			count++;
		}
		else if(string[line_end_pos - shift_pos] == ' ')
		{
			string[line_end_pos - shift_pos] = '\n';

			line_start_pos = line_end_pos - shift_pos + 1;
			line_end_pos = line_start_pos + lenght_line;

			if(line_end_pos >= string_size)
				break;

			shift_pos = 0;
			count++;
		}
		else
			shift_pos++;
	}
	return count;
}

[DeimoS]

1) Не очень понимаю зачем в такой функции искать символ переноса. Если строка уже отформатирована и в ней расставлены свои переносы, то для чего может понадобиться использовать ещё и данную функцию, с учётом того, что те самые символы переноса обычно расставляются не просто так?

2) Как минимум, условия с использованием "string[line_end_pos - shift_pos]" можно заменить на switch, дабы лишний раз не обращаться к массиву, а значение выражения "line_end_pos - shift_pos" можно просчитывать единожды в самом начале каждой итерации.

3) Какой смысл от условия "lenght_line < string_size" в шапке цикла, если ни первая, ни вторая переменные не изменяют своих значений по ходу работы цикла?

4) Какой смысл от поиска нуль-символа в твоей строке, если ты перебираешь ячейки от конца к началу?

Ну и напоследок хочется спросить: а ты вообще тестировал свой код? Ибо даже беглым вглядом я не особо вижу то, как у тебя должен происходить переход к новым ячейкам в случае, если ты нашёл пробел и заменил его на символ переноса. Текущий твой код либо в бесконечный цикл нырнёт, если значение "lenght_line" будет достаточно маленьким, либо у тебя strins просто вылезет за пределы массива и ты утечку памяти словишь. Ты не учитываешь то, что фактический размер строки может быть гораздо меньше размера массива, который передаётся в функцию. А если всё же размер массива совпадает с длинной строки, то ты не учитываешь, что strins не обращает внимание на третий свой параметр и записывает символ в строку даже если места для этого символа особо-то и нет.
В общем, крайне странный код, который непонятно как должен вообще работать.


UPD: Вообще за тебя уже всё давно придумали и продумали - zmessage.inc
Только тут идёт разделение сообщений и отправка их через SendClientMessage, а в твоём случае достаточно просто символ переноса добавлять.

Сообщение отредактировал DeimoS: 30 апреля 2021 - 08:18

[knox]

DeimoS (30 апреля 2021 - 08:11) писал:

1) Не очень понимаю зачем в такой функции искать символ переноса. Если строка уже отформатирована и в ней расставлены свои переносы, то для чего может понадобиться использовать ещё и данную функцию, с учётом того, что те самые символы переноса обычно расставляются не просто так?

Я хочу в БД хранить текст, там есть разделение по абзацам, но нет адаптации под диалоги(будет не очень красиво если будет слишком широкое окно).

DeimoS (30 апреля 2021 - 08:11) писал:

3) Какой смысл от условия "lenght_line < string_size" в шапке цикла, если ни первая, ни вторая переменные не изменяют своих значений по ходу работы цикла?

Я хотел цикл что будет идти пока мне нужно, ну и нужна была проверка, на то чтобы длина массива была больше, чем длина линии строки, вот и объединил две вещи под одним одеялом.

DeimoS (30 апреля 2021 - 08:11) писал:

4) Какой смысл от поиска нуль-символа в твоей строке, если ты перебираешь ячейки от конца к началу?

Чтобы лишний раз не бегать по массиву. Конечно можно было бы длину строки получить или же отдельным циклом его найти.

DeimoS (30 апреля 2021 - 08:11) писал:

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

Видимо магия происходит, я не знаю даже.

DeimoS (30 апреля 2021 - 08:11) писал:

Текущий твой код либо в бесконечный цикл нырнёт, если значение "lenght_line" будет достаточно маленьким, либо у тебя strins просто вылезет за пределы массива и ты утечку памяти словишь. Ты не учитываешь то, что фактический размер строки может быть гораздо меньше размера массива, который передаётся в функцию. А если всё же размер массива совпадает с длинной строки, то ты не учитываешь, что strins не обращает внимание на третий свой параметр и записывает символ в строку даже если места для этого символа особо-то и нет.
В общем, крайне странный код, который непонятно как должен вообще работать.

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

DeimoS (30 апреля 2021 - 08:11) писал:

UPD: Вообще за тебя уже всё давно придумали и продумали - zmessage.inc
Только тут идёт разделение сообщений и отправка их через SendClientMessage, а в твоём случае достаточно просто символ переноса добавлять.

Благодарю.

UPD: да, на счёт strins был прав

Сообщение отредактировал knox: 30 апреля 2021 - 16:01