Методы (can) copy, paste и.т.п

[Igors]

Добрый день

Задумал переделать одно место, работает, но уж очень коряво, и новый код плохо ложится. Банальная реакция на команду (часто из меню). Может, не мудрствуя лукаво, сделать так

Код

C++ (Qt)
struct CWinCommand {
 bool CanCopy( QString * txt ) const = 0;   // можем копировать? и что (подсветка и текст меню)
 bool DoCopy ( void  ) = 0;
 
 bool CanPaste( QString * txt ) const = 0;
 bool DoPaste ( const QVariant & ) = 0;
 
 void CanDelete(... 
 ...
};
 

Ну и долить этот интерфейс в 2 базовых класса множественным наследованием. Но как-то засомневался... Может лучше слотами/сигналами? Но каких-то аргументов в их пользу не нашел. Наоборот, нужно чтобы код текущих окон не компилился (abstract class). Предрассудки типа "множ наследование - зло" не рассматриваем. Что я еще мог упустить? Или еще как можно (без сигналов)?

Спасибо

[qate]

struct CWinCommand {
 bool Can(WhatEnum what, cost QString& desc) const = delete;   // можем что
 bool Do(WhatEnum what, cost QString& param = {}) = delete;
 };

?

[Igors]

struct CWinCommand {
 bool Can(WhatEnum what, cost QString& desc) const = delete;   // можем что
 bool Do(WhatEnum what, cost QString& param = {}) = delete;
 };

?

В "can" методах строка заполняется (чтобы подставить ее в меню), поэтому по указателю. Ну а в целом это ничего нового не вносит, подробности оформления.

[Авварон]

Не уверен, что это имеет отношение к паттерну "команда"...

[Igors]

Не уверен, что это имеет отношение к паттерну "команда"...

Ну слово "команда" можно употреблять и без всякой связи с паттернами  Есть ли здесь (разумная) альтернатива простецкому "интерфейсу" ?

[Авварон]

Не уверен, что это имеет отношение к паттерну "команда"...

Ну слово "команда" можно употреблять и без всякой связи с паттернами  Есть ли здесь (разумная) альтернатива простецкому "интерфейсу" ?

Очевидно, что если ваш код будут читать другие, то не надо называть копируемые объекты Синглтонами, синглтоны - Фасадами и тп:)
Просто потому, что это создаёт путаницу - например, сразу появляется вопрос - а может, это всё таки команда и вы что-то недоговариваете?

[Авварон]

Ну а так, можно например так - при смене фокуса поднимаемся по иерархии виджетов, кастим виджет к интерфейсу ICopyable { bool enabled() = 0; QString text() const; void trigger(); } и поджигаем нужные экшны если есть интерфейс и он enabled()

[Igors]

Ну а так, можно например так - при смене фокуса поднимаемся по иерархии виджетов, кастим виджет к интерфейсу ICopyable { bool enabled() = 0; QString text() const; void trigger(); } и поджигаем нужные экшны если есть интерфейс и он enabled()

Это Вы говорите о "синхронизации", т.е. как (или когда) напр метод "canXXX" получит управление. Для меню у меня все айтемы устанавливаются когда юзер тыкнул на меню (берем активное окно и парим copy, paste, delete и.т.п. - все что нужно в меню)

[Авварон]

Ну а так, можно например так - при смене фокуса поднимаемся по иерархии виджетов, кастим виджет к интерфейсу ICopyable { bool enabled() = 0; QString text() const; void trigger(); } и поджигаем нужные экшны если есть интерфейс и он enabled()

Это Вы говорите о "синхронизации", т.е. как (или когда) напр метод "canXXX" получит управление. Для меню у меня все айтемы устанавливаются когда юзер тыкнул на меню (берем активное окно и парим copy, paste, delete и.т.п. - все что нужно в меню)

Так у вас не совсем корректно будут работать шорткаты, ну да ладно.

[Igors]

Так у вас не совсем корректно будут работать шорткаты, ну да ладно.

Автоматом/пулеметом не будут, нужно перехватить QEvent::ShortcutOverride