What am I doing wrong?

Question

1.00/5 (1 vote)

See more:

I tried so hard but every time I do the problem, I at last get same unusual irritating messages. Please help me out!!! I will appreciate each and every suggestion...Thank you Patrice ......
-[Error] no matching function for call to 'Inventory::Inventory()
-I am trying to use Inventory class inside CashRegister class, but programme is not running.
-It should have printed total cost from cost of Inventory class multiplied by quantity of CashRegister class.

What I have tried:

C++

#include<iostream>
using namespace std;

class Inventory
{
	private:
		int quantity;
		double cost;
	public:
		Inventory(int q, double c){
			quantity = q;
			cost = c;
		}
		~Inventory(){
			cout<<"Quantity destroyed";
		}
		
		int getQuantity(){
			return quantity;
		}
		
		int getCost(){
			return cost;
		}
};

class CashRegister
{
	private:
		Inventory obj;
		int item, quantity;
		double total;
	public:
		CashRegister(Inventory object, int i, int q){
			obj = object;
			item = i;
			quantity = i;
			total = quantity*object.getCost();
		}
		double getTotal(){
			return total;
		}
};

int main()
{
	Inventory i(10, 11.5);
	
	int item, quantity;
	cout<<"Enter item and quantity:\n";
	cin>>item>>quantity;
	
	CashRegister c(i, item, quantity);
	cout<<"Total = "<<c.getTotal();	
}

Posted 3-Jul-19 5:55am

Member 14520059

Updated 3-Jul-19 21:16pm

KarstenK

v3

Add a Solution

Comments

Patrice T 3-Jul-19 11:59am

you forgot to tell us:
- the famous irritating messages.
- what it is supposed to do.
- what is not done as expected.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard MacCutchan · Accepted Answer · 2019-07-03T06:55:00

In your CashRegister class you are trying to instantiate an Inventory object with the statement:

C++

Inventory obj;

But in your Inventory class the only constructor for it is

C++

Inventory(int q, double c)

So the two do not match. It would be better to declare the object in the CashRegister as a pointer, so you can pass the created object in your CashRegister constructor call in main.

C++

class CashRegister
{
	private:
		Inventory* obj;
		int item, quantity;
		double total;
	public:
		CashRegister(Inventory* object, int i, int q){

// ...


int main()
{
	Inventory i(10, 11.5);
	
	int item, quantity;
	cout<<"Enter item and quantity:\n";
	cin>>item>>quantity;
	
	CashRegister c(&i, item, quantity);

Note: The name object is often a reserved word or class name in OOP languages, so it is best avoided as a variable name.