QSortFilterProxyModel - удаление строки

[Serega]

Удаляю итем в прокси модели через Контекстное меню и клавишу Del. Заодно покритикуйте решение или предолжите лучше.
Контекстное меню устроил так:

Код

C++ (Qt)
void proxyPaneWidget::showCustomMenu(const QPoint &pos)
{
  m_currentItemIndex = m_proxyList->indexAt(pos);
 
  m_proxyMenu->addAction( m_deleteAction );
  m_proxyMenu->exec( m_proxyList->mapToGlobal( pos ) );
 
  m_currentItemIndex...; // HOW TO DO?
}

В методе удаления проверяю как пользователь хочет удалить итем, через Контекстное меню или с помощью клавиши Del:

Код

C++ (Qt)
void proxyPaneWidget::deleteSelectedItem()
{
  QModelIndex index = m_proxyList->currentIndex();
 
  if( m_currentItemIndex.isValid() )
    index = m_currentItemIndex;
 
  m_source->removeItem(m_proxyModel->mapToSource(index).row());
}

Проблема в том, что если пользователь вызывал контекстное меню и передумал удалять, то индекс остается, соответственно есть шанс удалить клавишей не ожидаемый пользователем итем.
В модели дерева файловой системы меню удаления делал подобным же образом, но там можно было просто послать строку пути, которую не сложно было обнулить.
Собственно вопрос такой. Как сделать индекс m_currentItemIndex невалидным или занулить? Наверное подобный вопрос можно было бы задать в ветке для новичков. Но может само решение удаления не выдерживает критики?

[Serega]

Ладно будем думать, что молчание знак того, что ерундой занимаюсь и так не делают. Просто не хотел заводить вторую переменную если можно было обойтись одной. Но может это излишняя предосторожность.
Создал булеву переменную.

[gil9red]

По-моему, можно проще удалять:

Код

C++ (Qt)
void proxyPaneWidget::deleteSelectedItem()
{
  const QModelIndex & index = m_proxyList->currentIndex();
 
  if ( index.isValid() ) {
      int row = m_proxyModel->mapToSource(index).row();
      m_source->removeItem(row);
  }
}

Да и с вызовом контекстного меню тоже самое -- проверяйте валидность текущего индекса и если валидный, добавляйте действие удаления.
Смысл хранить текущий индекс в какой-то переменной я не вижу -- у вас уже есть виджет, который сможет текущий индекс вернуть

[Serega]

Так просто не получится иначе бы не заморачивался. Возможно переменную индекса неудачно назвал.
Как видим из меню пользователь не обязательно мог навести курсор на текущий инденкс. Иначе он мог бы удалить не ожидаемый пункт.
Как мне кажется выбрал более интуитивный вариант - отсылаем позицию курсора.

[gil9red]

Так просто не получится иначе бы не заморачивался. Возможно переменную индекса неудачно назвал.
Как видим из меню пользователь не обязательно мог навести курсор на текущий инденкс. Иначе он мог бы удалить не ожидаемый пункт.
Как мне кажется выбрал более интуитивный вариант - отсылаем позицию курсора.

Текущий индекс это тот, который выделен в *View, и выделить элемент можно не только курсором, но и клавишами клавиатуры

[Serega]

Поэтому и говорю, что неудачно назвал переменную индекса.
Дело в том, что в прокси представлении действительно выделен текущий индекс и пользователь может менять его клавиатурой или ЛКМ.
Но пользователь может навести курсор мыши на другой пункт и вызвать контекстное меню. И здесь стоит упоминуть, что действия правой кнопки мыши я переназначил и пункт не становится текущим. Сделано  это, что бы не правоцировать ненужные действия после выбора пункта.

[Igors]

И здесь стоит упоминуть, что действия правой кнопки мыши я переназначил и пункт не становится текущим. Сделано  это, что бы не правоцировать ненужные действия после выбора пункта.

А что мешает вызвать удаление сразу после закрытия контекстного меню? m_currentItemIndex просто застрелить, а удаление оформить напр так

Код

