Частое добавление и удаление строки в QAbstractTableModel

[YvenTitan]

Добрый день!

Стоит задача сделать таблицу обновлений с помощью QAbstractTableModel.

В небольшом тестовом примере таблица из 4 строк. В нем в отдельном потоке я раз 1 милисекунду добавляю в конец таблицы одну строку с помощью insertRows и одну строку удаляю из ее начала(removeRows). В итоге через 10-60 секунд работы приложение вылетает.
Если добавлять и удалять строки раз в 10 милисекунд, то работает нормально.

Как сделать, чтобы приложение работало стабильно?

[Bepec]

Перестать пороть чушь и сделать нормальную архитектуру приложения.

[YvenTitan]

Можно по конкретнее, что именно исправить

Вылетает с ошибкой:
ASSERT failure in QList<T>::at "index out of range" file qlist.h

This application has requested the Runtime to terminate it in an unusual way

[gil9red]

Можно по конкретнее, что именно исправить

Вылетает с ошибкой:
ASSERT failure in QList<T>::at "index out of range" file qlist.h

This application has requested the Runtime to terminate it in an unusual way

Выход за пределы диапазона. Вы у списка запрашиваете элемент, индекс которого нет.

[Bepec]

Пороть чушь - про добавление и удаление строки с интервалом 1мс.
А так если у вас проблемы в КОДЕ - покажите КОД.

[Old]

Пороть чушь - про добавление и удаление строки с интервалом 1мс.

А что в этом страшного?

А так если у вас проблемы в КОДЕ - покажите КОД.

Так у ТС проблема в коде или архитектуре?

[Old]

Можно по конкретнее, что именно исправить

Нельзя так просто работать со списками из разных потоков. Нужно использовать средств синхронизации.

[Bepec]

Таки не заметил прикрепленного архива, извиняюсь.

to Old: да, таки проблема в архитектуре и как следствие реализация в коде.

И я тоже склоняюсь, что виноваты потоки. Низя обращаться к простому списку из двух разных потоков. А если и обращаетесь, то мутексами защищайте чтоли.

[Igors]

Хммм... а что и как засисять? Налепить QMutexLocker во всех методах? Это выглядит уродливо но ничего не гарантирует - могут найтись неперекрытые базовые методы, а то и вообще заклинит

[Old]

могут найтись неперекрытые базовые методы, а то и вообще заклинит

Что значит "могут найтись": небыло-небыло, а потом раз и нашлись? Нужно постараться не забыть перекрыть.
А вообще утверждать работоспособность модели в не gui-потоке я не могу, все никак руки не дойдут попробовать. Я долгие операции выполняю в рабочих нитках, и по сигналу добавляю информацию в модель уже из контексте gui-нитки.

[Bepec]

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

PS да, просто мутекс локеры натыкать и всё становится хорошо.

[Igors]

Ладно, попробуем, ну не с мутексами конечно. В аттаче измененные файлы. Вроде "не летит", но увы - это еще ничего не значит.  

Не найдется...

Свободен

[YvenTitan]

В реальной задаче добавлять и удалять строку надо чаще, там приходит много обновлений. Это связано с тем, что они приходят с датчиков. Это простой пример, чтобы немного понятно было.

Мьютексы я пробовал делать, но не помогало. Если кто конкретно скажет, куда их поставить, то буду благодарен. В данном тестовом примере поможет qSleep(1) после beginInsertRows и removeInsertRows, но в реальном примере это тоже бессильно.

Мне кажется, что проблема в следующем. При вызове функции beginInsertRows испускается сигнал rowsAboutToBeInserted(), по нему объект QTableView выполняет в потоке вставку строк в себя. При endInsertRows испускается rowsInserted. Аналогично при вызове beginRemoveRows выделяется сигнал rowsAboutToBeRemoved и в потоке QTableView удаляет из View строку. Как синхронизировать эти потоки - для меня главный вопрос. Если где-то неправ - поправьте.

[Bepec]

Система сигнал слотов сама разбирает проблему с потоками, хотя в вашем случае хзкак оно работает

Если поставить QMutexLocker в функции data, insertRows, removeRows, то в данном конкретном проекте вылеты прекращаются.

to Igors: пора бы знать, что если никто не знает об объекте, то и обращаться к нему не могёт, так что "найтись" не могут моменты. Разве что сам напишешь.

[Old]

В реальной задаче добавлять и удалять строку надо чаще, там приходит много обновлений. Это связано с тем, что они приходят с датчиков. Это простой пример, чтобы немного понятно было.

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