While ([this](){ std::lock_guard<std::mutex> lg(jobs_mtx_); return !jobs_.empty(); }()) { } Норм практис? jobs_ - очередь

Question

supapro.cxx

Dmitry

While ([this](){ std::lock_guard<std::mutex> lg(jobs_mtx_); return !jobs_.empty(); }()) { } Норм практис? jobs_ - очередь

задач

#c #embedded #programming #russian

0

21.02.2021

7 ответов

14 просмотров

Ofee Oficsu

Ну, и вообще, этот код вызывает вопросы. Что, если в момент проверки очередь была не пуста, а сразу после кто-то уже забрал задачу в другом потоке? После открывающей фигурной скобки мы не можем полагаться на истинность условия

0

21.02.2021

Dmitry Автор вопроса

Ofee Oficsu
Ну, и вообще, этот код вызывает вопросы. Что, если...

ты прав, теперь я засунул вытаскивание в тот же захват

0

21.02.2021

Dmitry Автор вопроса

Ofee Oficsu
Ну, и вообще, этот код вызывает вопросы. Что, если...

Кстати ответ есть простой. Вытаскиваю из очереди я только в одном месте и отвечаю за это самостоятельно

0

21.02.2021

Ofee Oficsu

Ofee Oficsu
while (std::lock_guard<std::mutex>(jobs_mtx_), !jo...

Я бы не стал так использовать лямбду, предпочёл бы вынести логику в функцию с говорящим названием. Но если хочется однострочник — как я показал выше, в C++17 можно обойтись более компактной конструкцией: while (std::lock_guard(jobs_mtx_), !jobs_.empty()) { ... } Её понятность оставляет желать лучшего, но я всё ещё нахожу эту конструкцию читаемее вызова лямбды

0

21.02.2021

Dmitry Автор вопроса

Ofee Oficsu
Я бы не стал так использовать лямбду, предпочёл бы...

ага, спасибо!

0

21.02.2021

Ilya Zviagin

Ничего , что у тебя лямбда будет в каждой итерации цикла новая создаваться?

0

22.02.2021