ich möchte über die serielle Schnittstelle mit einem Modem kommunizieren.
Das Problem ist nicht die Verbindung, sondern die Übertragung der Befehle.
So sieht meine Funktion aus:

if(m_bConnectStatus == TRUE)
	{
		char* atCommand = "AT";

		port.WriteByte((unsigned char)atCommand);
	}

Ich möchte also nur einmal den AT Befehl schicken, um als Antwort "OK" zu bekommen. Die Funktion WriteByte sieht so aus:

BOOL CSerialPort::WriteByte(BYTE bybyte)
{
	iBytesWritten=0;
if(WriteFile(hComm,&bybyte,1,&iBytesWritten,NULL)==0)
return false;
else return true;
}

Die Klasse CSerialPort hab ich mir heruntergeladen.

Kann mir jemand helfen

MfG
Sebastian

Mein Problem ist folgendes:

Ich habe eine Verbindung über den COM Port zu meinem Handy hergestellt und möchte Daten senden bzw. empfangen. Dazu gibt es die sog. AT-Befehle (Hayes-Befehlssatz) mit denen man das interen Modem des Handy ansprechen kann.
Wenn ich also z.B. per Hyperterminal den Befehl "AT" eingebe, antwortet das Handy mit "OK".
Jetzt weiß ich nur nicht so recht, wie ich den Befehl angeben soll.
Mit dieser Funktion müsste das gehen:

BOOL CSerialPort::WriteByte(BYTE bybyte)
{
	iBytesWritten=0;
if(WriteFile(hComm,&bybyte,1,&iBytesWritten,NULL)==0)
return false;
else return true;
}

Wobei ja immer nur ein Zeichen geschrieben wird.

Also die Frage: Wie muss ich den Befehl/die Daten senden und was muss ich beim empfangen beachten (Pause einhalten/Timer?)?

MfG
Sebastian

Tschau

BOOL CSerialPort::WriteByte(const char* bybyte)
{
	iBytesWritten=0;
if(WriteFile(hComm,bybyte,strlen(bybyte),&iBytesWritten,NULL)==0)
	return false;
else return true;
}

Jetzt stellt sich für mich die Frage des Empfangens?
Denk mir mal, dass ich die Bytes/Char einzel empfangen muss und dann in einen String zusammenfügen,oder? Deshalb sieht meine Funktion im Moment so aus:

char CSerialPort::ReadByte(char text)
{
	ReadFile(hComm, &text, 1, &received, 0);

	return(text);	 		 
}

Aber so funzt es net wirklich?! Wie kann ich es besser machen?

Gruß
Sebastian

folgendes würde funzen:

char CSerialPort::ReadByte() 
{ 
    ReadFile(hComm, &text, 1, &received, 0); /* ist received und hComm global???*/
    return text;               
}

aber wenn man alle verfügbaren zeichen kriegen will:(hier mit Call-By-Reference)
im Hauptprgramm:

char * text;

und in der Klasse:

void CSerialPort::ReadByte( char * text) 
{ 
    ReadFile(hComm, &text, 100 /*denke das reicht...*/, &received, 0); /* ist received und hComm global???*/
    return text;               
}

ReadFile kehrt zurück, wenn keine Zeichen mehr anliegen... Deshalb sollte das mit 100 oder X funktionieren...

Tschau

void CHandytoolDlg::OnSendat() 
{
	// TODO: Code für die Behandlungsroutine der Steuerelement-Benachrichtigung hier einfügen

	if(m_bConnectStatus == TRUE)
	{
		const char* atCommand = "AT\n";  // bringt mir an dieser Stelle \n ein 'ENTER' ?
		port.WriteByte(atCommand);
	}

	Sleep(1000);

	char* response, *display;	

	display = port.ReadByte(response);

}

Ich möchte das so machen, wie es im Hyperterminal funzt. Man gibt AT ein und bestätigt mit 'ENTER' - danach kommt die Antwort OK.

