Serielleschnittstelle RS232 C++

warum verwendest du *.txt für c++ dateien?

Das lesen sieht eigentlich gut aus. Ein wait (oder besser Sleep) ist 
nicht notwendig. Aber du beachtest nicht das das die Anzahl die Maximale 
anzahl von bytes ist, das read return dir anzahl der gelesen bytes. Wenn 
also 5 byte ankommen dann kann es schon passieren das read dann schon 
beendet. Damit hast du aber nicht die 14 byte gelesen.

Was macht eigentlich der .net code zwischen deinem C++ code? Oder ist 
das C++.net

Fox Xx schrieb:
> lso in der Beschreibung steht das die Hardware mir 14 Byte als Antwort
> schickt. Hast du eine Lösung oder Ansatz für mich?

ja, warte bis du 14 byte zuammen hast. Wenn du erst 4 byte ampfangen 
hast dann musst noch auf weitere 10byte warten.

Fox Xx schrieb:

> gut und nun wartet er bis die kommen restlichen bytes kommen, wenn die
> natürlich nie kommen steht das programm.
> wie löse ich das problem
dann gibt es ein timeout, den kann man bei der schnittstelle einstellen.

>in c# ist das arbeiten mit der rs232 so schön einfach.
nein es ist dort genauso, wenn du es nicht beachtest hast dann hattest 
du bis jetzt glück das es funktioniert

http://msdn.microsoft.com/de-de/library/ms143549.aspx

Hex Oschi schrieb:
> Ein vernuenftiges Protokoll hilft da eine Menge
wenn keine Daten mehr kommen, hilft das Protokoll auch nicht mehr 
weiter.

Fox Xx schrieb:
> könnt ihr mir beim einstellen des timeouts helfen?

http://msdn.microsoft.com/en-us/library/aa363437%28v=vs.85%29.aspx

Hex Oschi schrieb:
> Doch, das Protokol hilft das sich die Software nicht blockiert. Man kann
> das Device dann ja neu booten und die software macht weiter.
nein denn ein Protokoll ist nur für den Gut-Fall da, wenn jemand den 
Stecker zieht hilft das Protokoll nicht weiter. Dann muss man mit 
sinnvollen Timeouts arbeiten und dann zu einem Status zurückgehen von 
den man neu beginnen könnte.

dafür muss man mehr wissen was du genau an daten erwartest bzw was von 
der gegenstelle gesendet wird.

Du hast gesagt das die Gegenstelle immer 14byte sendet. Dafür braucht 
man keinen Thread. Du sammelst einfach alle bytes bis du 14 hast und 
dann gibt du es zurück. Wenn du nach ein zu langen zeit nichts 
emfpfängst (über den Timeout festlege) dann muss du entscheiden wie es 
weiter gehen soll. In c++ ist es üblich eine Exception zu werfen.

Ich hab da auch mal nen Problem gehabt, man mußte zwingend nen 
BuildCommDCB ausführen damit die Kommunikation funktioniert. Weil ALLE 
Elemente müssen initialisiert werden.
Also alle initalisiern oder BuildCommDCB :
typedef struct _DCB {
  DWORD DCBlength;
  DWORD BaudRate;
  DWORD fBinary  :1;
  DWORD fParity  :1;
  DWORD fOutxCtsFlow  :1;
  DWORD fOutxDsrFlow  :1;
  DWORD fDtrControl  :2;
  DWORD fDsrSensitivity  :1;
  DWORD fTXContinueOnXoff  :1;
  DWORD fOutX  :1;
  DWORD fInX  :1;
  DWORD fErrorChar  :1;
  DWORD fNull  :1;
  DWORD fRtsControl  :2;
  DWORD fAbortOnError  :1;
  DWORD fDummy2  :17;
  WORD  wReserved;
  WORD  XonLim;
  WORD  XoffLim;
  BYTE  ByteSize;
  BYTE  Parity;
  BYTE  StopBits;
  char  XonChar;
  char  XoffChar;
  char  ErrorChar;
  char  EofChar;
  char  EvtChar;
  WORD  wReserved1;
} DCB, *LPDCB;

 Nen sch.. Fehler der mir tagelang kopfschmerzen gemacht hat.

Fox Xx schrieb:
> was musst du den noch genau wissen?

