Probleme mit strstr, strcmp und RS232 communication

[Genadi D. (gddilm)]

Hallo,

ich bin Anfänger mit dem Programmieren, ich wollte Kommunikation 
zwischen einer Steuerung(Atmel Chip) und einem 
Microblaze-Prozessor(Xilinx) erstellen. Dafür wollte ich erst ganz 
einfach die Steuerung beim Initializieren erkennen. Die Steuerung 
schickt seine Initializierungszeilen und ich wollte entweder mit 
"strstr" oder mit "strcmp" nach einem Wort suchen. Das Problem ist es 
funktioniert nicht ganz richtig und ich verstehe nicht warum. Hier mein 
Code im File und 2 Bilder, welche zeigen wie ich die 
Initializierungszeilen empfange. Das Bild1 bezieht sich auf 
xil_printf("%s",receivedBufferFromMotor); Bild2 auf 
xil_printf("%s/n",receivedBufferFromMotor);. Ich habe es gemacht um zu 
sehen wie ich die Bytes in Wirklichkeit bekomme. Etwas stimmt aber nicht 
weil die Bilder  nicht identisch sind und damit meine ich nicht "/n". 
Ich will wenn es möglich ist auch nach dem Wort READY! oder M1F suchen.

Danke im Voraus!!!

[Karl H. (kbuchegg)]

Genadi D. schrieb:

> "strstr" oder mit "strcmp" nach einem Wort suchen. Das Problem ist es
> funktioniert nicht ganz richtig und ich verstehe nicht warum.

Dein Problem ist, dass du davon ausgehst, dass du hier

         receivedBytesCountFromMotor = XUartLite_Recv(&RS232_1, receivedBufferFromMotor, fifoBytesSize);
          

                                if (receivedBytesCountFromMotor != 0)

eine komplette Zeile bekommst.
Dem ist nicht so.

Du bekommst das zurück, was zu diesem Zeitpunkt bereits in der UART war. 
Aber das muss nicht notwendigerweise eine komplette Zeile sein.

Eine komplette Zeile hast du dann, wenn du das <CR><LF> empfangen hast. 
Erst dann macht es Sinn die auszuwerten.

Du musst also solange Zeichen von der UART holen und in einem char-Array 
zusammensetzen, bis du in den Daten ein <CR><LF> findest (in C 
schreibweise also ein \r\n). Erst dann hast du deine Zeile.

[Genadi D. (gddilm)]

Du hast deine Gedanken sehr gut formuliert, genau das wollte ich fragen, 
wie kann ich es so machen dass ich erst nach dem Empfang alle Bytes oder 
die ganze Information meine Strings suche. Was ich noch sagen kann ist 
dass, jetzt suche ich nach dem "F" was nur ganz am Ende kommt bei "M1F", 
und es funktioniert, aber ich weiß das es nicht schön und auch nicht 
ganz richtig ist so zu machen.

[Karl H. (kbuchegg)]

Genadi D. schrieb:
> Du hast deine Gedanken sehr gut formuliert, genau das wollte ich fragen,
> wie kann ich es so machen dass ich erst nach dem Empfang alle Bytes


In dem du alle Bytes auch empfängst!

Zu Deutsch. Es reicht nicht einmalig XUartLite_Recv aufzurufen!

Bei jedem Aufruf kriegst du ein paar Zeichen. Aus dem Returnwert weisst 
du, wieviele das sind. Die hängst du an deinen String an (der natürlich 
am Anfang leer war). Und erst dann, wenn das letzte Zeichen ein CR war 
(ein \n) erst dann weisst du, dass du jetzt den String komplett hast.

Das dir XUartLite_Recv die Möglichkeit gibt, einen größeren Buffer 
anzugeben, ist zwar nett, aber in deinem Fall eher hinderlich. Es ist 
einfacher, wenn du die Zeichen einzeln holst.

So in etwa

char ReceiveBuffer[200];
int  ReceiveBufferCount;

char tmp;
...

   ReceiveBufferCount = 0;
   while( 1 ) {

     receivedBytesCountFromMotor = XUartLite_Recv( &RS232_1, &tmp, 1 );

     if( receivedBytesCountFromMotor > 0 ) {
       if( tmp == '\r' )
         ; ignorieren

       else if( tmp != '\n' ) {     // normales Zeichen
         ReceiveBuffer[ReceiveBufferCount++] = tmp;
       }

       else {                       // bleibt nur noch \n
                                    // wenn \n kommt, ist der String komplett
         ReceiveBuffer[ReceiveBufferCount] = '\0';

         xil_printf("Hab verstanden: %s\n",ReceiveBuffer);
         ReceiveBufferCount = 0;
       }
     }
  }