2 class / no default constructor available

[stephane]

I can't make this work it says I dont have a default constructor available.

Can someone tell me what's wrong with this please?

#include <iostream>
using namespace std;

#include "points.h"

int main ()

{
Trectangle rec1(2,5,10,6);

Trectangle rec2(rec1);

return 0;
}
// points.h

#ifndef POINTS_H
#define POINTS_H

#include <iostream>
using namespace std;

class Tpoints{

public:

Tpoints(double x,double y):m_x(x),m_y(y )
{
cout<<"constr points 1"<<endl;
}

Tpoints(Tpoints & p)
{
m_x = p.m_x;
m_y = p.m_y;

cout<<"constr copie Tpoints"<<endl;
}
private:

double m_x;
double m_y;
};
class Trectangle{

public:

Trectangle(doub le x,double y, double larg, double haut)
: m_hautGauche(x, y), m_basDroite(x+l arg, y+haut){cout<<" constr rec
1"<<endl;}

Trectangle(Trec tangle& r)
{
cout<<"constr copie rec"<<endl;

m_hautGauche = r.m_hautGauche;
m_basDroite = r.m_basDroite;
}
// no appropriate default constructor available


private:

Tpoints m_hautGauche;
Tpoints m_basDroite;
};

#endif

[Karl Heinz Buchegger]

stephane wrote:

I can't make this work it says I dont have a default constructor available.

Can someone tell me what's wrong with this please?

Exactly as the compiler indicates: There is no default constructor
( = one that takes no arguments ) for class Tpoints

So why does the compiler need one?
Easy: Because in class Trectangle, there are 2 members of type Tpoints.
Also class Trectangle has a copy constructor. But in this copy constructor
you do not specify how the Tpoints members should be constructed. Thus the
compiler tries to use the default constructor to do that. But there is none.
Trectangle( const Trectangle& r ) : m_hautGauche( r.m_hautGauche ),
m_basDroite( r.m_basDroite )
{
cout << "const copie rec" << endl;
}

Now the compiler is instructed to use the copy constructor for the Tpoints
members.

--
Karl Heinz Buchegger
kb******@gascad .at

[Dietmar Kuehl]

stephane wrote:

Trectangle(doub le x,double y, double larg, double haut)
: m_hautGauche(x, y), m_basDroite(x+l arg, y+haut){cout<<" constr rec
1"<<endl;}

Trectangle(Trec tangle& r)
{
m_hautGauche = r.m_hautGauche;
m_basDroite = r.m_basDroite;
}

You need to use a member initializer list for this constructor, too:

| Trectangle(Trec tangle const& r):
| m_hautGauche(r. hautGauche),
| m_basDrote(r.m_ basDrote)
| { /* ... */ }

I also added a 'const' to the parameter as you don't modify the
object. However, I haven't verified that 'TPoint's copy constructor
is correct declared. BTW, note that you don't need to provide this
copy constructor: if you don't mention it, the compiler will
automatically create an appropriate version in this case - it knows
how to copy the members...
--
<mailto:di***** ******@yahoo.co m> <http://www.dietmar-kuehl.de/>
<http://www.contendix.c om> - Software Development & Consulting

[Stephane Vollet]

If I write Trectangle(Trec tangle&
r):m_hautGauche (r.m_hautGauche ),m_basDroite(r .m_basDroite)

It compiles ok, but if I add "const" like this

Trectangle(cons t Trectangle&
r):m_hautGauche (r.m_hautGauche ),m_basDroite(r .m_basDroite)

it says there isn't any copy constructor! Why's that?
#ifndef POINTS_H
#define POINTS_H

#include <iostream>
using namespace std;

class Tpoints{

public:

Tpoints(double x,double y):m_x(x),m_y(y )
{
cout<<"constr points 1"<<endl;
}

Tpoints(Tpoints & p)
{
m_x = p.m_x;
m_y = p.m_y;

cout<<"constr copie Tpoints"<<endl;
}

private:

double m_x;
double m_y;
};

class Trectangle{

public:

Trectangle(doub le x,double y, double larg, double haut)
: m_hautGauche(x, y), m_basDroite(x+l arg, y+haut){cout<<" constr rec
1"<<endl;}

Trectangle(Trec tangle&
r):m_hautGauche (r.m_hautGauche ),m_basDroite(r .m_basDroite)
{
cout<<"constr copie rec"<<endl;

}
// no appropriate default constructor available

private:

Tpoints m_hautGauche;
Tpoints m_basDroite;
};

#endif
"Dietmar Kuehl" <di***********@ yahoo.com> a écrit dans le message de news:
11************* *********@z14g2 00...legr oups.com...

stephane wrote:

Trectangle(doub le x,double y, double larg, double haut)
: m_hautGauche(x, y), m_basDroite(x+l arg, y+haut){cout<<" constr rec
1"<<endl;}

Trectangle(Trec tangle& r)
{
m_hautGauche = r.m_hautGauche;
m_basDroite = r.m_basDroite;
}

You need to use a member initializer list for this constructor, too:

| Trectangle(Trec tangle const& r):
| m_hautGauche(r. hautGauche),
| m_basDrote(r.m_ basDrote)
| { /* ... */ }

I also added a 'const' to the parameter as you don't modify the
object. However, I haven't verified that 'TPoint's copy constructor
is correct declared. BTW, note that you don't need to provide this
copy constructor: if you don't mention it, the compiler will
automatically create an appropriate version in this case - it knows
how to copy the members...
--
<mailto:di***** ******@yahoo.co m> <http://www.dietmar-kuehl.de/>
<http://www.contendix.c om> - Software Development & Consulting

[Victor Bazarov]

Stephane Vollet wrote:

If I write Trectangle(Trec tangle&
r):m_hautGauche (r.m_hautGauche ),m_basDroite(r .m_basDroite)

It compiles ok, but if I add "const" like this

Trectangle(cons t Trectangle&
r):m_hautGauche (r.m_hautGauche ),m_basDroite(r .m_basDroite)

it says there isn't any copy constructor! Why's that?
[...]

Tpoints(Tpoints & p) ^^^^^^^^^^
Because of this. Shouldn't this be 'const' as well?
{
m_x = p.m_x;
m_y = p.m_y;
Prefer initialisation over assignment. See FAQ.

cout<<"constr copie Tpoints"<<endl;
}
[...]