utils_string.cpp

[bylins]

1. Составить список функций, выполняющих чисто обработку строки или строк, и располагающихся не в utils_string.cpp
2. Если со списком все ок - перенести в utils_string.cpp. Если не ок - подкорректировать список вручную

[bylins]

Список строковых утилит вне utils_string.cpp

utils.cpp - чистые строковые функции (кандидаты на перенос)

Функция	Строка	Описание
str_dup	108	Дублирование C-строки через malloc
str_cmp (4 overloads)	151-208	Case-insensitive сравнение строк
strn_cmp (4 overloads)	231-290	Case-insensitive сравнение с лимитом длины
str_add	96	Конкатенация строк с аллокацией
replace_str	379	Замена паттернов в строке
skip_dots	721	Пропуск точек и пробелов
str_str	727	Case-insensitive поиск подстроки (аналог strstr)
kill_ems	754	Удаление \r из строк
cut_one_word	769	Извлечение первого алфавитного слова
isname	965	Поиск слова в списке имен
one_word	1032	Извлечение первого слова с поддержкой кавычек
IsValidEmail	631	Валидация формата email
strl_cpy	834	Safe strcpy с лимитом (OpenBSD-стиль)
is_head	138	Проверка валидности заголовка
ReadEndString	804	Пропуск остатка строки из ifstream
StringReplace	813	Замена символа на строку
koi_to_alt/win/utf8	-	Конвертация кодировок KOI8-R
utf8_to_koi	-	Конвертация UTF-8 в KOI8-R

utils.h - макросы и inline

Функция	Описание
LOWER / UPPER	Макросы приведения регистра символа
skip_spaces	Шаблон пропуска пробелов

interpreter.cpp - парсинг аргументов

Функция	Описание
one_argument (2 overloads)	Извлечение первого аргумента с пропуском fill words
any_one_arg (2 overloads)	Извлечение первого аргумента без пропуска
two_arguments	Извлечение двух аргументов
three_arguments	Извлечение трех аргументов
delete_doubledollar	Удаление дублированных $
is_number	Проверка что строка содержит только цифры

handler.cpp

Функция	Описание
fname	Извлечение первого слова из списка имен

Рекомендация по группировке

Можно разделить на две группы:

1. Чистые строковые утилиты (str_cmp, strn_cmp, str_str, kill_ems, skip_dots, cut_one_word, replace_str, StringReplace, strl_cpy, LOWER/UPPER, skip_spaces, is_number, delete_doubledollar) -> utils_string.cpp
2. MUD-специфичные (isname, fname, one_argument, any_one_arg, two/three_arguments, one_word, IsValidEmail, is_head, ReadEndString, кодировки) -> могут остаться или переехать в отдельный модуль

[bylins]

Статус

Попытка автоматического переноса оказалась слишком грязной - функции сильно переплетены зависимостями (fill_word, half_chop, search_block, a_lcc/a_ucc и т.д.). Нужен аккуратный ручной перенос по одной функции за раз с проверкой зависимостей.

Откат сделан, ветка issue.3007 чистая. Продолжим в отдельной сессии.

[bylins]

Ну вот я говорил. Давай сразу под ИИ сформулирую.

Просмотреть функции строковых утилит в utils/strings и utils/mud_strings и их использование. Принять решение о целесообразности переработки этих утилит на perfect forwarding. Если решение положительное - переработать реализации функций с использованием perfect forwarding.

[bylins]

Анализ perfect forwarding для utils_string и mud_string

Функции с дублированными char*/string перегрузками

utils_string.h (существующие):

Функция	Перегрузки	Perfect forwarding?
RemoveColors	char*, std::string	Нет - разная логика (char* модифицирует in-place)
ConvertToLow	char*, std::string&	Нет - char* через указатель, string через ссылку
TrimLeft	char*, std::string&	Нет - разные реализации
TrimRight	char*, std::string&	Нет - разные реализации
Trim	char*, std::string&	Нет - разные реализации
colorCAP	char*, string&, string&&	Да - кандидат
colorLOW	char*, string&, string&&	Да - кандидат
CAP	char*, string	Нет - разный return type
IsAbbr	const char* + inline string обертки	Нет - inline достаточно
str_cmp	4 перегрузки char*/string	Нет - разная логика итерации
strn_cmp	4 перегрузки char*/string	Нет - разная логика итерации

utils_string.h (добавленные в #3008):

Функция	Перегрузки	Perfect forwarding?
PruneCrlf	char*, string&	Нет - разные реализации
first_letter	const char*, string	Нет - разный return type
is_number	const char*, string	Возможно - string версия просто вызывает c_str()
delete_doubledollar	char*, string&	Нет - разные реализации
skip_dots	char**, string&	Нет - разные типы параметров
str_str	char*, string	Нет - разный return type
kill_ems	char*, string&	Нет - разные реализации
IsValidEmail	const char*, string	Возможно - string версия просто вызывает c_str()
isname	const char* + inline обертки	Нет - inline достаточно

mud_string.h:

Функция	Перегрузки	Perfect forwarding?
one_argument	char*, const char*	Нет - шаблон уже используется (one_argument_template)
any_one_arg	char*, const char*	Нет - шаблон уже используется
two_arguments	шаблон T	Уже perfect forwarding по сути
three_arguments	шаблон T	Уже perfect forwarding по сути

Вывод

Perfect forwarding нецелесообразен для большинства функций потому что:

1. char* и std::string версии имеют разную реализацию (in-place vs erase/pop_back) - нельзя свести к одному шаблону
2. Разные return types (char* vs std::string, const char* vs size_type) - шаблон не может вернуть разные типы
3. Функции принимающие только const char* с string оберткой через .c_str() - inline обертка проще и понятнее чем шаблон с SFINAE/concepts
4. two_arguments/three_arguments уже шаблонные
5. str_cmp/strn_cmp имеют 4 перегрузки с разной логикой итерации для каждой комбинации

Два кандидата где perfect forwarding мог бы помочь:

• colorCAP / colorLOW - имеют string& и string&& версии, можно свести к одному шаблону
• Но выигрыш минимален (убрать 2 строки из хедера)

Рекомендация: оставить как есть. Текущие перегрузки явные и понятные.