ich will meine Anwendung mit mehreren Threads austatten. Aus gründen der Effizienz wollte ich die Threads selbständig arbeiten lassen, d.h. jeder Thread holt sich selbst immer wieder neue Arbeit (er ruft eine Funktion auf, die ein Array ausfüllt) bis es keine mehr gibt. Erst dann soll das Programm sich beenden.

Deklarationen etc.:

struct WorkItem
{
	char *data;
        bool finished;
};

#define ITEMS_PER_CALL 20

So starte und beende ich die Threads:

InitializeCriticalSection(&cs);

	for (int i = 0; i < ThreadCount; i++)
	{
		Threads[i] = CreateThread(NULL, NULL, ThreadFunc, NULL, NULL, NULL);
	}

	WaitForMultipleObjects(ThreadCount, Threads, true, INFINITE);

	DeleteCriticalSection(&cs);
	for (int i = 0; i < ThreadCount; i++)
	{
		CloseHandle(Threads[i]);
		Sleep(20);
	}

Die Threadfunktion:

DWORD WINAPI ThreadFunc(LPVOID param)
{
	srand(GetTickCount());
	WorkItem newitems[ITEMS_PER_CALL];

	bool exit = false;
	while (1)
	{
		EnterCriticalSection(&cs);
		GetNewItems(newitems);
		LeaveCriticalSection(&cs);

		for (int i = 0; i < Items_PER_CALL; i++)
		{
			if (newitems[i].finished == true) 
			{
				exit = true;
				break;
			}

			EnterCriticalSection(&cs);
			cout << "Work-Data: " << newitems[i].data << endl;
			LeaveCriticalSection(&cs);
		}

		if (exit)
		{
			return 0;
		}
	}
	return 0;
}

Das ist die GetNewItems-Funktion:

void GetNewItems(WorkItem *newitems)
{
	EnterCriticalSection(&cs); // brauch ich das überhaupt hier?
	for (int i = 0; i < ITEMS_PER_CALL; i++)
	{
		if ((CurrItem+1) >= ItemCount) // ItemCount is die gesamt anzahl, CurrItem das gerade verwendete Item
		{
                     newitems[i].finished = true;
		}

		newitems[i].data = data[CurrItem]; //data is ne char** variable
                newitems[i].finished = false;
		CurrItem++;
	}
	LeaveCriticalSection(&cs); //brauch ich das hier?
}

Also das Problem ist, wenn ich nur 1 Thread nehme, geht alles bestens. Wenn ich aber mehr nehme, dann geht es manchmal, manchmal bricht er nach ein paar Aufträgen ab.

Achja, ich kann die WorkItems nich direkt speichern und zurückgeben, das ganze muss über CurrItem und so gehen.

Weiß jemand woran das liegen könnte?
Gruß pyro

void GetNewItems(WorkItem *newitems)
{
    EnterCriticalSection(&cs); // brauchst du entweder hier ODER in ThreadFunc (ist aber auch kein Problem wenn es an beiden Stellen steht)
    for (int i = 0; i < ITEMS_PER_CALL; i++)
    {
        if (CurrItem >= ItemCount) // wir markieren Items NACH dem letzten
        {
            newitems[i].finished = true;
            break;
        }

        newitems[i].data = data[CurrItem]; //data is ne char** variable
        newitems[i].finished = false;
        CurrItem++;
    }
    LeaveCriticalSection(&cs); // brauchst du entweder hier ODER in ThreadFunc

}

Sonst sehe ich auf die Schnelle keinen Fehler.

Es wäre auch hilfreich wenn du den Code posten würdest den du auch verwendest, möglichst ein minimales Beispiel das compilierbar und ausführbar ist, mit dem der Fehler reproduzierbar ist. Der Code den du gepostet hast kann garnicht compilieren (es sei denn "Items_PER_CALL" ist kein Tippfehler sondern du hast wirklich irgendwo "Items_PER_CALL" UND "ITEMS_PER_CALL" definiert).

"Zusammengekürzte" Beispiele haben oft andere Probleme bzw. u.U. kürzt du sogar den Fehler raus, und dann können wir dir nicht helfen.