РЕШЕНО: std::find_if (Как правильно вызвать?)

[devilr]

Доброго времени суток!
Есть код

Код: Выделить всё

    typedef struct CMD
    {
        int cmd;
        QString cmd_text;
    } *cmd_t;
    QList<CMD> commands;
    
    ...
    foreach (CMD command, commands)
    {
        if(command.cmd == cmd)
        {
        	//что-то делаем
        	return;
        }
    }
    

До недавнего времени всё было хорошо, но потом появилось предупреждение "consider using std::find_if algorithm instead of a raw loop" на подобный способ поиска.
Как это можно правильно переписать, может кто сталкивался?

[ormorph]

Это всего лишь предложение компилятора, так как усматривает готовый алгоритм STL для поиска, не факт что это ускорит программу. Можно просто плюнуть, а можно попробовать создать свою функцию предикат, которая будет делать сравнение и передать ее в find_if. Методы начала и конца уже реализованы в контейнерах qt - begin, end.
Тут показывается как выглядит реализация данного цикла и пример использования.

Зы: просто предлагает замену цикла foreach.

[devilr]

Я читать умею.
Просто не понимаю, как в моём случае делать? Там ведь структура, а не просто переменная.

не факт что это ускорит программу

Так я не ускорить хочу, а просто понять, как это делается?

[Bizdelnick]

Как-то так:

Код: Выделить всё

auto cmd_it = find_if(
    commands.begin(),
    commands.end(),
    [cmd](CMD &command){ return command.cmd == cmd; }
);
if cmd_it != commands.end() {
    // что-то делаем
}

[ormorph]

ведь структура, а не просто переменная.

Ну так и смотрите, там в функцию предиката передается объект структуры(указатель), ну а что с ним делать, можно добавить отладочную печать в предикате.
Предикат функция bool с аргументом типа итератор Qlist.. , он и будет указывать на структуру. Можно сразу в качестве аргумента указать структуру, итератор же указывает на структуру. Ну а дальше весь функционал в предикате.

[devilr]

Bizdelnick, хитро.
Век живи - век учись! Спасибо!