Multithread in C++ in Klassen

[FlorianKr]

Hallo Community,

nachdem ich schon seit längerem mitgelesen habe, habe ich nun auch mal ein Problem an dem ich nicht weiter komme.
Ich hoffe ich habe das richtige Unterforum.

Compiler: Visual Studio 2013 C++

Im Rahmen einer Projektarbeit sollen wir an der FH den VolksBot des Fraunhofer Institut in bewegung setzen.
Zum auslesen der seriellen Schnittstellen wollte ich eigene Threads benutzen.

CommSeriell::CommSeriell(int a)
{
	printf("\nAlle Threads starten...\n");

	// Alle Threads starten
	int index = 1;
	hThread[0] = CreateThread(NULL,						// Sicherheitsinfos
		0,							// Größe des Stack
		CommSeriell::RecieveGPS,				// Threadfunktion
		(LPVOID)index,			// 32-bit paramter der Threadfunktion übergeben wird
		0,							// Creation Flags, 0=sofort starten.
		&dwThreadID[0]);	// Id speichern
}

DWORD WINAPI CommSeriell::RecieveGPS(LPVOID data)
{
	initComPort();  //Öffnen des ComPort
	recieveComPort();  //Empfangen und auswerten der Daten
	CloseHandle(hComm);//Closing the Serial Port  //Schliesen des ComPort

	return((DWORD)data);
}

Ich würde gerne in einer Klasse einen Thread starten bekomme das allerdings nicht hin.
Habe jetzt schon des öfteren gelesen dass man hierbei keine Funktion einer Klasse verwenden kann.
Allerdings benötige ich mehrere Daten aus der Klasse und würde auch gerne mehrere Daten zurückgeben.

Kennt hierzu jemand einen Lösungsweg oder eine Alternative?

MfG
Florian

[Mxt]

Hallo,

und willkommen im Forum.

Habe jetzt schon des öfteren gelesen dass man hierbei keine Funktion einer Klasse verwenden kann.

Das geht schon, allerdings hat man dabei das Problem, dass Memberfunktionen ja implizit this als ersten Parameter übergeben bekommen. Man muss da eventuell mit einer statischen Memberfunktion und allerlei Casts aushelfen. Das kriege ich jetzt aus dem Kopf nicht hin, müsste ich auch nachschauen.

Modernes C++ kennt Multithreading. Das geht auch in VC++ 2013
http://en.cppreference.com/w/cpp/thread
wir hatten da kürzlich hier mal was zu
https://www.roboternetz.de/community...ghlight=thread

Wenn es unbedingt die WinAPI sein muss, wie in deinem Code oben: In den Büchern von Jeffrey Richter ("Windows via C/C++") steht, man solle in C++ CreateThread nie direkt aufrufen, weil dabei einige Dinge der Laufzeitbibliothek nicht initialisiert werden. Ob das immer noch so ist, kann ich nicht sagen. Ich würde trotzdem seine Empfehlung weitergeben, besser _beginthread oder _beginthreadex zu nehmen. Microsoft nimmt heute in seinen eigenen Beispielen aber eher std::thread.

[schorsch_76]

Du kannst eine Thread Funktion in einer Klasse nutzen. Hier ein Bsp.

Test::Test()
{
	// create the worker thread
	m_run.store(true);
	m_thread = std::thread(std::bind(&Test::execute, this));
}
Test::~Test()
{
	m_run.store(false);
	if (m_thread.joinable()
	{
		m_thread.join();
	}

}
void Test::execute()
{

	while (m_run.load())
	{
	}
}

class Test
{
public:
	Test();
	~Test();
private:
	// the thread function
	void execute();

	// the thread
	std::atomic<bool> m_run;
	std::thread m_thread;
};

[FlorianKr]

Hallo,
vielen Dank für die raschen Antworten. Habe mir mal die links angeschaut und auch die Codes aus dem anderen Thread.
Desweiteren habe ich noch auf VS2015 umgestellt.

Gerade habe ich mal versucht meinen Code nach dem Beispiel von schorsch_76 umzuschreiben,
bekomme dabei allerdings den Fehler "C2039 'atomic': is not a member of std" gefolgt von weiteren Fehlern in dieser Zeile.
Desweiteren spuckt er bei jedem m_run einen Fehler aus, was aber denke ich daran liegt dass er die Deffinition nicht nimmt.

	std::atomic<bool> m_run;

Anbei noch mein geänderter Code:

//CommSeriell.cpp

CommSeriell::CommSeriell()
{
	m_run.store(true);
	m_thread = std::thread(std::bind(&CommSeriell::theThread, this));
}


CommSeriell::~CommSeriell()
{
	m_run.store(false);
	if (m_thread.joinable()
	{
		m_thread.join();
	}
}

void CommSeriell::theThread()
{
	initComPort();
	while (m_run.load())
	{
		recieveComPort();
	}
	
	CloseHandle(hComm);//Closing the Serial Port
	printf("\n +==========================================+\n");
};

//CommSeriell.h

#pragma once
class CommSeriell
{
public:
	CommSeriell();
	~CommSeriell();

private:
	std::atomic<bool> m_run;
	std::thread m_thread;

	HANDLE hComm;                          // Handle to the Serial port
	wchar_t  ComPortName[7];  // Name of the Serial port(May Change) to be opened,
	BOOL  Status;                          // Status of the various operations 
	DWORD dwEventMask;                     // Event mask to trigger
	char  TempChar;                        // Temperory Character
	char  SerialBuffer[4096];               // Buffer Containing Rxed Data
	DWORD NoBytesRead;                     // Bytes read by ReadFile()
	int i = 0;
	int error = 0;

	BOOL initComPort();
	BOOL setRecieveMask();
	BOOL recieveComPort();
	BOOL readNMEA();
	void theThread();

};

MfG
Florian

[Mxt]

bekomme dabei allerdings den Fehler "C2039 'atomic': is not a member of std" gefolgt von weiteren Fehlern in dieser Zeile.

Da fehlt dann wohl ein #include <atomic>

[Unregistriert]

Oh ja das war der fehler, danke.

Allerdings bringt er mir jetzt noch einen letzten Fehler:

Error C2280 'CommSeriell::CommSeriell(const CommSeriell &)': attempting to reference a deleted function ConsoleApplication1 c:\program files (x86)\microsoft visual studio 14.0\vc\include\type_traits 1466

Beim Doppelklick darauf öffnet sich eine Datei mit dem Namen type_traits.

Florian

[FlorianKr]

Habe meinen Fehler gefunden.
Hatte noch einen alten Threadaufruf in der Hauptschleife vom ausprobieren.

Jetzt funktioniert das ganze so wie ich es mir vorgestellt habe.

Vielen Danke für die schnelle Hilfe
Florian