Написать современно

@Igor3D · Регистрация: 01.10.2012

Студворк — интернет-сервис помощи студентам

Добрый день

Есть такой фрагмент

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
const char * singleC = strstr(src, "//");
const char * multiC = strstr(src, "/*");
const char * eol = strstr(src, "\n");
 
const char * min = nullptr;
if (singleC) min = singleC;
if (multiC && (!min || (multiC < min))) min = multiC;
if (eol && (!min || (eol < min))) min = eol;
 
if (!min) return;
 
if (min == singleC) {
 ...
}
 
if (min == multiC) {
 ...
}
 
if (min == eol) {
 ...
}

Бросается в глаза что код примитивный (тупой, убогий, говнокод и.т.п. - как хотите). Кроме того, его можно существенно ускорить.

Хорошо, как написать грамотно, современно? И заодно универсально, чтобы работал не только с char*

Спасибо

@Folian · 13.11.2025, 02:11

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

В данной теме не ставилась задача разбора комментов, просто фрагмент кода, цель - найти ближайшую подстроку. На такую (под)задачку программист должен потратить минут 5, ну 10, это пишется "рукой" (не головой). При этом код не обязан быть "самым быстрым", но он должен быть "достойным", так чтобы никакой питекантроп не смог ускорить его "в разы". Очень важно (если не важнее всего) чтобы код был понятным и посторонний человек смог в нем быстро разобраться. Заметим что код стартового поста этим качеством обладает.

Texas sharpshooter fallacy

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

Если нужно "выжать все".

Многоядерные и многопоточные процессоры, добро пожаловать в нынешнюю современность.

@Igor3D · 13.11.2025, 23:00 **[ТС]**

С "универсальностью" как-то совсем молчок. Надо полагать, следует юзать std::basic_string_view с нужным типом. Но меня смущает что Qt предлагает свой QStringView, причем без всяких темплейтов, заточенный на двухбайтовый юникод (QString но не только).

Мнения об альтернативных Qt классах/контейнерах различны. Одни считают Qt "великими троллями" и все их решения непогрешимыми. Другие наоборот, активно поднимают хвост и переводят все на std::, мол, сейчас это круче, и Qt должно использоваться "только для UI". Но по-любому Qt пишут хорошие профессионалы к которым стоит прислушаться.

Так почему у них свой QStringView и что лучше юзать?

@Royal_X · 13.11.2025, 23:32

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

Так почему у них свой QStringView и что лучше юзать?

Зачем у них свои классы - это очевидно:

1) для глубокой интеграции с остальным Qt. Если планируется использовать Qt-методы, то все равно придется использовать Qt-строковые классы, либо делать лишние конвертирования (в случае, если все-таки выбрать строки из стандартной библиотеки С++).

2) для юникода "из коробки". Библиотечные строки при работе с юникодом, грубо говоря, это просто массивы байт, которые ничего не знают о кодировке. А нац. кодировки это просто отстой, есть корявые локали, которые постоянно глючат. В то же время, строки Qt отлично работают со всеми языками и они исторически изначально были заточены под юникод, а именно UTF-16. Кстати, WinAPI тоже предпочитает двухбайтный юникод. Для передачи по сети UTF-8, конечно, крут, и вообще он заслужил статус универсальной юникодной кодировки, но практика показывает, что азиатские иероглифы там занимают 3-4 байта, тогда в UTF-16 всего 2. Именно по этой причине, как винапи, так Qt, предпочитают 2-байтный юникод.

А вот какой класс Qt-строк выбрать для того или иного случая - об этом очень подробно написано в доку https://doc.qt.io/qt-6/string-processing.html

***
Вот только при чем здесь Qt - мне не ясно. Тема была о написании современного кода на чистых плюсах.

@sdf45 · 14.11.2025, 00:39

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

Мнения об альтернативных Qt классах/контейнерах различны. Одни считают Qt "великими троллями" и все их решения непогрешимыми. Другие наоборот, активно поднимают хвост и переводят все на std::, мол, сейчас это круче, и Qt должно использоваться "только для UI".

откуда это все взялось?

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

Вот только при чем здесь Qt - мне не ясно.

Мне вообще непонятно что хочет ТС... Qt, STL, си-стайл?

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

