Local Event Loop

[Igors]

Добрый день

Хочу такой ф-ционал (псевдокод)

Код

C++ (Qt)
while (IsMouseDown()) {
  QPoint pt = GetMousePosition();
  if (pt != oldPt)
   DragSomething(pt);
  oldPt = pt;
}
 

Ну конечно цивильно, а не жрать проц в IsMouseDown. Какие есть решения?

Спасибо

[Racheengel]

например добавить

qApp->processEvents();
QThread::msleep(100);

[Igors]

например добавить

qApp->processEvents();
QThread::msleep(100);

"Please be more concrete". Что должно быть в IsMouseDown?

[RedDog]

локальный  QEventLoop нельзя подписать на moseDown ?

[Igors]

локальный  QEventLoop нельзя подписать на moseDown ?

"Прошу исполнить"

[Racheengel]

но вообще на mouseMove надо реагировать.
передвинулась мышь? проверили, зажата ли кнопка. если да - что-то драгаем.

[Igors]

но вообще на mouseMove надо реагировать.
передвинулась мышь? проверили, зажата ли кнопка. если да - что-то драгаем.

Это размазывает код между событиями, что далеко не всегда удобно, особенно при обильных и/или сложных драгах. Гораздо приятнее написать весь драг в одном месте, от начала до конца. Разве это запрещено? Нарушает какие-то прынцыпы? Удивляет неуверенность ответов, ведь гуй - Ваше родное, я же не о матрицах спрашиваю Почему не так (псевдокод)

Код

C++ (Qt)
extern bool isDragging = false;
 
bool IsMouseStillDown( void )
{
   if (!isDragging) {
    qApp->installEventFilter(&myFilter);
    isDragging = true;
   }
 
   qApp->processEvents(QEventLoop::WaitForMoreEvents);
 
   if (!isDragging)
    qApp->removeEventFilter(&myFilter);
 
  return isDragging;
}
 

Ну и в myFilter жрать события мыша пока не придет нужное MouseRelease, тогда сбросить флажок isDragging.

Хорошо ли это, какие здесь проблемы?

[Авварон]

Igors придумал корутины

[Igors]

Igors придумал корутины

Ну мыслящий человек обязан что-то придумывать Но корутины-то здесь причем? Открыл гугл

Корутины — это новый способ написания асинхронного, неблокирующего кода.

В чем его новизна - хз, просветите. Но никакой асинхронности здесь нет.

[Racheengel]

Хорошо ли это, какие здесь проблемы?

Хорошего ничего не вижу, кроме недостатков.
Пытаясь уйти от проблемы "размазывания кода", Вы создаёте несколько новых проблем.
1. Появляется второй ивент-луп в том же гуй-потоке.
2. Создается дополнительный фильтр событий, которому всё равно нужен отдельный обработчик.
3. Откуда будет зваться IsMouseStillDown() ?
4. Опять же, нужен обработчик логики драга. Что делать, если начали драг с зажатым Ctrl? Или если поведение логики дропа меняется, в зависимости от того, где отпустили мыш?

В общем без дополнительных методов не обойтись, ну а тогда в чём выигрыш данного подхода?

Вообще тема драг-дропа довольно гнилая с точки зрения "прынцыпов", потому что, как правило, приходится в любом случае либо городить огромную state-machine, либо много драг-хендлеров.

Я бы сделал вместо своего ивент лупа базовую обработку типа "нажал (mousePress) - подвинул мыш (mouseMove) - отпустил (mouseRelease)". А уже из mouseMove() вызывать что то типа void onDragging(CMyObject* draggedObject). А уже из данного метода, в зависимости от того, кем является draggedObject (и есть ли он вообще - вдруг юзер решил рабочее поле драгать? например, для выделения или скролла) - вызывать соответсвующие обработчики драгов.

"я так делал, так работает" (с) я

[Igors]

Хорошего ничего не вижу, кроме недостатков.

Пытаясь уйти от проблемы "размазывания кода", Вы создаёте несколько новых проблем.
1. Появляется второй ивент-луп в том же гуй-потоке.
2. Создается дополнительный фильтр событий, которому всё равно нужен отдельный обработчик.
3. Откуда будет зваться IsMouseStillDown() ?
4. Опять же, нужен обработчик логики драга. Что делать, если начали драг с зажатым Ctrl? Или если поведение логики дропа меняется, в зависимости от того, где отпустили мыш?