Also sende ich den Befehl an den COM Port:

BOOL CSerialPort::WriteByte(const char* bybyte)
{
	unsigned long size = strlen(bybyte);
	iBytesWritten=0;
	if(WriteFile(hComm,bybyte, size,&iBytesWritten,NULL)==0)
		return false;
	else return true;
}

Da es dann ja einige ms dauert bis die Antwort am COM Port liegt hab ich mal nen Sleep (großzügig) eingebaut. Bringt das was?

Die Funktion zum auslesen:

char* CSerialPort::ReadByte(char* text)
{
	DWORD dwBytesRead;	

	ReadFile(hComm, text, 100, &dwBytesRead, 0);

	return text;	 		 
}

hComm ist global.

Kann das alles so überhaupt funktionieren?

Gruß
Sebastian

Mit SetCommTimeouts könntest du dafür sorgen, dass ReadFile vorher zurückkommt.

NACHTRAG:
Es ist nicht sinnvoll, den Zeiger auf den Puffer zurückzugeben, der ist ja schon Parameter. Gib lieber die Anzahl der empfangenen Zeichen zurück.

void CHandytoolDlg::OnSendat() 
{
	// TODO: Code für die Behandlungsroutine der Steuerelement-Benachrichtigung hier einfügen

	if(m_bConnectStatus == TRUE)
	{
		const char* atCommand = "AT+CMGR=1\n";
		port.WriteByte(atCommand);
	}

	DWORD data = EV_RXCHAR;
	if(WaitCommEvent(port.hComm, &data, NULL) == 0)
		MessageBox("Es liegen keine Daten an.","Fehler",MB_OK+MB_ICONINFORMATION);
	else
	{
		char* response, *display;	
		display = port.ReadByte(response);
	}	
}

EV_RXFLAG The event character was received and placed in the input buffer. The event character is specified in the device's DCB structure, which is applied to a serial port by using the SetCommState function.

Irgendwie komm ich damit noch nicht so ganz klar. Ist mein erstes Projekt mit der seriellen Schnittstelle und ich weiß auch nicht so recht, wie und wo ich die Fehler suchen soll. Aber ich denke, dass genau da der Fehler liegt, weil die Verbindung an sich ja steht.

Vielleicht kannst du mir ja nen Rat geben

Gruß
Sebastian

Wenn der Rückgabewert von Null verschieden ist, musst du den Inhalt von data prüfen.

void CHandytoolDlg::OnSendat() 
{
	// TODO: Code für die Behandlungsroutine der Steuerelement-Benachrichtigung hier einfügen

	if(m_bConnectStatus == TRUE)
	{
		char* atCommand = "AT+CMGR=1\r";
		port.WriteByte(atCommand);
	}

	DWORD data;
	if(WaitCommEvent(port.hComm, &data, NULL) == 0)
		MessageBox("Es liegen keine Daten an.","Fehler",MB_OK+MB_ICONINFORMATION);
	else
	{
		if(data == EV_RXFLAG)
		{
			char* response, *display;	
			display = port.ReadByte(response);
		}
	}	
}

Aber irgendwie liefert die Funktion immer nur 0 zurück.

Woran könnte das liegen?

Gruß

In jedem Fall solltest du GetLastError aufrufen, wenn WaitCommEvent fehlschlägt.

hab jetzt mal meine Funktionen mit GetLastError() gecheckt und es funktioniert auch alles bis auf die ReadByte() Funktion. Hier ist sie nochmal:

char* CSerialPort::ReadByte(char* text)
{
	DWORD dwBytesRead;	

	ReadFile(hComm, text, 100, &dwBytesRead, 0);

	DWORD dwErrorReport;
	dwErrorReport = GetLastError();

	return text;	 		 
}

Die Fehlermeldung die ich bekomm ist:
Unzulässiger Zugriff auf einen Speicherbereich. [998]

Kann mir da jemand weiterhelfen?

Gruß
Sebastian