[SCAIRBORN]

Hi, I'm writing a program for a DVD collection to save to a file and be read later.
I have 2 classes, DVD and Collection.
I'm having collection read the DVD data from a file one element at a time separated by white space.
When the designated data is read it is initialized into a dvd and then put into a vector of dvd's for use while the program is open. When the user is done the program will write the vector to the file.

My problem is that all the data goes into the DVD and then a pushback is performed and every thing is fine (using debugger the data is put into the new allocation at the end of the vector). But all previous data in previous allocation is dropped.

here is the simple code for this function:

void Collection::set(vector<dvd>& cVector)
{
	vector<dvd> dummy_vector;
	dummy_vector.reserve(1);
	int j = 0;
		string a,b,c;
		int d,e;
		double f;

  ifstream collectionFile ("stefandvd.txt");
  if (collectionFile.is_open())
  {
	while (! collectionFile.eof() )
	{

		
			collectionFile >> a;
			collectionFile >> b;
			collectionFile >> c;
			collectionFile >> d;
			collectionFile >> e;
			collectionFile >> f;
	
		dvd n(a,b,c,d,e,f);

		/* PROBLEM HERE */
		dummy_vector.push_back(n);	//PROBLEM HERE!!!
		dummy_vector[j]=n;
		cout<<dummy_vector.at(j)<<endl;
		j++;
		/* END PROBLEM */
	}
	cVector = dummy_vector;
	//cout << "The contents of cStefan are:";
	//for (unsigned int i=0; i < cVector.size(); i++)
	//cout << " " << cVector[i];
	//cout << endl;

	collectionFile.close();
  }

  else cout << "Unable to open file";
}

I can post the entire project in here if you would like to run it for yourself to see whats going on.
Thanks for any help/advice... (this has been driving me up the wall for the last month!)

-Stefan

Attached File(s)

•  dvdproject.zip (780.52K)
  Number of downloads: 36

[EdwinNameless]

SCAIRBORN, on 17 Dec, 2009 - 11:00 AM, said:

		dummy_vector.push_back(n)
		dummy_vector[j]=n;

You have to choose, it's either or.

This post has been edited by EdwinNameless: 17 December 2009 - 06:01 AM

[SCAIRBORN]

EdwinNameless, on 17 Dec, 2009 - 05:00 AM, said:

SCAIRBORN, on 17 Dec, 2009 - 11:00 AM, said:

		dummy_vector.push_back(n)
		dummy_vector[j]=n;

You have to choose, it's either or.

Oddly enough if I take out the dummy_vector[j]=n; then I end up with a completely blank vector. The push_back(n) is only creating new containers but not adding any information to them.

If I take out dummy_vector.push_back(n) then I receive an out of bounds error.

I understand its improper programming, i'm just throwing everything i've got in the vector stl at this thing trying to make it work.

[Bench]

Why are you using dummy_vector? Seems as if it would be easier to fill your cVector directly

Also - never use "EOF" in a while loop (read here for the reason why: http://www.dreaminco...topic145699.htm )

    ifstream collectionFile ("stefandvd.txt");
    while ( collectionFile >> a >> b >> c >> d >> e >> f )
        cVector.push_back( dvd(a,b,c,d,e,f) ); 

If cVector is empty after that, then its likely to be a problem with your "stefandvd.txt". Since you haven't shown us what the file contains, we can only assume you have already made sure that >> is OK to read your file (If its not OK, then your vector will remain empty).

This post has been edited by Bench: 17 December 2009 - 07:07 AM

[SCAIRBORN]

Turns out the problem was with my dvd class copy constructor.
I know things were redundant, I was just throwing everything at this problem trying to force it to work.

Bench, THANKS for the the EOF help that was another problem I was having!