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

Добрый день

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

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

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

Спасибо

0

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

Texas sharpshooter fallacy

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

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

0

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

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

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

0

Сообщение от 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 - мне не ясно. Тема была о написании современного кода на чистых плюсах.

0

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

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

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

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

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

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

0

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

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

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

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

0

Сообщение от 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 выше ответил (и про юникод тоже).

0

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

Сообщение от Igor3D без поддержки юникода

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

0

Сообщение от sdf45 std::locale utf8_locale(std::locale(), new std::codecvt_utf8<wchar_t>);

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

0

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

0

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

0

Сообщение от Igor3D А данные в QString

внезапно...

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

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

0

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

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

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

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

0

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

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

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

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

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

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

1

Сообщение от Igor3D время жизни

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

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

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

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

0

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

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

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

1

0

Сообщение от sdf45 C++ 1 2  QString q_str="some";  std::string_view std_str=q_str.toStdString();

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

0

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

0

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

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

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

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

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

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

Сообщение от sdf45 А как тогда? так что-ли

Да.

1

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

0