wie das übertragungsprotokoll aussieht. Du behauptest das immer 14 
Zeichen gelesen werden sollen, das wiederspricht sich mit

PortLesen() == ">"

denn das ist nur 1 Zeichen.

Dann ist es wichtig zu wissen ob das Gerät von sich aus sendet oder ob 
du eine Anfrage hinschickt und darauf immer eine Antwort kommt (und in 
welcher zeit).

Wegen UWE:
es ist auch einfach möglich mit GetCommState die alten werte auszulesen 
und dann die werte anzupassen und dann sie wieder mit SetCommState zu 
setzten dann kannst du auf jeden fall keinen Vergessen.

Fox Xx schrieb:
> Zeichen als Antwort zurück senden, beginnend mit ">".

steht eventuell auch etwas über das ende drin? Zeilenumbruch? 0byte? 
Ohne ein Definiertes ende ist das Protokoll müll. Könnte ja sein das 
nach 10min noch etwas kommt.

Fox Xx schrieb:
> wobei x für Zahlen als ASCII stehen
> also in c# hatte ich immer sofort die Antwort.

und woher weiss du das die entwort vollständig war? Schau doch mal nach 
ob am ende ein 0x13 oder 0x12 kommt.

naja dann los.

Schreibe eine funktion die daten von der Serielen schnittstelle abruft 
bis eine CR kommt und sie dann als String zurück liefert

std::string s;
char c;
while( true ) {
  size_t r = readfile( c, 1 ... );
  if ( r != 1 ) {
     throw Exception("Fehler beim empfang");
  }
  if ( c == '\n' ) {
     break;
  }

  s += c;
}

return s;

so in der Art, man könnte auch mehre zeichen gleichzeitig empfangen aber 
man muss dann aber eh einzeln schauen ob das ende erreicht ist - für den 
anfang sollte es so aber gehen.

http://msdn.microsoft.com/en-us/library/ms810467.aspx

Fox Xx schrieb:
> und wenn das CR nie kommt?
> bzw wenn überhaupt nichts kommt?
dann gibt es ja die exception - weil read nicht 1 zurückliefert. Wenn 
aber immer etwas kommt und nie ein \n dann geht dieser code nicht. Aber 
man könnte ja auch noch die länge begrenzen.

Fox Xx schrieb:
> ennt ihr evt eine Klasse die sich um alles kümmert?
> In der ich nur ein paar Methoden aufrufen muss wie "DatenLesen()"?

was ist denn so schwer daran die funktion so zu schreiben wie ich sie 
schon hingeschrieben habe? Einfacher geht es nun wirklich nicht.

Versuch es doch mit den Timeout die du oben gefunden hast, etwas passen 
sie oder du musst sie verlängern oder kürzen.

Das von UWE brauchst du nicht wenn du

GetCommState
-> parameter anpassen
SetCommState

machst, den rest kannst du alles so lassen.

Selbst ohne Timeouts geht es, du könnte es passieren wenn jemand den 
stecker zieht das dein programm dann hängt.

Hallo Fox Xx

in Deinem Code stimmt die Reihenfolge nicht.

DCB dcb;
dcb.DCBlength = sizeof(DCB);

zuerst Port-Config lesen
  GetCommState(m_PortHandle, &dcb)

// Konfigurationen des Ports setzen

wieso hier schon SetCommState und dann erst DCB-Parameter setzen?
Das  gehört hinter dcb.Binary=TRUE

*        if (SetCommState(m_PortHandle, &dcb))
*    {
*    m_StatusMeldung = "Fehler: Konnte Konfigurationen
*                nicht setzen!";
*      return false;
*    }
|
|        dcb.BaudRate    = m_Baudrate;      // Baud Rate
|        dcb.Parity      = m_Parity;        // Parity
|        dcb.ByteSize    = m_Databit;      // Data Bits
|        dcb.StopBits    = m_Stopbit;      // Stop Bits
|        dcb.fBinary = TRUE;
|--->

Fox Xx schrieb:
> wenn ich jetzt keine ASCII Zeichen senden will, sondern Hex bzw Dezimal,
> gibt es da eine Lösung?
ja.

Wo sieht du den unterschied zwischen ASCII Hex und Dezimal? Wenn du dir 
darüber klar bist, dann sollte es auch klar sein wie man sie sendet.