В данной теме не ставилась задача разбора комментов, просто фрагмент кода, цель - найти ближайшую подстроку. На такую (под)задачку программист должен потратить минут 5, ну 10, это пишется "рукой" (не головой).

Если под STL или Qt - там все уже есть, и универсальное в том числе - бери и пользуйся.

@Igor3D · 14.11.2025, 03:35 **[ТС]**

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

Мне вообще непонятно что хочет ТС... Qt, STL, си-стайл?

То что в названии темы и стартовом посте. Использование std::string_view никак не универсально, это глухой char *. Да и говорить что это "современно" неуместно без поддержки юникода.

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

Если под STL или Qt - там все уже есть, и универсальное в том числе - бери и пользуйся.

Где "там"? (и что "все есть"). Никакой универсальности в QStringView не видно, это не template. Какие выгоды я получу используя QStringView вместо std::basic_string_view?

@sdf45 · 14.11.2025, 13:41

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

Использование std::string_view никак не универсально, это глухой char *

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

цель - найти ближайшую подстроку

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

Где "там"? (и что "все есть")

std::string::find (люди выше дали примеры) , QString::indexOf (и еще куча всего, в том числе из std::algorithm)

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

Никакой универсальности в QStringView не видно

В QString, QStringView напихано столько функционала, что можно на марс лететь.

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

Какие выгоды я получу используя QStringView вместо std::basic_string_view?

Royal_X выше ответил (и про юникод тоже).

Не по теме:

По абстрактным рассуждениям я всё-вышел из чата, -вижу смысл только в решении конкретных, четко сформулированных задачь.

@sdf45 · 14.11.2025, 20:29

Кстати, хоть оно мне и не надо, -пошуршал по форуму, -получилось вывести кириллицу с файла utf-8, используя строки std::wstring(::iterator, _view), и std::wifstream. С такими настройками:

C++
1
2
3
4
5
    setlocale(LC_ALL, "");
    std::wifstream file("mainwindow.h");
    std::locale utf8_locale(std::locale(), new std::codecvt_utf8<wchar_t>);
    file.imbue(utf8_locale);
    std::wcout.imbue(utf8_locale);

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

без поддержки юникода

Так что поддержка юникода есть, я правильно понимаю?

@eva2326 · 14.11.2025, 21:18

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

std::locale utf8_locale(std::locale(), new std::codecvt_utf8<wchar_t>);

deprecated in C++17, removed in C++26

@sdf45 · 14.11.2025, 21:49

eva2326, хм, спасибо,- надо глянуть, чем заменили.

@Igor3D · 14.11.2025, 23:54 **[ТС]**

Не по теме:

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

По абстрактным рассуждениям я всё-вышел из чата, -вижу смысл только в решении конкретных, четко сформулированных задачь.

Это Ваше право, я в свои темы никого на аркане не тяну

В данном случае "универсальность" - простая, очевидная вещь. Вот вверху несколько примеров кода который использует std::string_view. А данные в QString. Как мне из них получить std::string_view? Перегонять юникод в std::string? Это коряво и ненадёжно. Если уж перегонять - то наоборот, в юникод, хотя тоже не блеск, так, workaround. Надо юзать др классы вместо std::string_view, какие? Необязательно std::, такого условия/ограничения не было. Наоборот, сравнение разных подходов (std:: и Qt) может оказаться интересным/полезным.

@sdf45 · 15.11.2025, 01:29

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

А данные в QString

внезапно...

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

Как мне из них получить std::string_view?

C++
1
2
3
4
5
6
7
    QString q_str="some";
    std::string_view std_str=q_str.toStdString();
    std::cout<<std_str<<"\n";
    
    q_str="нечто";
    std::wstring_view std_wstr=q_str.toStdWString();
    std::wcout<<std_wstr<<"\n";

Вот только зачем? И каким боком тогда const char* если QString?
Почему бы не работать с Qt типами? Зачем это всё?

@Igor3D · 15.11.2025, 12:48 **[ТС]**

Не по теме:

По ходу дела

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

C++
1
2
QString q_str="some";
std::string_view std_str=q_str.toStdString();

А что, и так можно? Разве "время жизни" возвращаемой std::string будет здесь автоматически продлено чтобы std_str ссылался на валидные данные?

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

И каким боком тогда const char* если QString?

Задача - заменить старый код на новый. Вполне возможно (даже скорее всего) новый код будет/должен использовать QString, хотя возможно и std::string и что-то еще. Поэтому требуется универсальность, см стартовый пост.

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

