Thread: class with enum and constructor

Hey guys
I really confused with constructor
(default constructor and constructor with parameters)

I coded this problem


and I worked almost, but I stock in constructor

Code:

class Tier 
{ 
public: 
enum TIER_MAKE 
{ 
nexen, 
kankook, 
kumho 
}; 
private: 
TIER_MAKE make; 
public: . 
Tier(TIER_MAKE p_make) :make( p_make ) {} 
void print( void ) 
{ 
switch ( make ) 
{ 
case nexen: 
{ 
cout << "nexen" << endl; 
} 
break; 
case kankook: 
{ 
cout << "kankook" << endl; 
} 
break; 
case kumho: 
{ 
cout << "kumho" << endl; 
} 
break; 
} 
} 
};

this is tier class and I have to finish constructor in class car
(for simple, I skip detail code) -red things are the parts from class Tier

Code:

 Car() 
: make(NULL), passengers(0), fuelcap(0.0), efficiency(0.0), tier(Tier::nexen) 
{ }

 Car(char *p_make, int p_passengers, int p_fuelcap, float p_efficiency, Tier::TIER_MAKE p_tiermaker) 
: passengers(p_passengers), fuelcap(p_fuelcap), efficiency(p_efficiency), tier(p_tiermaker)
 { }

 ~Car()
 { }

 Car(const Car &c) 
: passengers(c.passengers), fuelcap(c.fuelcap), efficiency(c.efficiency), tier(c.tier)
{}

Is the parts from class tier in class Car right?
I think default constuctor part should be written about p_tiermaker..
like this : p_tiermaker(Tier::nexen) but it rangs error.

And someone said default constructor part has to be this

Code:

car( Tier::TIER_MAKE p_tiermaker = Tier::nexen ) //after i skip

but default constructor should be no parameter...? isn't it?
I really confused... give me some words

Originally Posted by antirain

Is the parts from class tier in class Car right?

Don't know. All I see is that your initialize some member variable in the Car class. You tell us if it's right or not.

I think default constuctor part should be written about p_tiermaker..
like this : p_tiermaker(Tier::nexen) but it rangs error.

What error? For that matter, I don't see your definition for the Car class, so I can't even check if you implemented it right.

And someone said default constructor part has to be this

Code:

car( Tier::TIER_MAKE p_tiermaker = Tier::nexen ) //after i skip

but default constructor should be no parameter...? isn't it?
I really confused... give me some words

Yes, the default constructor have no parameters, but this one accepts an optional parameter, which means it does not have to be specified, in which case it acts as a default constructor.

Also, for your own sanity, please use std::string instead of char*. The later is very, very dangerous.
Indenting your code wouldn't be a bad idea, either.