Why doesn't this code print?

[neel3123]

Write your question here.
Hey guys, when I choose option 7 in my program it doesn't print. Can anyone help?

// Store.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <iostream>
#include <vector>
#include <string>
#include <iomanip>
using namespace std;

class Product
{
public:
	int productId;
	string productName;
	double buyPrice;
	double sellPrice;
	int qty;
	int soldQty;
public:
	Product(int id, string name, double bprice, double sprice, int q);
	int getProductId() const;
	string getProductName() const;
	double getBuyPrice() const;
	double getSellPrice() const;
	int getSoldQty() const;
};

Product::Product(int id, string name, double bprice, double sprice, int q)
{
	productId = id;
	productName = name;
	buyPrice = bprice;
	sellPrice = sprice;
	qty = q;
	soldQty = 0;
}

int Product::getProductId() const {
	return productId;
}

string Product::getProductName() const {
	return productName;
}

double Product::getBuyPrice() const
{
	return buyPrice;
}

double Product:: getSellPrice() const
{
	return sellPrice;
}
int Product::getSoldQty() const
{
	return soldQty ;
}
class Store
{
private:
	int calcprofits;
	vector <Product> products;
public:
	void addProduct (Product &p);
	void deleteProduct (int id);
	void displayMenu ();
	void displayInv ();
};

void Store::addProduct (Product &p)
{
	products.push_back (p);
}
void Store::deleteProduct (int id)
{
	products.erase(products.begin() + id);
}
void Store ::displayMenu ()
{
	cout<< "What Would You Like To Do" <<endl;
	cout<< "1. Sell" << endl;
	cout<< "2. Restock" <<endl;
	cout<< "3. CalcProfits" <<endl;
	cout<< "4. CalcSales" <<endl;
	cout<< "5. Add Product" <<endl;
	cout<< "6. Delete Product" <<endl;
	cout<< "7 Print Inventory" <<endl;
	cout<<  "8. Exit" <<endl;
	cout<< "(Enter A Number That Corresponds With Choice)" <<endl;
}
void Store ::displayInv ()
{
	for(int x = 0; x < products.size() ; x++)
	{
		Product p = products[x];
        cout<< p.getProductId();
		cout<< p.getProductName ();
		cout<< p.getBuyPrice ();
		cout<< p.getSellPrice ();
		cout<< p.getSoldQty ();
	}
}

int _tmain(int argc, _TCHAR* argv[])
{
	Store ucStore;
	cout<< "Welcome to The Union City Store";
	while(true)
	{
		ucStore.displayMenu();
		int choice;
		cin>> choice;
		int id;
		string name;
		double bprice;
		double sellprice;
		int qty;
		int product;
		switch (choice)
		{
		case 7:
			ucStore.displayInv();
			break;
		case 5:
			cout<< "Enter product id";
		    
			cin>> id;
			cout<< "Enter product name";
			
			cin>> name;
			cout<< "Enter buy price";
			
			cin>> bprice;
			cout<< "Enter sell price";
			
			cin>> sellprice;
			cout<< "Enter Quantity";
			
			cin>> qty;
			Product p (id,name,bprice,sellprice,qty);
			ucStore.addProduct(p);
			break;
		case 1:
		    ucStore.displayInv();
			cout<< "What would you like to buy";
			cin>> product;
		}
	
}

  return 0;
  
}
  
  
}

[giblit]

Have you added items to the "inventory" first?

Also, why does your "product" class manage the "product" and the "inventory" shouldn't they be two distinct classes?

[cachinnatory]

I think this topic will be helpful: http://www.cplusplus.com/doc/tutorial/basic_io/

[neel3123]

how will basic io help me? And yes there already is products in inventory.

[neel3123]

someone?

[Computergeek01]

And yes there already is products in inventory.

No there aren't. You exit main right after your switch statement so there is no loop and no way that case 5 and case 7 both execute in the same instance. The contents of memory are destroyed once your program exits.

[giblit]

Oh I'm really sorry...I didn't realize you had your return inside the while loop. Move line 145 after the brace on line 146.

[neel3123]

I did that and now it's giving me this error:
1>c:\users\ashish\desktop\neels programming folder\store\store\store.cpp(145): error C2360: initialization of 'p' is skipped by 'case' label
1> c:\users\ashish\desktop\neels programming folder\store\store\store.cpp(142) : see declaration of 'p'

[giblit]

Would you please show the code you have currently?

[neel3123]

code updated

[neel3123]

Did you see it?

[giblit]

Ah, I overlooked earlier. The thing is

		case 5:
			cout<< "Enter product id";
		    
			cin>> id;
			cout<< "Enter product name";
			
			cin>> name;
			cout<< "Enter buy price";
			
			cin>> bprice;
			cout<< "Enter sell price";
			
			cin>> sellprice;
			cout<< "Enter Quantity";
			
			cin>> qty;
			Product p (id,name,bprice,sellprice,qty);
			ucStore.addProduct(p);
			break;
		case 1:
		    ucStore.displayInv();
			cout<< "What would you like to buy";
			cin>> product;

You didn't put braces around the object created in a certain case so it will exist in the following cases though it will not be initialized and probably contain garbage information.

Just explicitly tell it you want it only in that case

{
    Product p (id,name,bprice,sellprice,qty);
    ucStore.addProduct(p);
}

This may help you understand better http://stackoverflow.com/questions/5685471/error-jump-to-case-label

Also, why are your cases in descending instead of ascending order? :P

[neel3123]

Thanks, that helped but can you explain to me why option 7 doesn't work?

[neel3123]

Is anyone on?

[giblit]

Well, you are very vague. What exactly do you mean by "option 7 doesn't work?"

[neel3123]

When I select option 7 it just goes back to the main menu.

[Computergeek01]

Put the main menu in a function then and call it when the user enters '7'. There is no reason to over analyse this one.

[neel3123]

But my program is supposed to add new items to the displaymenu as they are added.

[mutexe]

are you actually using a debugger? If you step through the code you'd get a better understanding of what's going on.

[neel3123]

I Have no idea how to use a debugger