Strcat with filenames

[tofugirl]

I get a name from the user and output it to a file "name.txt". For instance, if the user entered 'tofugirl' I write to a file called "tofugirl.txt". However, how would I get "name" ("tofugirl") back since it is concatinated to the ".txt"? Is there an "un concatinate'?

Any help greatly appreciated. Code at the bottom.

char *name;
FILE *ofp;
   

   printf("Please enter name\n");
   scanf("%s", name);

   strcat(name, ".txt");

   ofp = fopen(name, "w");
   
   if(ofp == NULL)
   {
      printf("Couldn't open %s\n", name);
      exit(-3);
   }

[Quantum1024]

Your name pointer should be set to point to something.
If your always going to be adding 4 chars (.txt) then you can just use strlen to get the length of the string and either copy that many minus 4 to another string or change the '.' to a NULL.
A more rubust way would be to create a function that scans your string from the end to begining looking for the '.'

[jmd15]

Why not just strcpy the original onto another variable named something like nameorig, before you append .txt to it? That way you have a variable that is a valid filename, and a variable that is just the name they entered.

[tofugirl]

Do you perhaps have a code snippet for this? I tried to do that but I'm getting core dumps. Here's my code.

char *name, *origname;
FILE *ofp;
   

   printf("Please enter name\n");
   scanf("%s", name);
  strcpy(origname, name);
   strcat(name, ".txt");

   printf("Original Name: %s\n", origname);
   ofp = fopen(name, "w");
   
   if(ofp == NULL)
   {
      printf("Couldn't open %s\n", name);
      exit(-3);
   }

[cwr]

You're getting core dumps because you didn't listen to the initial reply to your original post.

char *name;

This declares enough space to hold a pointer.

scanf("%s", name);

This reads in a string, and puts it where name points to. Where does name point to?

strcpy(origname, name);

This copies the string that name points to, to the address that origname points to. Where does origname point to?

You need to allocate space to store strings, either by using a pointer and malloc, or using an array of chars.

[jmd15]

Yeah, try initializing the two char variables like this:

char name[256]={0};
char origname[256]={0};

[tofugirl]

Thanks for all your suggestions. I went with jmd15's answer. It works now.

A big thank you!

[Barnzey]

Or you can use strncpy,

strncpy(name,filename,(strlen(filename)-4));

Or

strtok:

name = strtok(filename, ".");