Funktion utoa funktioniert nicht richtig

[R. B. (britzi)]

Hallo zusammen,

ich habe ein kleines Problem mit der Funktion utoa.

Ziel ist es einen Integer aus dem ADC in einen String für UART-Ausgabe 
zu schreiben.

Ich habe schon einiges probiert um den Fehler zu finden, aber bin 
mittlerweile völlig Ratlos.

Der Code sieht wie folgt aus:

int main(void)
{
  
  uint16_t adc_wert=50;
  char spannung[20];
  uint16_t acd_flag=1;
  init_uart();
        
   sei();
   
    while(1)
    {
    
    if (adc_wert!=acd_flag)
       {utoa(adc_wert, spannung, 10);
       strcpy(spannung, "\n\r");
       //put_string(spannung);
       put_string("test\n\r");
       adc_wert=acd_flag;
      }
    
    
     }

So wie gezeigt gibt Uart korrekt einmal "test" aus.
Wenn ich die Zeile mit dem erzeugten String auskommentiere liefert die 
Ausgabe ausschließlich die Zeichenfolge 0A 0D 0A 0D und auch das Wort 
"test" wird nicht mehr ausgegeben.

Sieht jemand den Fehler?

Vielen Dank :-)

[Rolf Magnus (Gast)]

R. B. schrieb:
> Sieht jemand den Fehler?

R. B. schrieb:
> strcpy(spannung, "\n\r");

Hier ersetzt du das, was du vorher reingeschrieben hast, durch ein 
Newline.

[Helfer (Gast)]

R. B. schrieb:
> strcpy(spannung, "\n\r");

Mit diesem Ausdruck überschreibst du die vorher durch utoa() erzeugten 
Daten aus deinem String.
0A und 0D sind die Hexadezimalen Darstellungen deiner Zeichen '\n' und 
'\r' Lasse diese Zeile weg, oder ersetze sie durch:
strcat (spannung,"\n\r");

[Helfer (Gast)]

Ups, da waren Rolfs finger wohl flinker ;)

[Non-Softi (Gast)]

>        strcpy(spannung, "\n\r");

Tach, werden die hier nicht die ersten beiden Bytes des Strings, der die 
Spannung enthalten soll, wieder mit CR-LF überschrieben?

Gute Nacht, im Zweifel versuch's morgen früh nochmal.

[Chris L. (kingkernel)]

Die Funktion die du suchst, lautet strcat. strcat hängt einen string an 
einen bestehenden an

[Kaj (Gast)]

Alternative:
Du weißt ja wie groß dein Wert ist, dann kannst du das im zweifel auch 
von hand machen, z.B.: Spannung[5] = '\n'; Spannung[6] = '\r';

Grüße

[Karl H. (kbuchegg)]

Kaj schrieb:
> Alternative:
> Du weißt ja wie groß dein Wert ist, dann kannst du das im zweifel auch
> von hand machen, z.B.: Spannung[5] = '\n'; Spannung[6] = '\r';



Das ist die schlechteste Idee, die du haben kannst.

Alternativ:
Du kannst auch deine Texte in mehreren Happen ausgeben. Der Empfänger 
kann das sowieso nicht unterscheiden, ob du einmal oder mehrmals 
put_string aufgerufen hast. Der sieht ja nur, was an Zeichen über seine 
Schnittstelle reinkommt.

       utoa( adc_wert, spannung, 10 );
       put_string( spannung );
       put_string( "\r\n" );

[R. B. (britzi)]

Vielen Dank für die Antworten. :-)

Das Problem hat sich sofort gelöst ;-)

Ihr seit echt super!

[Axel S. (a-za-z0-9)]

Und weil Karl Heinz es zwar korrigiert, aber nicht erwähnt hat: die 
korrekte Zeilenende-Sequenz für (Win)DOS Systeme ist CR + LF (als C 
Literal: "\r\n") und nicht umgekehrt.


XL

[R. B. (britzi)]

Axel Schwenke schrieb:
> Und weil Karl Heinz es zwar korrigiert, aber nicht erwähnt hat: die
> korrekte Zeilenende-Sequenz für (Win)DOS Systeme ist CR + LF (als C
> Literal: "\r\n") und nicht umgekehrt.


Danke :-)