Как правильно обрабатывать изображения в стороннем потоке?

Question

[Гиббон Чо]

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

DWORD id;
HANDLE thread = CreateThread(NULL, 0U, MainThread, (LPVOID)td1,THREAD_PRIORITY_NORMAL, &id);
CloseHandle(thread);

Так создаю bitmapы и тд:

Vcl::Graphics::TBitmap *bmp1 = new Vcl::Graphics::TBitmap();
	Vcl::Graphics::TBitmap *bmp2 = new Vcl::Graphics::TBitmap();
	TPngImage *png = new TPngImage();
	TJPEGImage *jpeg = new TJPEGImage();
	Rxgif::TGIFImage *gif = new Rxgif::TGIFImage();
	TPicture *pic = new TPicture();
	TPicture *picTrash = new TPicture();

Так вот работаю с ними:

bmp1->Canvas->Lock();
			bmp2->Canvas->Lock();
			bmp2->Canvas->FillRect(bmp2->Canvas->ClipRect);
			bmp2->Canvas->Draw(0,0,bmp1);
			bmp2->Canvas->Unlock();
			bmp1->Canvas->Unlock();

Главная форма во время работы дико подвисает, и проц грузит почти на 100%, но есть же программы, которые быстро и без зависов обрабатывают изображения.

Answer 1

[Mercury13]

Что я пока вижу.
1. В любой высокоуровневой библиотеке CreateThread использовать ЗАПРЕЩЕНО, вместо этого используй beginthreadex (или что-то ещё, предоставленное библиотекой). Или, раз уж ты по-чёрному используешь VCL, особо не убудет, если будешь использовать TThread.
2. Lock и Unlock — это обыкновенный мьютекс. Все конкурирующие потоки, кому захочется рисовать на холсте, ждут и курят, ничего не делая.
3. Самое-то главное ты упустил. Если ждёшь помощи, не нужно так секретничать, что не поймёшь, в чём дело. Где — хотя бы примерно — тело потока, и что делается в главном потоке?
4. А как надо работать? Есть некая структура данных, в которой мы возвращаем из потока обработанные картинки, и volatile bool isWorking, который отвечает за то, работает поток или нет. Когда этот флажок false, главный поток имеет право работать с нашей структурой. Он переключается в true — в структуре может быть любой мусор, обращаться к ней запрещено! Если ещё и надо предупредить, когда поток закончился и можно забирать информацию — TThread.Synchronize или PostMessage. То и другое работает только в GUI-программах, если консольная — то через события (CreateEvent).
5. А почему «нельзя этого делать»? Если партия приказала, значит, надо. Просто многопоточность — это тяжёлое дело, ошибки на каждом шагу, но «принципиально нельзя» — нет такого.

Comments

[Гиббон Чо]

После отправки вопроса сюда еще много читал и экспериментировал. Сейчас упрощаю функцию, которая обрабатывается в потоке и исправляю косяки по мере их нахождения. Первый косяк - В функцию потока передаю Jpeg и сохраняю его оттуда же, все хорошо (и в несколько потоков и в один), потом вместо сохранения пишу bmp->Assign(jpeg); и сохраняю bmp. В один поток все прекрасно, но в несколько картинки сохраняются искаженными. Если делать все то же самое через TThread Synchronize, то картинки сохраняются замечательно, но форма висит намертво во время выполнения.

[Mercury13]

Что ты, извини, делаешь в Synchronize? Он только для коротких операций, т.к. просто выполняет что-то в потоке VCL.

[Гиббон Чо]

Mercury13: Слушай, я согласен с тем, что не знаю как мне правильно сделать и мне не хватает знаний, за этим я сюда и пришел, ты можешь подсказать или объяснить как мне лучше сделать? Мне нужно обрезать изображения в потоке например. то есть в потоке bmp->assign(jpeg); делаем операции с bmp->canvas, затем обратно jpeg->assign(bmp) и сохранить.

[Mercury13]

Скорее всего, ты устроил гонку между VCL и потоком. Я-то напишу вечером, когда вернусь, но почти уверен, что когда будешь восстанавливать прикладную функциональность, гонка снова начнётся.

Вопрос: что ты передаёшь внутрь потока и что из него возвращаешь? Имена файлов? Набор TJpeg?