Minifrage - gcc strcat() - erweitert eigenständig die die Arrays

[Jens (Gast)]

Hallo,

ich hab folgendes Mini-Programm. (Zahlen in Zeichen umwandeln
und mit strcat() in einem gemeinsamen Array speichern)

char mydata[100];

void myadd(u16 data) {
   char tmp[80];
   
   ltoa(data, tmp, 10);
   strcat(mydata, tmp);
   strcat(mydata, "-");
   
   printf("%s\n", mydata); // Debug...
}

int main(void) {
   u16 counter = 0;
   while (1) {
       counter++;
       myadd(counter);
   }
}

Wie oft ich auch die Zahl in mydata schreibe,
die Ausgabe wird immer größer - vermutlich
bis kein Flashspeicher mehr da ist.

Ich find nur nirgends diesbezüglich Infos.

Das Verhalten könnte später kritsch werden.
Wollte nur kurz fragen ob das so richtig ist... :)
Möchte ungern etwas einbauen was ich nicht richtig
verstehe ...

Vielen Dank
Jens

[Klaus W. (mfgkw)]

Das Ganze ist unkritisch, solange du in den 80 Bytes bleibst
(inkl. abschließender 0).
Alles danach ist Glückssache.

Außerdem hat das mit Flash nix zu tun, es geht hier um RAM.

> Das Verhalten könnte später kritsch werden.

Das wird es sicher.

> Wollte nur kurz fragen ob das so richtig ist... :)

Es ist grundfalsch.

> Möchte ungern etwas einbauen was ich nicht richtig
> verstehe ...

C-Buch lesen und verstehen!
Das hat schon vielen geholfen.
Aber viel mehr probieren es ohne...

[Klaus W. (mfgkw)]

> Ich find nur nirgends diesbezüglich Infos.

Dann will ich nicht fragen, wie du gesucht hast :-)

[strcat (Gast)]

eine gute idee wäre schon einmal, auf deine max array size zu testen. 
Sonst schreibst du dir über das array hinaus in speicherbreiche rein. da 
kommen dann noch ganz andere sachen. vergiss beim überprüfen der länge 
aber nicht das \0

[Jens (Gast)]

super danke, für die schnelle hifle

[Johann L. (gjlayde)]

Jens schrieb:
> Hallo,
>
> ich hab folgendes Mini-Programm. (Zahlen in Zeichen umwandeln
> und mit strcat() in einem gemeinsamen Array speichern)
>
>

> char mydata[100];
> 
> void myadd(u16 data) {
>    char tmp[80];
> 
>    ltoa(data, tmp, 10);
>    strcat(mydata, tmp);
>    strcat(mydata, "-");
> 
>    printf("%s\n", mydata); // Debug...
> }
> 
> int main(void) {
>    u16 counter = 0;
>    while (1) {
>        counter++;
>        myadd(counter);
>    }
> }
> 
>

>
> Wie oft ich auch die Zahl in mydata schreibe,
> die Ausgabe wird immer größer - vermutlich
> bis kein Flashspeicher mehr da ist.

Nö, bis kein RAM mehr vorhanden ist.

myadd hängt nur was an mydata an, aber niemand "leert" mydata. Das 
könnte zB erledigt werden durch

mydata[0] = '\0';

am Ende von myadd oder zu Beginn. Dann lassen sich allerdings durch 
mehrfache Aufrufe keine längeren Strings mehr zusammenbasteln, was ja 
unerwünscht ist. Andernfalls müsste es ausserhalb von myadd geschehen, 
etwa

   ...
   if (counter % 4 == 0)
      mydata[0] = '\0';
   counter++;
   ...

[High Performer (Gast)]

strcat hängt zwei Strings aneinander. Das terminierende Nullzeichen des 
Zielstrings wird überschrieben mit dem ersten Zeichen des Quellstrings. 
Nach dem Kopiervorgang wird wieder ein Nullzeichen angehängt.
Beim nächsten Aufruf wird dann an diesen String wieder das neue Zeug 
angehängt.

Wie die anderen schon geschrieben haben: es gibt Beschreibungen der 
Funktionen im Netz. Ich will nicht so streng sein, da ich nicht 
verlangen möchte, dass man, bevor eine Frage gestellt wird, erst 
stundenlang selbst recherchiert. Dann wär's hier sehr ruhig ;-)