Basic Restaurant Bill with OOP

[aprendiz]

Hi guys, am trying to learn c++... but this code is giving me a strange error..
([Error] name lookup of 'i' changed for ISO 'for' scoping [-fpermissive] )
can some one help me to correct it..
Thanks in advance.. God Bless u all.

#include <iostream>
#include <string>
#include <iomanip>
#include <cstdlib>
using namespace std;
////////////////////////////////////////////////////////////////////
class restaurant
{
	private:
		string dish [10];
		int price [10];
		int num, cost;
	public:
		void inputorder()
		{
			cout<<"Enter number of dishes (<=10): ";
			cin>>num;
			for (int i=0; i<=num; i++)
				cout<<"Enter dish name and price: ";
				cin>>dish[i]>>price[i];
			cout<<endl;
		}
		void calculatebill()
		{
			for (int i=0; i<=num; i++)
				cost += price[i];
		}
		void showbill()
		{
			system("cls");
			cout<<setw(15)<<"Dish Name"<<setw(20)<<"Price"<<endl<<endl;
			for (int i=0; i<=num; i++)
				cout<<setw(15)<<dish[i]<<setw(20)<<"Rs. "<<price[i]<<endl;
			for(int l=1; l<=45; l++)
				cout<<"-";
			cout<<endl;
			cout<<setw(15)<<"Total Price"<<setw(20)<<cost<<endl<<endl;
		}
};
//////////////////////////////////////////////////////////////////////////
int main()
{
	restaurant order, bill, show;
	order.inputorder();
	bill.calculatebill();
	show.showbill();
	system("pause");
	return 0;
}

i got it compiled by declaring i outside loops.. but its not working correctly.. help plz..

[jlb]

Look closely at this snippet:

			for (int i=0; i<=num; i++)
				cout<<"Enter dish name and price: ";
				cin>>dish[i]>>price[i];

Since the second line is not part of the loop i is undefined.

In pre-standard C++ when initializing a variable in the initialization section of a for(;;) loop the variable was actually initialized as being before the loop. In standard C++ this variable is contained within the loop and goes out of scope at the end of the loop.

[aprendiz]

Thank u so much.. now its taking inputs.. but calculatebill and showbill function are not working.. After taking inputs, an error occurred, "code.exe has stopped working".. Now Whats wrong?

[jlb]

Please post your current code.

[aprendiz]

Here's the current code:

#include <iostream>
#include <string>
#include <iomanip>
#include <cstdlib>
using namespace std;
////////////////////////////////////////////////////////////////////
class restaurant
{
	private:
		string dish [10];
		int price [10];
		int num, cost;
	public:
		void inputorder()
		{
			cout<<"Enter number of dishes (<=10): ";
			cin>>num;
			for (int i=0; i<=num-1; i++)
				{
					system("cls");
					cout<<"Enter dish name and price: ";
					cin>>dish[i]>>price[i];
				}
			cout<<endl;
		}
		void calculatebill()
		{
			for (int i=0; i<=num-1; i++)
				cost += price[i];
		}
		void showbill()
		{
			system("cls");
			cout<<setw(15)<<"Dish Name"<<setw(20)<<"Price"<<endl<<endl;
			for (int i=0; i<=num-1; i++)
				cout<<setw(15)<<dish[i]<<setw(20)<<"Rs. "<<price[i]<<endl;
			for(int l=1; l<=45; l++)
				cout<<"-";
			cout<<endl;
			cout<<setw(15)<<"Total Price"<<setw(20)<<cost<<endl<<endl;
		}
};
//////////////////////////////////////////////////////////////////////////
int main()
{
	restaurant order;
	order.inputorder();
	order.calculatebill();
	order.showbill();
	system("pause");
	return 0;
}

Its giving results.. but with incorrect total price..

[jlb]

I think you need to review how classes work. You have created three instances of your class but you only initialize the values in one of these instances (order). All the values in bill and show are uninitialized. You probably only want one instance for this assignment.

int main()
{
	restaurant order;
	order.inputorder();
	order.calculatebill();
	order.showbill();
	system("pause");
	return 0;
}

[aprendiz]

I initialized cost=0 and its working fine.
Actually today was my first day to classes and objects.. so silly mistakes happened. but thanks alot. You helped me really patiently.
God bless u. Take care.