mein quellcode sieht wie folgt aus
Quellcode:#include <string.h>


void changeDat(char *player)
{  
    char *str1;
    char *plDat = player;

    strcpy(str1,".dat");
    strcat(plDat,str1);
}
wird ohne fehler kompiliert
verwende ich die funktion innerhalb eines programmes stürzt es zur laufzeit ab
weiß jmd. woran das liegen könnte?

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

guck dir noch mal pointer an
da is nich nur ein fehler drin

erstens mal hat str1 keinen speicher zur verfügung
du tust also ".dat" "irgendwo rein"
2tens (is nur mal so ne vermutung wegen dem was du oben auch shcon falsch gemacht hast) bezweifle ich mal dass player und somit plDat genug platz für str1 haben (sofern str1 speicher hätte)

Zitat:

CN postete
erstens mal hat str1 keinen speicher zur verfügung
du tust also ".dat" "irgendwo rein"

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

musst ja auch bla[5] machen

'.','d','a','t','\0'

Against TCPA | Resourcecode.de | Blender 3D | [ Darkzone | Pandorra | Alpine ]

0-4 oder?
ich hab das jetzt so gelöst:
Quellcode:void changeDat(char player[10])
{  
    char str1[4];
    char plDat[10];
    
    strcpy(str1,".dat");
    strcpy(plDat,player);
    strcat(plDat,str1);
} schade eigentlich dass man die länge einschränken muss.
danke für eure hinweise

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

hast du den post von voice überhaupt gelesen?
du brauchst platz für 5 chars weil am ende noch der 0-terminator kommt

übrigens macht das oben auch keinen sinn weil plDat nach aufruf der funktion nicht mehr verfügbar ist
(abgesehen davon dass die gesamte funktion eigentlich nur ein strcat() macht ..)

du kannst (und solltest) das auch dynamisch mit pointern machen (was ist zb wenn du "123456" als param übergibst .. dann is da kein platz mehr für ".dat"), dann musst du aber auch den speicher selbst zuweisen
deswegen sagte ich ja auch "guck dir noch mal pointer an", am besten noch mal im zusammenhang mit dynamischer speicherzuweisung

char str1[4], das heisst str1[0] bis str1[3]... wenn du 0-4 willst musst du str1[5] schreiben.

achja: plDat sollte dann 15 elemente gross sein. 10 + 5 != 10

alternativuser ( www.opera.com & www.trillian.cc & www.photonplayer.org )
coder @ www.public-enemy.org

Zitat:

CN postete
hast du den post von voice überhaupt gelesen?
du brauchst platz für 5 chars weil am ende noch der 0-terminator kommt

Zitat:

unleash postete
0-4 oder?

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

Quellcode:char str1[4]; das ist das problem
da ist nur platz für
{'.', 'd', 'a', 't'}

0-4 sind bei mir 5 elemente (0 1 2 3 4), nicht 4

dein programm mag funktionieren aber .. die funktion nützt nichts
du speicherst ein paar werte zwischen und fuchtelst da rum aber im endeffekt wird einfach nur ".dat" an player gehängt, was du aber eh nicht nutzen kannst weil plDat nur lokal in der funktion verfügbar ist

ich hab hier nicht die gesamte funktion drinne.
ich erstelle nachher noch eine datei mit diesem namen deshalb das ganze

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

nochmal:
ch[0] = '.' - eins
ch[1] = 'd' - zwei
ch[2] = 'a' - drei
ch[3] = 't' - vier
ch[4] = '\0' - fünf

--> ch[fünf] -> ch[5] -> in der eckigen klammer steht die anzahl elemente, nicht der index des letzten elements

alternativuser ( www.opera.com & www.trillian.cc & www.photonplayer.org )
coder @ www.public-enemy.org

unleash was hat das damit zu tun dass du dafür keine extra funktion brauchst sondern strcat reicht

in der funktion mache ich das mit dem datei öffnen/schreiben.
das ist die aufgabe der funktion
aber da es viel code ist habe ich nur das betreffende kopiert

@spin ja das ist mir klar, war mir schon nach deinem erstem post klar. ursprünglich war ich nur der meinung man gibt nicht die anzahl der elemente an sondern den feldschlüssel bis zu dem es geht.

ich denke mal, es dauert nicht mehr allzulange, dann wird dein Nick sanft von unten rot angeleuchtet..
by WW
to iterate is human, to recurse, slow!

ok, sry... hat nach deiner antwort nicht so ausgesehen :)

alternativuser ( www.opera.com & www.trillian.cc & www.photonplayer.org )
coder @ www.public-enemy.org