Unhandled exception при QFile::copy

[Disaron]

Привет всем.

Собственно сабж на котором сломал голову. Есть многопоточная прога, которая в несколько потоков перекладывает файлы между разными ресурсами.
Вся беда в том, что в один прекрасный момент она вылетает к хренам и я никак не могу отловить на какой стадии. Может 100 раз переложить одни и те-же файлы, а может 3 и вылететь. Дебаг показывает сегфолт и смерть потока (последнее время вообще gdb отказывается запускаться - разбираюсь). Весь процесс c файлами происходит в TaskExecuter.

ЗЫ: Замечено, что в это время наблюдается активность монитора антивируса - может в этом дело, но тогда это баг Qt? Или все-таки я недостаточно еще знаком со всеми подводными камнями MT и где-то дико накосячил?

В общем прошу помощи у аудитории, я уже, чувствую, замылил взгляд...

XP + Qt 4.8.4 + MinGW.

[carrygun]

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

[Disaron]

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

Спасибо. Завел отладчик и проблема, как выяснилось, была совсем не в бобине, а вот в этом коде:

Код

C++ (Qt)
QList<JobData>::iterator it = m_jobList.begin();
while (it != m_jobList.end()) {
    if ((*it).state == JS_DONE) {
        m_jobList.removeOne(*it); 
    }
    ++it;
}

//И как вообще такое в голову пришло?!

Замена на некошерный

Код

C++ (Qt)
foreach (JobData jb, m_jobList) {
    if (jb.state == JS_DONE)
        m_jobList.removeAt(m_jobList.indexOf(jb));
}
 

работает.

[carrygun]

Для кутэшных контейнеров foreach и не такая уж плохая штука.

[GreatSnake]

Для кутэшных контейнеров foreach и не такая уж плохая штука.

Только вот эту штуку не мешало бы с умом применять, а не как автор.

[Пантер]

Я бы std::remove_if заюзал.

[Igors]

Код

C++ (Qt)
foreach (JobData jb, m_jobList) {
    if (jb.state == JS_DONE)
        m_jobList.removeAt(m_jobList.indexOf(jb));
}
 

Зачем использовать затратный indexOf, лучше так

Код

C++ (Qt)
int i = 0;
while (i < m_jobList.size()) {
    if (m_jobList[i].state == JS_DONE)
        m_jobList.removeAt(i);
    else
        ++i;
}
 

Для кутэшных контейнеров foreach и не такая уж плохая штука.

Лично я не вижу в нем ничего хорошего. Конечно пользоваться никто не заставляет, дело вкуса

Я бы std::remove_if заюзал.

В данном случае remove_if не очень хорошо 

[Пантер]

Я бы std::remove_if заюзал.

В данном случае remove_if не очень хорошо 

Почему?

[Igors]

Почему?

Код с removeAt удаляет объекты и сдвигает хвост массива указателей (внутри QList). А remove_if переставляет/обменивает сами элементы вызывая оператор присваивания. Это куда хуже для элементов - приличных структур.

[Igors]

Строго говоря и removeAt не оптимально. Думаю правильно так

Код

C++ (Qt)
#include <QtGui>
 
template <class T, class TShouldDel>
void QListRemoveIf( QList<T> & lst, TShouldDel shouldDel )
{
	int i = 0, last = lst.size();
	while (i < last) {
		if (shouldDel(lst[i]))
			lst.swap(i, --last);
		else
			++i;	
	}
 
	lst.erase(lst.begin() + last, lst.end());
}
 
bool MyShouldDel( int val )
{
	return val == 1;
}
 
int main( void ) 
{
	QList <int> test;
	test << 2 << 3 << 1 << 9;
	qDebug() << test;
 
	QListRemoveIf(test, MyShouldDel);
	qDebug() << test;
 
	return 0;
}
 

QList::swap - умный метод, переставляет только указатели

Update: и кстати о птичках - после такого удаления адреса всех оставшихся элементов не меняются

[Пантер]

На больших наборах данных согласен.

[twp]

а чем не подошел erase?

Код

C++ (Qt)
QList<JobData>::iterator it = m_jobList.begin();
while (it != m_jobList.end()) {
    if ((*it).state == JS_DONE)
        it = m_jobList.erase(it); 
    else
        ++it;
}
 

[Disaron]

Зачем использовать затратный indexOf, лучше так

Код

C++ (Qt)
int i = 0;
while (i < m_jobList.size()) {
    if (m_jobList[i].state == JS_DONE)
        m_jobList.removeAt(i);
    else
        ++i;
}
 

Лучше, с т.з. затрат - в последнем варианте так и есть. Там вообще много чего по-другому, включая конструирование потоков "по фэн-шуй".
Кстати человеку, первый раз читающему код на Qt-контейнерах, совсем не очевидно, что removeAt сдвигает массив, поэтому возникнет вопрос "а почему оно работает?". Придется лезть в доки, а то и в код Qt. Мне, пишущему от случая к случаю, да на коленке, и то пришлось вспомнить это.
Вопрос производительности на момент исправления не стоял - был вопрос быстрого исправления бага.

Но это так - отмазки...

ЗЫЖ foreach ИМХО этакий удобный костыль, с которого потом надо лишний раз слезать, жду повсеместной поддержки for (T x : list<T>) {}

[carrygun]

ЗЫЖ foreach ИМХО этакий удобный костыль, с которого потом надо лишний раз слезать, жду повсеместной поддержки for (T x : list<T>) {}

А разве последний мингв не умеет?

[Disaron]

А разве последний мингв не умеет?

Умеет, и не только последний. Но его прикрутить надоть, скачать - а это лень...