Thread: Reading a txt file into a struct problem

Hi,

I have wrote a program which places the contents of three array's into a txt file.

The 3 arrays (total of 10 different items):-

char string (food's name)
double (food's price)
bool (is food's price over £5)


Now instead of using the 3 arrays, I want to read contents from this txt file and place the contents into an array of structs.

Being new to this, I'm just starting with trying to read the txt file, but the lines in bold below are causing me problems, and am looking for some help.

Code:

#include <iostream>
#include <fstream>

using namespace std;
const int MAXCHARS = 20;
const int MAXITEMS = 10;

struct mystruct
{
	char itemname[MAXITEMS][MAXCHARS];
	double price;
	bool luxury;
};

int main()
{
	mystruct mystructname;

	ifstream mystream("myfile2.txt");

	if(!mystream)
	{
		cout << "phail " << endl;
	}


	mystream >> mystruct.price;

        cout << mystructname.price;
		

	mystream.close();


	system ("pause");

	return 0;

Been struggling with all this all day, so any help will be a big help.

Many thanks!

You need to create an array of mystruct. Right now, you have one mystruct, and inside there is one price, one luxury boolean and an array of MAXITEMS names. You want the names to be a single char array (because they are strings) and make an array of MAXITEMS mystructs.

Then you can input into the array and access each member with something like mystructarray[index].price.

Thanks Daved

Like this ? :-

Code:

mystruct thestructsarrayname[MAXITEMS]; //this will be the array for the 10 structs

Code:

#include <iostream>
#include <fstream>

using namespace std;
const int MAXCHARS = 20;
const int MAXITEMS = 10;

struct mystruct
{
	char itemname[MAXCHARS];
	double price;
	bool luxury;
};

int main()
{
	mystruct mystructname;

	mystruct thestructsarrayname[MAXITEMS]; 

	ifstream mystream("myfile2.txt");

	if(!mystream)
	{
		cout << "phail " << endl;
	}

	for(int i = 1;i < 10;i++)
	{
		cout >> thestructsarrayname[i].price;
	}	

	mystream.close();

	system ("pause");

	return 0;
}

I know it's easy to some people, I'm finding it hard to figure out as I've never done it before, and whenever I ask people for help (not Daved) they either say something I don't understand, or go wayy over my head saying use vectors, or overload this and that.

If anyone could PLEASE just help me out with reading from the txt file so that the first food item goes into the first struct I would be EXTREMLY grateful. I appreciate just giving someone the answers isn't helpful, but being stuck with one issue for this long is not helpful either. I know my code may not be 'optimised', or have other issues I'm not aware of, but that's because I'm new to all this and trying to teach myself.

Sorry for the long post, but I've become really frustrated with this, because I know that once I do have a working program, I'll be able to go through it slowly with the debugger step by step, and that way I'll get it into my head, it's just the way my brain works.

Many many thanks to anyone who can help me with this.



Swerve

P.S. I've attached a screen shot of the txt file, because I don't understand how it knows which bit to read.

Start even smaller. You seem to be throwing code in and hoping it works. You have to understand what each and every line of the program does.

First, let's assume that your struct is correct. You are holding only prices, which is fine.
Now, inside main, the first thing you want to do is create an array of MAXITEMS items named itemlist. Is that what you did? It's close, but not quite.

Then, you want to open the myfile2.txt file in an ifstream named mystream. Ok, that looks fine. Your check to make sure the file opened looks fine, too, although I agree that you should return from main there is the file failed to open.

Ok, now that you have created your item array and opened the file, the next step is to start reading from the file. Since only prices are in this new file, you want to read a value into the price member of each item in your array from the mystream file stream.

Your first line of code reads from mystream into the itemlist array, but what does that mean? You can't just read into the array itself like that, you want to read into each item and read into the specific member of each item.

Your for loop tries to do that. You loop over each item in the array and try to read into the price member. That's good. But what are you reading from? In your code, you're reading from cout? Is that where you should be getting the data? If you're trying to output to cout, then you've got the >> backwards, it should be <<. But then you still need to populate the price member variables for each item from the file.

I have managed to read the first line from the text file into the struct:-

Code:

#include <iostream>
#include <fstream>

using namespace std;

struct items
{
	char itemname[10];
	double price;
	bool luxury;
};

int main()
{
	items structname;
	char arrayname[10];
	ifstream myfile("myfile2.txt");

	if(!myfile)
	{
		cout << "phail " << endl;
	}

	myfile >> structname.itemname;
	myfile >> structname.price;
	myfile >> structname.luxury;


	myfile.close();
	system ("pause");
	return 0;
}

but I have to create "an array of structs" for /all/ the items.

You've got all the pieces in the various part s of code you've posted, you just have to put them together correctly.

You were able to declare an array of items earlier in this thread. Find that code and add it to the beginning of main().

well Swerve, I'm glad you've reached the solution finally.
But there is still one important thing you are missing:

Code:

    if(in_stream.fail())//checks the file could be opened.
    {
        cout << "Input file opening failed.\n" << endl;
    }

It's true, in_stream.fail () checks wether the file could be opened.
But, imagine, what would happen if the file doesn't exist?
Your program prints "Input file opening failed." and then it normally continues trying to read from a file which doesn't exist, so the program will simply crash in that case.
This could be definitely called a BUG, and it have to be fixed some way.

NOTE: At point 3 in my first post I'd already told you that, but probably you didn't pay enough attention. To say the truth, you didn't pay attention to the great part of my post, which contains the whole solution of your first problem.
So, you ask for help when you're not going to take it?