Версия для печати темы

Автор: RazrFalcon 15.7.2011, 1:52

В QMenu есть 3- QAction. Они отвечают за отображение текста в проге.
После выбора нужного, мне нужно со всех остальных снять "галочку". В итоге выходит вот что (реальный, рабочий кусок моего кода):

void QMediaInfo::on_actionAsText_triggered()
{
  if (actionAsText->isChecked() && cmbBoxFiles->count()!=0) {
    actionAsTree->setChecked(false);
    actionAsPseudoTree->setChecked(false);
    updateInfo();
  } else
    actionAsText->setChecked(true);
}

void QMediaInfo::on_actionAsTree_triggered()
{
  if (actionAsTree->isChecked() && cmbBoxFiles->count()!=0) {
    actionAsText->setChecked(false);
    actionAsPseudoTree->setChecked(false);
    updateInfo();
  } else
    actionAsTree->setChecked(true);
}

void QMediaInfo::on_actionAsPseudoTree_triggered()
{
  if (actionAsPseudoTree->isChecked() && cmbBoxFiles->count()!=0) {
    actionAsText->setChecked(false);
    actionAsTree->setChecked(false);
    updateInfo();
  } else
    actionAsTree->setChecked(true);
}

Если видов будет не 3-и, а 10-ть - я убьюсь...
Как правильнее реализовать задуманное? Хотелось более компактный код. Спасибо.

Автор: Алексей1153 15.7.2011, 9:47

нужно поиметь массив указателей/ссылок на элементы (составить самому, либо вытащить из объекта меню динамически). В каждом обработчике вызвать единую функцию с параметром - указатель на текущее выбранное действие. Там необходимо пробежаться по массиву и сбросить всё, что не указатель_из_параметра

Ещё вариант: иметь программную модель набора галочек (std::bitset , к примеру). При выборе галочки модель меняется так, что установлена только одна галочка. Затем модель "распечатывается на экран" (то есть из модели выводится текущее состояние галочек)

Автор: Авварон 15.7.2011, 10:32

А теперь приду я и скажу что есть http://doc.qt.nokia.com/latest/qactiongroup.html

Автор: Алексей1153 15.7.2011, 10:40

Авварон, выходит, что второй вариант уже реализован в классах Qt ))

Автор: RazrFalcon 15.7.2011, 11:29

Уже сделал как предложил Алексей1153, но с QActionGroup получилось намного проще ( -100 строк кода =) ), и теперь выглядит не как чекбокс, а к как радиобаттон, что и нужно было.
Пример:
http://itmages.ru/image/view/231396/c618559a