Reading a string from an input file into a structure

Does anyone know how to read a whole line into a structure?
I'm trying getline and failing.

I'm getting sci. notation. Why doesn't this code work? Thanks.

The text file includes:
Plain Egg
1.45
Bacon and Egg
2.45

Code:

void getData(menuItemType menuList [], ifstream& inFile)
{
	
	int counter=0;
	while (inFile)
	{
		getline(inFile,menuList[counter].menuItem);
		getline(inFile,menuList[counter].menuPrice);
		counter++;
	}
}

what types are menuItem and menuPrice? getline() only reads into strings. If they aren't strings then the code won't work. Also, you might have an off by one error in the number of items you read in from the file given the conditional of the loop, maybe not, depending how the file is written.

Thanks for the reply. menuItem is a string and menuPrice is a double. I changed the second read to inFile>>

I didn't understand the second part of your post. The new code:

Code:

void getData(menuItemType menuList [], ifstream& inFile)
{
	
	int counter=0;
	while (inFile)
	{
		getline(inFile,menuList[counter].menuItem);
		inFile>>menuList[counter].menuPrice; 
		counter++;
	}
}

the output is now
(1) Plain Egg 1.45
(2) -9.25596e+061
(3) -9.25596e+061
(4) -9.25596e+061
(5) -9.25596e+061
(6) -9.25596e+061
(7) -9.25596e+061
(8) -9.25596e+061

try while (!inFile.eof()) ...or post more code, so we can test run it...

Ok, here it is:

Code:

#include <iostream>
#include <fstream>
#include <string>
#include <cassert>
#include <iomanip>
using namespace std;
struct menuItemType
{
	string menuItem;
	double menuPrice;
};

struct custItemType
{
	string menuItem;
	double menuPrice;
};

void getData(menuItemType menuList[], ifstream& inFile);
void showMenu(menuItemType menuList[], custItemType custItem[]);  
//void printCheck(void);
int main()
{	
	menuItemType menuList[8];
	custItemType custItem[25];
	ifstream inFile;
	ofstream outFile;
	inFile.open("c:breakfast.txt");
	if(!inFile)
	{
		cout<<"Cannot open input file."<<endl;
		return 1;
	}
	//assert(inFile);
	outFile.open("c:Menu.out");
	assert(outFile);
	
	getData(menuList, inFile);
	showMenu(menuList, custItem);
	inFile.close();
	outFile.close();
	return 0;
}
void getData(menuItemType menuList [], ifstream& inFile)
{
	
	int counter=0;
	while (inFile)
	{
		getline(inFile,menuList[counter].menuItem);
		inFile>>menuList[counter].menuPrice;
		counter++;
	}
}

void showMenu(menuItemType menuList[], custItemType custItem[])
{
	  
	int g=0;
	int counter=0;
	int num=0;
	int i=0;
	for (counter=0; counter<8; counter++)
	{
		cout<<"("<<counter+1<<")  "<<menuList[counter].menuItem<<"  "<<menuList[counter].menuPrice<<endl;
	}
	cout<<endl;
	
	cout<<"Enter item's # [1-8] (0 to stop entering): ";
	cin>>num;
		cout<<"  "<<menuList[num-1].menuItem<<endl;
		custItem[i].menuItem==menuList[num].menuItem;
		i++;cout<<i<<" items. ";
	while (num!=0)
	{	
		cout<<"(inside loop) Enter item's number [1-"<<counter<<"] (0 to stop entering): ";
		cin>>num;
		//cout<<"  "<<menuList[num-1].menuItem<<endl;
		custItem[i].menuItem==menuList[num].menuItem;
		i++;
		cout<<custItem[0].menuItem<<i<<" items. ";

	}
	cout<<"First Item Entered: "<<custItem[0].menuItem<<endl;
	for (g=1; g<i; g++)
		cout<<custItem[g].menuItem;

}

First of all: I don´t use stream classes, but I´ll try to help anyway!
Second:

here:
inFile>>menuList[counter].menuPrice;
you are reading char from the file, I mean, the data within the file is char. So, between chars and doubles there is a huge difference(7 bytes if I´m not mistaken). First you should create a program that stores the data within a file. So, the data written will be doubles...(and strings).

I hope that you understood me... (like I always say: sorry for the bad english, too lazy to learn it !).

Five seconds and I had a better ideia... if won´t compromise your logic you could change that double to string too... it worked here!
The output for your program was:

(1)Plain Egg
(2)1.45 Bacon
(3)and Egg
(4)2.45
(5)
(6)
(7)
(8)

Now note that Bacon stays in number two. This is because

inFile>>menuList[counter].menuPrice;

just reads until the first blank space.
So... You could use Plain_Egg(with the underline), or you could use something less "beatiful" like read and write. So you would change string to char []. And always read the same amount of bytes. The problem is that this wastes memory. But I don´t know better solutions, sorry

like someone said, you should try
while(!infile.eof())

but also, you could do this:
while(!infile.eof() && !infile.fail());



it might work, but if it doesn't it might be a good idea to keep it in there anyways, but in the end, it's your call...

I still think that the problem is the struct that contains a double instead of a string. I´ve used the same loop as maxwell, I just changed the double to a string and everthing runs well...

Ok, I've changed the double to a string. Thanks everyone.

Then how could I add the amounts, to calculate a bill? Would static_cast work?

I try while(!infile.eof()) and it fails each time.

Have a look at the function ..atof();

An example of how to use atof. It is C, but who cares?

Code:

#include <string>
#include <ctype.h>
#include <conio.h>
using namespace std;

int main(){

   string s("5.64");

   cout << atof( s.c_str() ); //c_str converts string to char*

   getch();
}

Sorry, but after reading the fine print, I realize menuItem MUST be a double and not a string.

Now how should I read it in?

Code:

//this is your own function with some changes
void getData(menuItemType menuList [], ifstream& inFile)
{
	
    int counter=0;
    while (inFile)
    {
        getline(inFile,menuList[counter].menuItem);
        string tempString; //declares a temporary string
        inFile >> tempString;
        menuList[counter].menuPrice = atof(tempString.c_str());
        counter++;
    }
}

I not tested it ( too lazy today, sorry ), but should work. Just remember that >> reads until the first blank space or new line. But how I don´t know how to read a whole line I can´t help you further... sorry. But any other problems I still want to aid you

I modified your function:

Code:

void showMenu(menuItemType menuList[], 
custItemType custItem[]){

   int g=0;
   int num=0;
   int i=0;
   for (int k=0; k<counter; k++)
   {
      cout<<"("<<i+1<<")  "<<menuList
            [i].menuItem<<"  "<<menuList[i].menuPrice<<endl;
   }
   cout<<endl;
   //the rest of the function

Your old function had an problem: it loops until 8 every time, so, this is way you got garbage on screen. In this function above, counter is a global(too lazy to make it a parameter) that is the sum of the lines read(the same counter of the functions that reads the file).
As I said before(and before) I´m too lazy now, so if you got any more problems, ask...

Note for all: is does anyone know how to read a whole line from a file please tell me. The getline function in the Maxwell code don´t compile in Borland C++.