Почему бы не работать с Qt типами?

Ну так покажите как с ними работать

@Royal_X · 15.11.2025, 13:51

Igor3D, касательно продления жизни... мне когда-то нужно было перевести wxString, полученный из текстового поля, в const char* и передать в качестве аргумента в функцию. Причем, нужно было, чтобы текст рассматривался как UTF-8. Т.е. чтобы в зависимости от того или иного входного символа выделялось от 1 до 4 char.

Допустим, wstr это строка типа wxString. И прикол был в том, что в wxWidgets есть методы, которые быстро конвертируют в нужный тип

C++
1
const char* utf8str = wstr.utf8_str();

или

C++
1
const char* utf8str = wstr.mb_str(wxConvUTF8);

но вот когда я передавал utf8str в функцию, то хоть никаких ошибок не было, но программа работала некорректно. И проблема связана как раз со временем жизни. Хоть и мой объект wxString не выходил из области видимости и я явно не убивал, он все равно уничтожался автоматически. А соответственно, умирал и указатель utf8str.

Правильное решение будет так:

C++
1
2
wxScopedCharBuffer buffer = wstr.ToUTF8();
const char* utf8str = buffer.data();

и тут буфер будет жить столько, сколько нужно

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

@sdf45 · 15.11.2025, 14:37

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

время жизни

Насколько я понимаю, QStringView - это синоним константной ссылки, const QString & и живет, пока жива строка, на которую он ссылается, - а контроль за этим лежит на погроммисте. Хорошо использовать при передаче параметров в функцию, а для возврата значений - плохо и тд и тп.

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

Ну так покажите как с ними работать

Это абстрактный вопрос, что я должен тут ответить? Выше есть примеры с std::string::find - его можно переписать с использованием QString(View)::indexOf, один в один.

зы: извиняюсь, что не по теме - выше мне указали, что std::codecvt_utf8 deprecated, но я не понял, чем он заменяется в промежутке между с++17 и с++26?

@Igor3D · 15.11.2025, 16:36 **[ТС]**

Не по теме:

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

Насколько я понимаю, QStringView - это синоним константной ссылки, const QString & и живет, пока жива строка, на которую он ссылается, - а контроль за этим лежит на погроммисте

Так в том-то и дело что возвращаемая std::string быстро станет неживой

C++
1
2
QString q_str="some";
std::string_view std_str=q_str.toStdString();

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

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

Это абстрактный вопрос, что я должен тут ответить? Выше есть примеры с std::string::find - его можно переписать с использованием QString(View)::indexOf, один в один.

А почему не переписать? Это нетрудно, а читать полное решение всем легче чем разрывать тему. И впечатление что Вам все равно: string::find или string_view::find, QStringView или QString. А в данном случае разница принципиальная. Ну и программизд который (упорно) избегает приводить код всегда выглядит подозрительно

И рассуждаете как-то.. "А, так тут QString? Ну значит QStringView!". Да не знает человек (пишущий этот фрагмент) у кого какие строки. Это как std::vector, что в нем хранить? Ну вот, только что говорил "int", а теперь уже "double" - видите, задача поставлена нечетко, неконкретно

@sdf45 · 15.11.2025, 18:30

Название: 0.jpg
Просмотров: 140

Размер: 13.0 Кб

@DrOffset · 16.11.2025, 12:12

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

C++
1
2
 QString q_str="some";
 std::string_view std_str=q_str.toStdString();

Здесь у вас действительно спровоцировано UB.
Временный объект std::string уничтожится в конце выражения, а std_str будет содержать подвисший указатель.

@sdf45 · 16.11.2025, 12:38

DrOffset, я думал std::string_view ведет себя как константная ссылка и продлевает временный обьект.
А как тогда? так что-ли

C++
1
2
auto buf = q_str.toStdString();
std::string_view std_str= buf;

@DrOffset · 16.11.2025, 12:46

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

что std::codecvt_utf8 deprecated, но я не понял, чем он заменяется в промежутке между с++17 и с++26?

Ничем. Своей реализацией заменяется.

Причины - здесь:
https://www.open-std.org/jtc1/... 618r0.html

Добавлено через 3 минуты

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

я думал std::string_view ведет себя как константная ссылка и продлевает временный обьект.

