USART: Merkwürdiges Sendeverhalten

[Dennis S. (dennnis)]

Hallo Zusammen,

ich habe folgendes Problem und weiß nicht weiter:
Ich sende über den PC einen Befehl an meinen MC, der sieht z.B. so aus: 
LED001;
Über den USART-Interrupt lasse ich den String zerlegen und will ihn im 
Grunde zum Test direkt wieder zurück schicken.
Hier der entsprechende Codeabschnitt:

volatile char uart_string[11] = "";
volatile uint8_t uart_str_count = 0;

ISR(USART_RXC_vect)
{
  unsigned char nextChar;
  nextChar = UDR;
  //SendChar(nextChar);
  if( nextChar != ';')
  {  //String ist noch im Aufbau
    uart_string[uart_str_count] = nextChar;
    uart_str_count++;
  }
  else 
  {   //String ist Komplett!
    uart_string[uart_str_count] = '\0';
    uart_str_count = 0;
    unsigned char Empfaenger[3];
    unsigned char Wert[3];
    
      Empfaenger[0] = uart_string[0];
      Empfaenger[1] = uart_string[1];
      Empfaenger[2] = uart_string[2];
      Empfaenger[3] = '\0';
        
      Wert[0] = uart_string[3];
      Wert[1] = uart_string[4];
      Wert[2] = uart_string[5];
      Wert[3] = '\0';
      
      sendstring(Empfaenger);
      sendstring(Wert);
  }
}


void sendstring(char *s)
  {
     while (*s)
     {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" */
       SendChar(*s);
       s++;
     }
  }
  
void SendChar(unsigned char c)
{
  while ( !( UCSRA & (1<<UDRE)) );
  UDR = c;
  return 0;
}

Das Problem ist nun das ich am PC nur den "Wert" empfange, ohne das 
vorherige "Empfaenger". Wenn ich es aber z.B. so schreibe:

sendstring(Empfaenger);
sendstring(Empfaenger);

oder so:

sendstring(Wert);
sendstring(Wert);

kommt alles passend (doppelt) an.
Drehe ich es um:

sendstring(Wert);
sendstring(Empfaenger);

kommt nur der "Wert" an, "Empfaenger" wird wieder verschluckt.

Hat Jemand eine Idee?

[Reinhard R. (reinhardr)]

Vermutlich keine Erklärung für deinen Fehler, aber du mischt "char" und 
"unsigned char" wenn es um die Stringverarbeitung geht.

Etwas Offensichtliches fällt mir jetzt nicht auf. In so einer Situation 
bereinige ich gerne das Projekt (z.B. make clean) und erstelle es neu. 
Das hilft meistens zwar nicht (weil der Fehler im Code liegt), aber wenn 
es hilft hat man meistens schon lange an anderen Stellen gesucht. ;-)

Gruß
Reinhard

[Karl H. (kbuchegg)]

    unsigned char Empfaenger[3];
    unsigned char Wert[3];
    
      Empfaenger[0] = uart_string[0];
      Empfaenger[1] = uart_string[1];
      Empfaenger[2] = uart_string[2];
      Empfaenger[3] = '\0';

zähl nach.
Du hast 4 Zuweisungen an Empfänger, hast aber Empfänger nur mit einer 
Größe von 3 Elementen definiert.

Du überlaufst dein Array, dasselbe mit 'Wert'.

Je nachdem, wie der Compiler die Arrays im Speicher angeordnet hat, 
beschriebst du mit Wert[3] eigentlich Empfänger[0]. Und damit hast du 
dir den String dort gleich an der ersten Stelle gekappt.

[Reinhard R. (reinhardr)]

Karl Heinz Buchegger schrieb:
> zähl nach.

Autsch! So viel zu meiner Aussage mit "nichts Offensichtliches".

Duck und weg,
Reinhard

[Dennis S. (dennnis)]

Tatsache:

unsigned char Empfaenger[4];
    unsigned char Wert[4];

      Empfaenger[0] = uart_string[0];
      Empfaenger[1] = uart_string[1];
      Empfaenger[2] = uart_string[2];
      Empfaenger[3] = '\0';

      Wert[0] = uart_string[3];
      Wert[1] = uart_string[4];
      Wert[2] = uart_string[5];
      Wert[3] = '\0';

      sendstring(Empfaenger);
      sendstring(Wert);

funktioniert. Besten Dank.
Ich war der Auffassung das die Größe [3] insgesamt 4 Zeichen zulässt, 
also 0..3.

Naja, so kann man sich täuschen.

Vielen Dank

[Karl H. (kbuchegg)]

Dennis Scholz schrieb:

> Ich war der Auffassung das die Größe [3] insgesamt 4 Zeichen zulässt,
> also 0..3.
>
> Naja, so kann man sich täuschen.

Soll ich's sagen?

Du brauchst ein C-Buch

[Purzel (Gast)]

> interrupt :
>     sendstring(Empfaenger);
>     sendstring(Wert);

Was soll das ? Im Rx-Interrupt sendet man sicher nicht. Noch duemmer 
waer's wenn die Senderoutinen blockierend waeren.

[Karl H. (kbuchegg)]

Purzel schrieb:
>> interrupt :
>>     sendstring(Empfaenger);
>>     sendstring(Wert);
>
> Was soll das ? Im Rx-Interrupt sendet man sicher nicht. Noch duemmer
> waer's wenn die Senderoutinen blockierend waeren.

Ich bin mir sicher, dass das nur ein erster Test war und nicht der reale 
Code.

[Purzel (Gast)]

Auch wenn's nur ein Test war ... er soll den Kuchen bringen und dann 
tschuess. Gilt eigentlich fuer alle dazwischen auch. Keiner hat's 
gemerkt.