Zitat von Klotzkopp

Code:

char* fill = "000000";
printf(" %s%s \n", pointer1, fill + strlen(pointer1));

Du musst allerdings sicherstellen, dass der erste String niemals länger als 6 Zeichen wird.

Hallo,

vielen Dank.

Ich habe das merkwürdige Phänomen, das wenn ich mit der Funktion strcat Leerzeichen anhänge, dass sich ab größer 10 Leerzeichen die Anwendung aufhängt.

Also

strcat(Var1," Hier 10 Leerz. ");

klappt noch

aber mit mehr als 10 Leerz. ist feierabend, obwohl in Var 1 nur 6 Zeichen enthalten sind,
und der reservierte Speicher 30 Char Zeichen groß ist.

Hast Du ne Ahnung woran das liegen könnte.

Gruß
Eleu

Aus dieser einzelnen Codezeile und deiner Beschreibung lässt sich der Fehler nicht erkennen. Zeig bitte mehr Code, am besten ein minimales Beispielprogramm, das den Fehler reproduziert.

Hallo Klotzkopp,

es klappt jetzt und ich glaube ich weiß auch welchen Fehler ich gemacht habe.

Funktionsprinzip als verkürzte Fassung:

Variante 1 (Urfassung die im großen Quellcodeblock den Fehler verursacht hat, bei größer 10 Leerzeichen):

Code:

{
char *var1 = NULL;
char var2[255];
var1 = strcpy(var2,"123456");
strcat(var1,"                    ");
printf(var1);
}

Variante 2:

Code:

{
char *var1 = NULL;
char var2[255];
var1 = strcpy(var2,"123456");
strcat(var2,"                    ");
printf(var2);
}

Variante 3 (Endfassung):

Code:

{
char *var1 = NULL;
char var2[255];
var1 = strcpy(var2,"123456");
strncat(var2,"                    ",20);  
printf(var2);
}

Das Ergebnis der Ausgabe:ist bei allen 3 Codeschnipseln 123456 mit 20 Leerzeichen im Anhang.
Ich denke aber mal, dass es nicht so gesund ist, wenn man mit strcat einen Zeiger verändert, und das daher der eigentliche Fehler rührt. (Dieser dumme Eleu !!! )
Bei der letzten Variante wird strncat anstatt strcat verwendet.
Diese Variante soll sicherer sein. Hab jedenfalls im Netz gelesen.
Jedenfalls funktioniert es jetzt auch mit mehr als 10 Leerzeichen


Gruß
Eleu

Zitat von Eleu

Das Ergebnis der Ausgabe:ist bei allen 3 Codeschnipseln 123456 mit 20 Leerzeichen im Anhang.

Alle 3 Varianten sind funktional identisch. Wenn die erste bei dir Probleme gemacht hat, steckt vermutlich ein Bug irgendwo im Rest des Programms.

Zitat von Eleu

Ich denke aber mal, dass es nicht so gesund ist, wenn man mit strcat einen Zeiger verändert, und das daher der eigentliche Fehler rührt.

Nachdem du den Rückgabewert von strcpy var1 zuweist, zeigt var1 auf den Anfang von var2. Es ist also völlig egal, mit welchen von beiden du str(n)cat aufrufst.

Es hat bei mir auch einmal mit mehr als 10 Leerzeichen geklappt, als ich den Zeiger (Var1) mit strncat, anstatt mit strcat geändert hab.
Dann muss es irgendwie an strcat liegen ?
Sonst weiß ich nicht woran es gelegen hat.

Das alte Programm läuft schon lange störungsfrei.
Geändert wurde jetzt eigentlich nur, dass an Nutzdaten Leerzeichen angehangen werden, anstatt sie mit Nullen aufzufüllen.

Ich habe es jetzt ein paar mal ausprobiert....alles oki.

Zitat von Eleu

Es hat bei mir auch einmal mit mehr als 10 Leerzeichen geklappt, als ich den Zeiger (Var1) mit strncat, anstatt mit strcat geändert hab.

Der Längenparameter von strncat dient dazu, anzugeben, wieviel Platz im Zielstring noch ist, damit nicht mehr reinkopiert wird. Wenn du da die Länge des anzuhängenden Strings angibst, verhält es sich genauso wie strcat, und ist damit auch kein bißchen sicherer.

Zitat von Eleu

Dann muss es irgendwie an strcat liegen ?

Nein, wie gesagt: So wie du strncat benutzt, sollte es sich genau wie strcat verhalten. Entweder ist im Zielstring nicht genug Platz, oder die Speicherstruktur wurde vorher schon irgendwo beschädigt. Solche Fehler machen sich selten an der Stelle bemerkbar, an der sie verursacht werden.

Zitat von Eleu

Das alte Programm läuft schon lange störungsfrei.

Ich habe es jetzt ein paar mal ausprobiert....alles oki.

Das ist das "Schöne" an undefiniertem Verhalten: Es kann auch gut gehen. Dann änderst du etwas, und es knallt. Ausprobieren hilft da leider nicht.

Kannst du ein Minimalbeispiel erstellen, das das Problem reproduziert?

Ich kann es nicht reproduzieren.
Es klappt jetzt merkwürdigerweise auch mit strcat

Die Empfehlung strncat zu nehmen habe ich hierher:
Probleme im C-Script mit strcat - Siemens Industry Online Support - Technical Forum (Guest)

Es könnte auch sein, dass ich diesen Fehler gemacht habe:
Siemens Industry Online Support - Automation Service, Automation Support, Simatic Service, Simatic Support, Technical Support, Technical Consulting

Ist jedenfalls nicht unmöglich

Zitat von Eleu

Die Empfehlung strncat zu nehmen habe ich hierher:

Die Empfehlung ist an sich richtig, aber die Umsetzung ist falsch.

Code:

strncat(x, y, strlen(y))

ist genau dasselbe wie

Code:

strcat(x, y)

strncat so zu benutzen, ist komplett sinnlos.

Zitat von Eleu

Es könnte auch sein, dass ich diesen Fehler gemacht habe:

Keine Ahnung, kann sein. In deinem Beispielcode machst du ihn jedenfalls nicht.

Zitat von Klotzkopp

Keine Ahnung, kann sein. In deinem Beispielcode machst du ihn jedenfalls nicht.

Ja, und das liegt daran, dass mein Beispiel, mit dem ich eine Analogie herstellen wollte, ein schlechtes Beispiel ist.

Ursprünglich stand dort:

Code:

sprintf(Var2," %030s \n",Var1);

Wobei "Var1" eine Pointer Variable ist, für die kein Speicher reserviert wurde.

Und ich habe dann das davon gemacht:

Code:

strcat(Var1,"                      ");
sprintf(Var2,Var1);

Was natürlich Käse ist.

Jetzt sieht es so aus:

Code:

strcpy(Var2,Var1);
strcat(Var2,"                      ");

Und das klappt, weil für "Var2" Speicher reserviert wurde
Danke für Deine Unterstützung.