einen char an einen char* hängen

habe in der zwischen zeit auch rumgespielt und kompilieren tut ers jetzt!
aber dann bekomme ich beim ausfürne nen :

Program received signal SIGSEGV, Segmentation fault.
0x281fe7c6 in strcat ()

char* Stmp=0;
char searchstring[2]={'<',0};
Stmp=strcat(Stmp,searchstring);  //hier liegt der Fehler!!

so sieht es vereinfacht aus!

also was mach ich falsch?

thx

Shade Of Mine

Stmp ist ein NULL Zeiger. Der hat _keinen_ Speicher.
Wie willst du da etwas dranhaengen?

ok sorry mein fehler
habe daraus jetzt das gemacht:

char* Stmp='\0';

und habe immer noch nen error

habe hier mal den gesamten code gepostet:

http://nopaste.php.cd/14926

hier hängt er:
http://nopaste.php.cd/14926#77 (zeile 77)

damit ihr versteht was ich will.

thx

Shade Of Mine

newuserxx424 schrieb:

ok sorry mein fehler
habe daraus jetzt das gemacht:
char* Stmp='\0';
und habe immer noch nen error

lies mal über Zeiger nach.
Stmp hat Speicher für 1 Zeichen, welches bereits fest mit \0 belegt ist. Du kannst \0 weder überschreiben noch etwas daranhängen.

ok mach ich danke!
aber das hilft mir immoment auch nicht weiter.
wie mach ich den das dann?
(habe vorher immer in delphi programmiert da gibts "normale strings", also sorry aber muss mich an die strings von c erst gewöhnen. währe für tipps sehr dankbar!)

daa531

#include <stdio.h>
#include <string.h>
#include <malloc.h>
int main()
{
char a='X'; 
char* b="test"; 
char buffer[10];
char *c; 

sprintf(buffer,"%s%c",b,a);
c = buffer;
printf("%s\n",c);

Waere eine loesung.

mfg Olli.

Gunnar

Wie gesagt, der Zielstring braucht genug Speicher um das Ergebnis zu halten.
z.B:

char str[64]; // reservier 64 Chars auf dem Stack
strcpy(str, "test"); // kopiert "test" nach str, das 5. Zeichen ist 0 und markiert
damit das Stringende

char tmp[2] = {'x', 0};
strcat(str, tmp); // 64 Bytes langen dicke um auch noch das 'x' unterzubringen..

(Solche Sachen sind aber genau die Dinge, die Bufferoverflows oder Exploits ermoeglichen... in C++ gibt es Strings die den Delphistrings etwas aehnlicher sind)

Shade Of Mine

was haltet ihr eigentlich davon:

char* charcat(char* str, char c)
{
  size_t pos=strlen(str);
  str[pos]=c;
  str[pos+1]=0;
  return str;
}

THX an alle!!!!!

@Shade Of Mine
das sieht zwar gut aus geht aber net.
er sagt dann:

Program received signal SIGBUS, Bus error.
0x080488fd in charcat(char*, char) (str=0x804930b " ", c=60 '<') at rsssucker.C:46

Shade Of Mine

newuserxx424 schrieb:

@Shade Of Mine
das sieht zwar gut aus geht aber net.

geht sehr wohl, allerdings muss der char* ein string sein (0 terminiert) und genug speicher haben

#include <iostream>

int main(void)
{
char* test;
char test2='X';
test="xxx\0";          //habe auch ohne \0 probiert.
size_t pos=strlen(test);
  test[pos]=test2;
  test[pos+1]=0;
std::cout<<test;
return 0;
}

dieses prog gibt bei mir nen core dump!
warum? was mach ich falsch? (frage ist ernstgemeint! und wenn ihr tutorials über string und c kennt her damit!)

thx

PAD

mit char *test legst du nur einen Pointer auf einen String ohne jeglichen Platz für den Inhalt an.

mit char *test="bla"; legst du einen Pionter auf eine String mit exact vier Zeichen an. nähmloch b,l,a,\0;

somit ist deine Zuweisung test="xxx\0" unzulässig und verursacht den core dump, außerdem kennt c diese Art der Initialisierung nur bei der Deklaration eines Strings, an allen anderen Stellen mußt dur dies mit Funktionen wis sprintf, strcst, strcpy,... zuweisen

Damit es funktioniert mußt du entweder

char test[Irgend_Eine_Sinnvolle_Länge]={"xxx"}; 
// mindestlänge ist die Länge deines Textes + das anzuhängende Zeichen + die schließende \0

oder

char *test;
if (NULL == (test=calloc(Irgend_Eine_Sinnvolle_Länge,siozeof(char)))) //sinnvolle Länge siehe oben
     printf("ERROR konnte keinen Speicher allokieren");

benutzen. Damit funktioniert dann jeder Vorschlag aus diesem Thread :p

ein fettes THX

Bashar

PAD schrieb:

mit char *test="bla"; legst du einen Pionter auf eine String mit exact vier Zeichen an. nähmloch b,l,a,\0;

somit ist deine Zuweisung test="xxx\0" unzulässig und verursacht den core dump, außerdem kennt c diese Art der Initialisierung nur bei der Deklaration eines Strings, an allen anderen Stellen mußt dur dies mit Funktionen wis sprintf, strcst, strcpy,... zuweisen

Leider falsch. Die Zuweisung test = "xxx\0" ist nicht unzulässig. Sie hat den gleichen Effekt wie die entsprechende Initialisierung mit demselben Wert. Das Stringliteral "xxx\0" (warum eigentlich das NUL-Zeichen an der Stelle? Doppelt terminiert hält besser? Na von mir aus ) steht statisch im Speicher, und da darf natürlich jeder char-Zeiger drauf zeigen, der grad Lust hat. Verändern des Stringliterals ist dagegen nicht erlaubt.

Du denkst vielleicht, da würde irgendwie intern strcpy oder so aufgerufen werden. Nope. Es wird nur die Adresse zugewiesen.

Der Core Dump passiert an der Stelle:

test[pos]=test2;

Weil dort versucht wird, das Stringliteral, auf das test zeigt, zu verändern.

PAD

@Bashar Danke für die Korrektur

Hatte meist Fehlerfälle dieser Art.

{
  char test[20]="Peter";
....
....
  test="Mist";
}

d:\work\sourcemain.cpp(1124) : error C2440: '=' : 'char [5]' kann nicht in 'char [20]' konvertiert werden
        Es gibt keinen Kontext, in dem diese Konvertierung moeglich ist

Und das ist etwas anderes als dieses Konstrukt (welches ich bisher noch nicht benötigt habe)

{
  char *test="Peter";
....
....
  test="Mist";
}