DaniWeb Forum Index > Software Development > C++

C++ RSS

Reading a file. Copying over to other file. Renaming. Renaming = -1?

Please support our C++ advertiser: Intel Parallel Studio Home

Thread Solved

•

Join Date: Nov 2008

Posts: 29

Reputation: Lokolo is an unknown quantity at this point

Solved Threads: 1

Lokolo

Offline

Light Poster

Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 18th, 2008

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
void writingTransactionToFile(string transaction)
{
	string temp;
	int test;
	ifstream myfile("transactionlog.txt");
	ofstream tempfile ("tempfile.txt", ios::app);
	if (tempfile.is_open())
	{
		tempfile << transaction;
		if(myfile.is_open())
		{
			while(!myfile.eof())
			{
				getline(myfile,temp);
				cout << temp;
				tempfile << temp << endl;
			}
		} else {
			cout << "Unable to open file.";
		}
		tempfile.close();
		myfile.close();
	} else 
		cout << "Unable to open file";
	test = rename("tempfile.txt", "transactionlog.txt");
	if(test != 0)
		perror("Error: ");
	cout << test;
}
void writingTransactionToFile(string transaction)
{
	string temp;
	int test;
	ifstream myfile("transactionlog.txt");
	ofstream tempfile ("tempfile.txt", ios::app);
	if (tempfile.is_open())
	{
		tempfile << transaction;
		if(myfile.is_open())
		{
			while(!myfile.eof())
			{
				getline(myfile,temp);
				cout << temp;
				tempfile << temp << endl;
			}
		} else {
			cout << "Unable to open file.";
		}
		tempfile.close();
		myfile.close();
	} else 
		cout << "Unable to open file";
	test = rename("tempfile.txt", "transactionlog.txt");
	if(test != 0)
		perror("Error: ");
	cout << test;
}

Ok so its mainly the rename function that doesn't work. It just returns the value -1. perror doesn't return any error.

Not sure why it doesn't like it?

•

Join Date: Apr 2008

Posts: 670

Reputation: Freaky_Chris is a jewel in the rough

Solved Threads: 113

Freaky_Chris Freaky_Chris is offline

Offline

Practically a Master Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 18th, 2008

call me silly but why not just send the data straight to the correct file name. Also i would recomend you do all file copying in binary mode to prevent loss of any data.

Chris

Knowledge is power -- But experience is everything

•

Join Date: Nov 2008

Posts: 29

Reputation: Lokolo is an unknown quantity at this point

Solved Threads: 1

Lokolo

Offline

Light Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 18th, 2008

•

Originally Posted by Freaky_Chris

call me silly but why not just send the data straight to the correct file name. Also i would recomend you do all file copying in binary mode to prevent loss of any data.

Chris

Because I want the latest data to be at the top and this is the only way to do it.

•

Join Date: Apr 2008

Posts: 670

Reputation: Freaky_Chris is a jewel in the rough

Solved Threads: 113

Freaky_Chris Freaky_Chris is offline

Offline

Practically a Master Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 19th, 2008

What exactly do you mean by 'at the top'?

Chris

Knowledge is power -- But experience is everything

•

Join Date: Nov 2008

Posts: 29

Reputation: Lokolo is an unknown quantity at this point

Solved Threads: 1

Lokolo

Offline

Light Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 19th, 2008

•

Originally Posted by Freaky_Chris

What exactly do you mean by 'at the top'?

Chris

At the top of the text file.

i.e. its a list of transactions and I want the most recent transactions at the top. At the bottom of the file is the first transaction.

My text file would look something like this:

17/11/2008 - Something happened
14/11/2008 - Something happened before
11/11/2008 - etc.
10/11/2008 - etc.

So when I then print the stuff from the text file on screen it looks like this.

The only otherway I could think of is to read the text file backwards - but I have no idea how to do this either and cannot find a way of doing it. (whereas the method above seems to work, but the rename bit isn't, even though the error returns 'File Exists'

)

•

Join Date: Oct 2006

Posts: 2,819

Reputation: niek_e has a reputation beyond repute

Solved Threads: 297

niek_e

Offline

Roasting Maven

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 19th, 2008

Are you deleting the previous transactionlog.txt somewhere? Else rename will fail because the file already exists.

You should also avoid using eof().
If you change this:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
while(!myfile.eof())
{
    getline(myfile,temp);
    cout << temp;
    tempfile << temp << endl;
}
while(!myfile.eof())
{
    getline(myfile,temp);
    cout << temp;
    tempfile << temp << endl;
}

to this:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
while(getline(myfile,temp))
{
    cout << temp;
    tempfile << temp << endl;
}
while(getline(myfile,temp))
{
    cout << temp;
    tempfile << temp << endl;
}

The eof() function is gone, and it also looks a lot cleaner

•

Join Date: Nov 2008

Posts: 29

Reputation: Lokolo is an unknown quantity at this point

Solved Threads: 1

Lokolo

Offline

Light Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 19th, 2008

1. No I thought it would overwrite it (like if you try to open a file, if it doesn't exist, it creates one)... guess it doesn't. Will look up file delete later (I funnily enough have a C++ tutorial now)

2. Ok will change code to that, not much difference - doesn't eof check if there is a line available? So its just doing the same (I could be wrong)

•

Join Date: Nov 2008

Posts: 29

Reputation: Lokolo is an unknown quantity at this point

Solved Threads: 1

Lokolo

Offline

Light Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 19th, 2008

Ok realising I am retarded. I should have taken the 'Error Message' literally. It kept saying 'File Exists', which I then thought yes, it does exist, so rename it to that! Raging at my laptop.

However it now works. Thanks.

•

Join Date: Apr 2008

Posts: 670

Reputation: Freaky_Chris is a jewel in the rough

Solved Threads: 113

Freaky_Chris Freaky_Chris is offline

Offline

Practically a Master Poster

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

Nov 20th, 2008

•

Originally Posted by Lokolo

2. Ok will change code to that, not much difference - doesn't eof check if there is a line available? So its just doing the same (I could be wrong)

EOF can return true for more than that reason i.e certain characters can cause it to return true. I believe '\v' is one of those. This is why it should be avoided.

Chris

Knowledge is power -- But experience is everything

•

Join Date: Oct 2007

Posts: 1,951

Reputation: Duoas has much to be proud of

Solved Threads: 214

Duoas

Offline

Posting Virtuoso

Re: Reading a file. Copying over to other file. Renaming. Renaming = -1?

#10

Nov 20th, 2008

The documentation for rename() plainly states that if a file with the new name already exists, results are implementation-defined. You are better off deleting any existing 'new name' file first.

The problem with eof() is more insidious. Imagine what would happen to his code if an error flag were set in the middle of that loop. (An infinite loop.)

Always check against good(), and if that fails, you can check to see which flag caused the failure: whether a benign (and wanted) eof() or an obnoxious bad() or fail() of some sort.

Ctrl-D is the EOF character on *nix and in ASCII (EOT).
Ctrl-Z is considered an EOF character on DOS.
However, all this is system dependent and you should not generally see these characters when doing file I/O --as EOF is technically a condition, not a part of a stream.

Hope this helps.