C++ (Qt)
void proxyPaneWidget::deleteSelectedItem( QModelIndex * delIndex )  // delIndex = 0 в прототипе
{
  QModelIndex index = delIndex ? *delIndex : m_proxyList->currentIndex();
 

[Serega]

И здесь стоит упоминуть, что действия правой кнопки мыши я переназначил и пункт не становится текущим. Сделано  это, что бы не правоцировать ненужные действия после выбора пункта.

А что мешает вызвать удаление сразу после закрытия контекстного меню? m_currentItemIndex просто застрелить, а удаление оформить напр так

Код

C++ (Qt)
void proxyPaneWidget::deleteSelectedItem( QModelIndex * delIndex )  // delIndex = 0 в прототипе
{
  QModelIndex index = delIndex ? *delIndex : m_proxyList->currentIndex();
 

Признаю. Ломал голову и ничего не понял. Как устроен слот ясно. Как до него донести индекс с контекстного меню? Естественно в меню у меня не единственный пункт.

[Igors]

Признаю. Ломал голову и ничего не понял. Как устроен слот ясно. Как до него донести индекс с контекстного меню? Естественно в меню у меня не единственный пункт.

Необязательно прямо из меню "сигналить", exec возвращает выбранный QAction.

[Serega]

Спасибо за подсказку. У нас ночь уже. Днем буду разбираться.

[Serega]

Снова наэкспериментировался. Запустить слот удалось. Проблема в том, что клавиша Del не работает. Видимо в этом случае ей не хватает соединения сигнал-слот. Но стандартного сигнала для экшина отправляющего хотя бы позицию курсора я не нашел. Не заводить же неаследника для создания сигнала? Как то затратно. Можно навесить событие на Del, но хрен редьки не слаще.
В общем вопрос. Что не так? Можно ли как то исправить?

Код

C++ (Qt)
void proxyPaneWidget::showCustomMenu(const QPoint &pos)
{
  QAction * m_deleteAction = new QAction( this );
  m_deleteAction->setText( tr( "Delite" ) );
  m_deleteAction->setShortcut( QKeySequence( QKeySequence::Delete ) );
  m_deleteAction->setShortcutContext( Qt::WidgetWithChildrenShortcut );
 
  m_proxyMenu = new QMenu( m_proxyList );
  m_proxyMenu->addAction( m_deleteAction );
 
  QModelIndex selectedItemIndex = m_proxyList->indexAt(pos);
 
  QAction *menuAct = new QAction(this);
  menuAct = m_proxyMenu->exec( m_proxyList->mapToGlobal( pos ) );
 
  if( menuAct->text() == "Delite" )
    deleteSelectedItem(&selectedItemIndex);
}

Код

C++ (Qt)
void proxyPaneWidget::deleteSelectedItem(QModelIndex * delIndex)
{
  m_currentItemIndex = delIndex ? *delIndex : m_proxyList->currentIndex();
  m_source->removeItem( m_proxyModel->mapToSource( m_currentItemIndex ).row() );
}

Или я что то недопонимаю, что скорее всего. Или это тупиковая затея.

[Igors]

Поняли меня правильно, только зачем сорите членами класса? m_proxyMenu (кто за Вас удалять его будет?) и m_deleteAction - не видно никакого смысла делать их членами.

Проблема в том, что клавиша Del не работает.
..
В общем вопрос. Что не так? Можно ли как то исправить?

Не так = непродуманная постановка. При выпадании контекстного меню Вы решили не менять selection а действия меню прилагаются к тому айтему на котором меню появилось. Возможно этот айтем сейчас и не выбран. Это решение спорно. Ну ладно, но тогда причем тут клавиша Del? Что должно произойти если без всякого меню юзер давит Del? Должны удалиться выбранные. А в выпавшем контекстном меню у Вас совсем другое удаление - так чего Вы туда shortcut Del лепите?

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

Delete <имя айтема>

И без всяких щорткатов

[Serega]

Спасибо! По поводу членов класса. Есть такое дело. Необучен. Не всегда представляю (если вообще представляю это правильно) когда я сам должен удалять мусор, когда за меня это сделает мусорщик. Но это был бы следующий ученический вопрос по поводу правильности оформления данного кода (как бы забежали вперед на счет работы над ошибками). Учту и исправлю. Экшин и Меню изначально инициализировались в конструкторе виджета, теперь вроде как осознаю, что такой подход не верен. Хотя вроде как часто видел в кодах, что Акшины именно там готовились, потом заряжались в методе меню.

По поводу непродуманной постановки. Попробую объяснить. Пользователь не просто просматривает список, при выборе пункта списка открывается информационное окно для данного пункта. И когда надо удалить пункт списка вовсе не обязательно, что бы фокус уходил с уже открытого инфоокна, следовательно необязатель, а возможно и не желательно делать айтем текущим. Если же захочет, то для этого ЛКМ.
Считал, что клавиша Del для того и существует, что бы пользователь мог работать клавиатурой без применения мыши, манипулируя клавишами стрелок и удаления. Это происходит гораздо быстрее при быстром списковом просмотре. Изначально предложенная http://www.prog.org.ru/index.php?topic=28819.msg211186#msg211186 реализация именно так и работает (совместно и мышь как надо и клавиша Del). Подумывал сделать это более правильно если есть способ. В контекстном меню просто напоминание, что работает клавиша Del. Зачем же пользователю открывать контестное меню, что бы нажать клавишу Del? И что в данном случае я сделал не так, мне не понятно. Если пользователь без всякого контекстного меню давит Del - удаляется текущий пункт, только и всего.

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

В любом случае спасибо. Пока решаю те или иные вопросы, набираюсь опыта как это делают профи. В данном случае думаю пригодится такой способ работы с контекстным меню. Хотя пока и не удалось применить его для данной задачи. Но зато научился отсылать сигналы и вызывать методы в зависимости от выбранного пунтка меню. Признаюсь был в тупике как это сделать.

[gil9red]

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

В с++ нет сборщика мусора

[Serega]

Разве в QT концепция отцов и детей это не один из способов реализации сборщика мусора?