emit сигнала

[Авварон]

Нормально ли эмитить сигнал void keyChanged(const QString &key); _до_ смены значения key?:

Код:

void Edition::setKey(const QString &key)
{
    Q_D(Edition);

    if (d->key != key) {
        emit keyChanged(key);
        d->key = key;
    }
}

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

[asvil]

void keyChanged(old, new)?

Postscriptum: void keyChanged(new, old)? Можно приконнектить однопараметровый слот.

[Пантер]

А если keyChanged(const QString &oldKey, const QString &newKey)?

Опередил.

[Авварон]

а со скриптами оно будет дружить? там в NOTIFY секцию не однопараметровый ли сигнал надо?
Так бы и сделал бы, просто скриптовость останавливает

[Пантер]

Ну, а ежели 2 сигнала?

[Авварон]

А это уже криво) Имплементация торчит наружу. в принципе там скорость не критична. А можно и обратную мапу забабахать. Однако ж интересны мнения, как написать "красиво".

[asvil]

Две проперти?

[Авварон]

Оригинально:) Но тоже криво. Что 2й сигнал, что 2я пропертя бесполезны всему остальному приложению, они нужны только 2му классу (пусть будет в дальнейшем Base), к-ый содержит мапу.

[asvil]

Немного не в тему, но мощно. Звучит примерно так:
Каждый QObject-based класс должен реализовывать сигнал propertyChanged(propertyName, newValue, oldValue).

Ради одного класса лучше пренебречь красотой. Или патчик moc.

[_govorilka]

Ты можешь посылать в сигнале структуру. Тогда если тебе понадобиться еще один параметр, ты просто добавишь поле в эту структуру и тебе не придется переделывать сигнал. Если под скриптами - подразумеваешь QScript, то добавить структуру в скрипт не сложно (в библиотеки есть соотвествующие примеры).

[Авварон]

Не, меня декларатив интересует. Это там ограничение на сигнатуру кажется присутствует. В кьютскрипт любой сигнал можно подцепить.

[Mikhail]

Скрипты могут получать сигналы более чем с одним параметром.
Главное создать в скрипте слот с требуемым числом параметров.
Проверено на опыте. Работает

[Авварон]

вы про qml?

[Sahab]

http://www.anyqsoft.com

<offtop>the sexadecimal editor written completely on Qt</offtop>

[twp]

если я правильно понял, был установлен кастом виджет в QML сцену через QGraphicsProxyWidget и регистрацию QML_DECLARE_TYPE. Тогда можно не использовать стандартный сигнал в секции NOTIFY декларации Q_PROPERTY (но можно и оставить как сигнал для стандартной реализации) и добавить сигнал с двумя параметрами как было предложено выше. В QML-коде его можно перехватить как обычный сигнал на изменение проперти