Russian Qt Forum

Добрый день

Не раз замечал что у любителей std преобладают имена из 3 букв. Но у меня-то нет

C++ (Qt)
if (std::find(mUsedThresholds.begin(), mUsedThresholds.end(), tindex) != mUsedThresholds.end())
  mUsedThresholds.push_back(tindex);
 

Да еще и неправильно (контейнер может быть пустым). Дело не в том что мне, мол, "лень писать", но оно выглядит отвратно :'( Ф-ционала на копейку, текста на рупь.

Прошло неск лет как это обсуждали, может появились средства языка позволяющие обойтись без таких длинных соплей? (новые стандарты и все такое)

Спасибо

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

C++ (Qt)
class Threshols
{
public:
    bool isContains( int val ) const
    {
        return std::find( mUsedThresholds.begin(), mUsedThresholds.end(), val ) != mUsedThresholds.end();
    }
 
    void append( int val )
    {
        if( !isContains( val ) )
            mUsedThresholds.push_back( val );
    }
 
private:
    std::vector<int> mUsedThresholds;
};
 

Заодно, когда надоест постоянно бегать по вектору для поиска, это позволит незаметно для остального кода проекта это оптимизировать.


Если делать как вы привыкли и все располагать в классе MainWindow, то всегда можно добавить шаблонную функцию isContains, тогда можно будет делать так:

C++ (Qt)
if( !isContains( mUsedThresholds, tindex) )
  mUsedThresholds.push_back( tindex );
 

Но зачем вам об этом заморачиваться - копипастите и все.

Ну конечно... Вы то у нас о-го-го. Судя по вашим то темам. :)

Ну и что? Если контейнер пустой, то begin() == end(). Не переживайте за стандартные алгоритмы, они с этим справляются. :)
Но вы для верности всегда можете добавлять перед find - !mUsedThresholds.empty() &&
Тогда, читающие ваш код специалисты, сразу будут понимать с чем имеют дело.

а почему бы в данном примере std::set не заюзать?
тогда не нужно будет это длинное условие

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

Мда, похоже что "а воз и ныне там". Нормально было бы так

C++ (Qt)
if (!mUsedThresholds.contains(tindex))
  mUsedThresholds.push_back(tIndex);

Кстати "обветшавшие" Qt контейнеры все это имеют: find, contaiins, indexOf и.т.п. А добиться того же великом оказывается не так уж просто. Напр

C++ (Qt)
template<class T, class Val>
bool MyContains( const T & vec, Val val )
{
  return std::find(vec.begin(), vec.end(), val) != vec.end();
}

Это запросто сработает для мапы убивая ф-ционал. На мой взгляд лучше скромное

C++ (Qt)
template<class T, class Val>
bool VecContains( const std::vector<T> & vec, Val val );

Ну и дублить если еще понадобится (напр для деки). Все равно хорошего впечатления не производит. "Костыль", а хотелось бы "ис каропки"

Вам нужно лучше изучить инструмент, который вы пытаетесь использовать.
Для мапы код не скомпилируется (поэтому убийтсва функционала не последует :) ), итератор мапа при разименовании не дает валуе.
К тому же у мапа есть свой метод для этого.
Даже если в коде был вектор и его заменили на мап, то при первой же сборке возникнет ошибка и разработчик замент этот вызов на соответствующий метод мапа.

Это что, идентификаторы из 3 букв? :)
А где же "Но у меня-то нет", или тайный любитель std? :)

Может симпатичнее оформить как namеspace и юзать так

C++ (Qt)
if (std_ext::contains(mUsedThresholds, tindex)) ..

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

Возможно, не до конца понял суть вопроса. Но что насчёт https://en.cppreference.com/w/cpp/ranges (https://en.cppreference.com/w/cpp/ranges)?

Что-бы было не "хз", нужно книжку почитать по c++. :)
Но можно и дальше на форуме решения клянчить. :)

Да "суть" тут "под каждым кустом" :) Просто хочется схлопнуть код что выглядит неопрятно. Неужели для этого надо иметь последний стандарт, освоить новую концепцию и.т.п? Ну как-то "неадекватно".

Что касается ranges - назрело давно, много было великов, один я даже приводил в "кладовой" (уничтожающая критика местных знатоков :)). Но как range поможет схлопнуть тот же std::find который тупо требует пару итераторов ?

https://en.cppreference.com/w/cpp/algorithm/ranges/find (https://en.cppreference.com/w/cpp/algorithm/ranges/find)

Боюсь что эта вещь не предназначена для моих скромных целей (компактный вызов). И это лишь неск конкретных ф-ций

Кстати интересно, а зачем нужна std::ranges::find, почему не обойтись std::find или в чем разница? Судя по примеру реализации, в std::invoke. Немного копнул, да, познавательно, и наверное "небесполезно". Но не вижу ни одного примера в своих проектах где в этом нужда. В общем, "беллетристика" :)

Вздох

Вы не замечали что перевод устойчивых буржуйских словечек неизменно уродлив? Так зачем самому повторять эту ошибку? :) Хоть бы творчески переводили (по смыслу это  "эх")

Не знаю можно ли (с новыми стандартами) записать v.begin/end, может в этом смысл? Но лаконичностью и выразительностью такой код точно не блещет (даже с калечным "v"). Так что нечего там (томно) вздыхать  :)

против


я даже не знаю что короче

что вам не нравится? итератор? ну напишите функцию, которая возвращает range (например, ленивый findAll), будете писать