entry->eventSink_contextMenu.addHandler([this, subnode](auto &ev) {

Question

entry->eventSink_contextMenu.addHandler([this, subnode](auto &ev) {

if (m_pEditor->canDeleteNode(subnode)) {
auto entDelete = gnaNew<grUi::DropDownMenuEntry>(GTXT("Delete"));
entDelete->eventSink_performAction.addHandler([this, subnode](auto &ev) {
m_pEditor->deleteNode(subnode);
return true;
});
ev.dropDown->add(entDelete);
}
return true;
});

а насколько идиоматично такое? (ну, если не делать такое декларативно)

#c++ #embedded #programming #russian

0

13.11.2023

17 ответов

44 просмотра

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

Гражданин Котейко
зависит от того кого спрашивать я был лично знаком...

моё мнение - ересь, но проблема в том, что её писал я)))

0

13.11.2023

Ilya Zviagin

В меню добавляют пункт , и вешают на него лямбду ?

0

13.11.2023

Гражданин Котейко

usernameak
моё мнение - ересь, но проблема в том, что её писа...

а на текущей работе мой начальник при виде камелКейса становится весь красный и начинает выпускать пену изо рта

0

13.11.2023

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

Ilya Zviagin
В меню добавляют пункт , и вешают на него лямбду ?

так.

0

13.11.2023

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

Гражданин Котейко
а на текущей работе мой начальник при виде камелКе...

у меня там как видишь снейк_камелКейс местами)))

0

13.11.2023

Ilya Zviagin

usernameak
так.

И что тут плохого может быть ? Надо только быть уверенным, что захват лямбды не протухнет до конца жизни этого пункта меню.

0

13.11.2023

Ilya Zviagin

usernameak
у меня там как видишь снейк_камелКейс местами)))

Верблюд оседлал змею...

0

13.11.2023

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

Ilya Zviagin
И что тут плохого может быть ? Надо только быть ув...

так он же по значению

0

13.11.2023

Гражданин Котейко

Ilya Zviagin
И что тут плохого может быть ? Надо только быть ув...

ну или хотя бы не проживёт дольше чем объект на который кажет захваченый this

0

13.11.2023

Ilya Zviagin

usernameak
так он же по значению

В коде этого не видно, да и this что по значению, что по ссылке...

0

13.11.2023

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

Ilya Zviagin
В коде этого не видно, да и this что по значению, ...

хмммм для этого надо как минимум каким-то образом закрыть окно не закрывая меню :) что в принципе невозможно, потому что меню закрывается сразу же перед этим =)

0

13.11.2023

Гражданин Котейко

usernameak
хмммм для этого надо как минимум каким-то образом ...

"life finds a way" ™

0

13.11.2023

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

Ilya Zviagin
И что тут плохого может быть ? Надо только быть ув...

ну а subnode вообще рефкаунтед, он там тупо не может протухнуть :)

0

13.11.2023

Гражданин Котейко

usernameak
ну а subnode вообще рефкаунтед, он там тупо не мож...

ну мы по этому куску кода этого знать то не можем, это надо в контекст быть погруженным

0

13.11.2023

Ilya Zviagin

usernameak
ну а subnode вообще рефкаунтед, он там тупо не мож...

если ты думаешь, что это хорошо, то знай, что это не так, это плохо. Объект существует пока есть его обработчик, а обработчик - пока есть объект... "Классно" (нет)

0

13.11.2023

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

Ilya Zviagin
если ты думаешь, что это хорошо, то знай, что это ...

не совсем так :) у меня много где протыканы вручную weak референсы

0

13.11.2023

Гражданин Котейко · Accepted Answer

Гражданин Котейко

зависит от того кого спрашивать я был лично знаком с людьми, которые про это скажут "ересь" и также был лично знаком с людьми, которые скажут "шикарно, добротно сделано"

0

13.11.2023

170 похожих чатов

entry->eventSink_contextMenu.addHandler([this, subnode](auto &ev) {

17 ответов

Похожие вопросы