Нет, не ведет. string_view, как и span - это как скальпели, быстро и эффективно, но в неумелых руках - опасно.

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

А как тогда? так что-ли

Да.

@sdf45 · 16.11.2025, 12:49

DrOffset, спасибо за разьяснение, буду знать на будущее.

Новые блоги и статьи Все статьи Все блоги /
Baragoz - DPS/HPS бот для WoW 3.3.5a, многоязычный AmbA 26.10.2026 Хотите ещё ботов? У меня есть один! А я пока отдохну от него. https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=12062&stc=1&d=1793007952 Инструкция внутри. Сразу отвечу: А в PvP с ним. . .	Особенности gif сгенерированных в Си russiannick 21.10.2026 Рассмотрим генерацию gif файлов, которые в Си имеют свои особенности. Gif представляет собой графический формат популярный в интернете. Посмотрим на палитру. Gif файл сгенерированный в си отличается. . .	FPS контроллер с анимированными Y-Bot и 3D-звуком miniaudio (Box3D, Recast, BehaviorTree.CPP, ozz-animation, ImGui, OpenGL и SDL3) 8Observer8 18.10.2026 Содержание блога \| English Version Полное мультиплатформенное руководство по стартовому проекту с FPS-контроллером игрока, анимированными роботами Y-Bot с 4-слойным блендингом, 3D-физикой Box3D,. . .	Локальный http сервер Argus19 17.10.2026 Локальный http сервер На работе, на двух локальных компьютерах круглосуточно работают программы опроса приборов с записью в книги Excel,которые раз в сутки записываются на сервер компании. Т. к. . . .
Копировать Windows на новый диск на UEFI damix 17.10.2026 https:/ / uriesk. wordpress. com/ 2016/ 11/ 01/ move-window-10-to-ssd-on-a-uefi-dualboot-system-with-basic-linux-tools/ TL DR Вставляем диск, а старый подключаем через переходник, грузимся с лайва. . .	ИИ не может найти нужный язык в списке Supersumestria 05.10.2026 Я ему даю вот такое изображение и прошу найти и подчеркнуть немецкий язык. Возвращает он вот это: https:/ / i. **********/ vqBWLe2. png Нужную строчку в 3й колонке просто выдумал. . Это. . .	Новая последняя моя музыка в SUNO zorxor 05.10.2026 Здравствуйте, дорогие мои друзья! С большой радостью я хотел бы представить вам свою новую последнею музыку, которую сгенерировала мне по моей просьбе нейросеть SUNO. С уважением, zorxor. Это. . .	Программный домашний кинотеатр russiannick 27.09.2026 Сподобился на программный домашний кинотеатр. В качестве ЯВУ по традиции выбрал js. В помощники взял Яндекс-Алису. Было создано три зала на разные интересы. исторические и ретро сериал Хичкок. . .

@Igor3D 1980 / 836 / 115 Регистрация: 01.10.2012 Сообщений: 5,229 Записей в блоге: 2
	13.11.2025, 23:00 [ТС]
	С "универсальностью" как-то совсем молчок. Надо полагать, следует юзать std::basic_string_view с нужным типом. Но меня смущает что Qt предлагает свой QStringView, причем без всяких темплейтов, заточенный на двухбайтовый юникод (QString но не только). Мнения об альтернативных Qt классах/контейнерах различны. Одни считают Qt "великими троллями" и все их решения непогрешимыми. Другие наоборот, активно поднимают хвост и переводят все на std::, мол, сейчас это круче, и Qt должно использоваться "только для UI". Но по-любому Qt пишут хорошие профессионалы к которым стоит прислушаться. Так почему у них свой QStringView и что лучше юзать? 0

@sdf45 Покинул чат. 1133 / 728 / 195 Регистрация: 30.03.2021 Сообщений: 2,379
	14.11.2025, 21:49
	eva2326, хм, спасибо,- надо глянуть, чем заменили. 0

@sdf45 Покинул чат. 1133 / 728 / 195 Регистрация: 30.03.2021 Сообщений: 2,379
	15.11.2025, 18:30
	0

@sdf45 Покинул чат. 1133 / 728 / 195 Регистрация: 30.03.2021 Сообщений: 2,379
	16.11.2025, 12:49
	DrOffset, спасибо за разьяснение, буду знать на будущее. 0

Написать современно

Решение