copying a file

I want to read the filename that is data in one file and open that filename specified as data.

say, file1="filetoread";
file2="filetowriteto";

I want to read the data from file1 (say line 1 of file1 is "c:\myfile.txt"), then I want to open the file listed on line 1 of file1 and copy the data from c:\myfile.txt to file2.

Try a board search

Post the code you attempted to do this with.

This is what I have so far:
It works up to: " if ((fnewsource = fopen(sourcefile,"r"))==NULL)"...it's not able to open the file.

see the attached file...

Code:

---

fgets(fileName,256,fold);
sourcefile=fileName;
/* ... */
if ( (fnewsource = fopen(sourcefile,"r"))==NULL )

---

If you don't pick off the '\n' that fgets may leave in, it's still there. So if you are intending to open "file.txt", you may be trying to open "file.txt\n". More info.

Thanks. I will try that and get back to you.

fgets() retrieves and stores the newline in the buffer - you have to strip it out:

Code:

---


char * get(char * buffer, unsigned max, FILE * stream)
{
 char * crlf = 0, * line = fgets(buffer, max, stream);
 if(line) crlf = strstr(line, "\n");
 if(crlf) *crlf = 0;
 return line;
}

---

Also, look at this:

>> char *scenarioNum;
>> gets(scenarioNum);

'scenarioNum' doesn't point to any memory!

Don't use gets() either, since it doesn't provide buffer-overflow protection.

Finally, look at the code that copies a file - wouldn't that make a great 'reusable' function? [hint hint]

Thank you all. Your suggestions worked! I needed to strip the \n off of the end of the string (thanks for the code for that too).

Quote:

---

Originally posted by Sebastiani fgets() retrieves and stores the newline in the buffer - you have to strip it out:

Code:

---

char * get(char * buffer, unsigned max, FILE * stream)
{
 char * crlf = 0, * line = fgets(buffer, max, stream);
 if(line) crlf = strstr(line, "\n");
 if(crlf) *crlf = 0;
 return line;
}

---

---

How about:

Code:

---

char * get(char * buffer, unsigned max, FILE * stream)
{
    int i;
    line = fgets(buffer, max, stream);
    i = strlen(line) - 1;
    if(line[i] == '\n') line[i] = '\0';
    return line;
}

---

This saves the character by character search the strstr() uses. And if there is a return, it must be at the end of the string.

But then when you think about it, what does strlen() do? Hmmm....

But I have only one if. That must count for sumtin'

The extra 'if' is necessary because fgets() might return NULL, in which case you wouldn't want to pass the pointer to a string-handling function anyway.

Quote:

---

Originally posted by Sebastiani
The extra 'if' is necessary because fgets() might return NULL, in which case you wouldn't want to pass the pointer to a string-handling function anyway.

---

Good point...