В общем без дополнительных методов не обойтись, ну а тогда в чём выигрыш данного подхода?

Давайте "говорить об одном и том же" (первый закон логики). Я предложил обсудить проблемы реализации IsMouseStillDown. Да, они непросты, не исключено что именно поэтому желаемый ф-ционал недостижим. Вы же утверждаете что данный ф-ционал вообще неполноценен и не дает выгод. Это не так, выгоды как раз очевидны. Во-первых, IsMouseStillDown пишется один раз на все случаи жизни - уже хорошо. Зовется она при всяком драге, обычно из обработчика MousePressed, тут возможно не понял что Вы имели ввиду. И уж тем более не понял последнего пункта

4. Опять же, нужен обработчик логики драга. Что делать, если начали драг с зажатым Ctrl? Или если поведение логики дропа меняется, в зависимости от того, где отпустили мыш?

Тут как раз все прекрасно. Нужны доп клавиши - просто снял их перед циклом while или или внутри него или передал аргументом от исходного MousePressed. Изменился target под мышей -ну зафиксировали это в локальной объявленной до цикла while (напоминает выгоды лямбды). Наоборот, в событийной схеме все это пусть несложно, но очень хлопотливо.
 
Я совсем не предлагал реализовывать какую-то логику драга в IsMouseStillDown, она просто возвращает false (= мыш отпущен) или true (= мыш зажат, но что-то случилось) - и все 

"я так делал, так работает" (с) я

У меня тоже многое (или почти все) работает, но далеко не всем я могу гордиться 

[Igors]

Да, и забыл пожалуй главное  "Модальность". Напр драгается нечто по MouseMoved - но это никак не запрещает др событий, глупый юзвеоь может напр закрыть окно и.т.п. Женщины в таких случаях советуют "А зачем Вы это нажимали? Не нужно было этого делать" 

[Racheengel]

Да, и забыл пожалуй главное  "Модальность". Напр драгается нечто по MouseMoved - но это никак не запрещает др событий, глупый юзвеоь может напр закрыть окно и.т.п. Женщины в таких случаях советуют "А зачем Вы это нажимали? Не нужно было этого делать" 

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

[Old]

Зато на Alt-F4 нажать сможет легко.

[Racheengel]

Получается следующая петрушка:

если делать на "стандартных" мышесобытиях, то выглядеть будет примерно так:

Main Thread (GUI)
"Main Event Loop":

Код:

::onMousePressed(MouseEvent *e):
   m_clickPosition = e->pos();
   m_objectDragged = findObjectAt(m_clickPosition);
   m_isDrag = true;

::onMouseReleased(MouseEvent *e):
   hangleRelease(e->pos(), m_objectDragged);
   m_isDrag = false;

::onMouseMoved(MouseEvent *e):
   if (m_isDrag)
     handleDrag(e->pos(), m_objectDragged)

"Свои имплементации":
hangleDrag() - тут тянем объект (меняем координаты whatever)
hangleRelease() - тут дропнули объект



А если на "локальном эвентлупе":

Main Thread (GUI)
"Main Event Loop":

Код:

::onMousePressed(MouseEvent *e)
{
   m_clickPosition = e->pos();
   m_objectDragged = findObjectAt(m_clickPosition);
   m_isDrag = true;

  m_oldPosition = e->pos();

  installFilter(this);

   while (m_isDrag)
   {
      if (m_oldPosition != m_lastEvent ->pos())
      {
         m_oldPosition = m_lastEvent ->pos();
         hangleDrag(m_lastEvent , m_oldPosition);
      }

      qApp->processEvents();
      msleep(...);
   }

  uninstallFilter(this);

  hangleRelease(m_lastEvent ->pos(), m_objectDragged);
}


::onMouseReleased(MouseEvent *e)
{
  m_lastEvent = e;
  m_isDrag = false;
}


::onMouseMoved(MouseEvent *e)
{
   if (m_isDrag)
     m_lastEvent = e;
}

То есть стало больше кода, появился фильтр эвентов, промежуточные переменные, и всё равно надо имплементировать hangleDrag() и hangleRelease()...
Не понятно, где мы выигрываем