Discussion :

[hary66]

Bonjour.

J'essaie de faire cet exercicehttps://erlerobotics.gitbooks.io/erl...cises_oop.html dont la solution est donnée :

Write a program that defines a shape class with a constructor that gives value to width and height. The define two sub-classes triangle and rectangle, that calculate the area of the shape area (). In the main, define two variables a triangle and a rectangle and then call the area() function in this two varibles.

La solution proposée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
// Exercises: OOP#include <iostream>
// Exercise 1
 
using namespace std;
 
class Shape
{
protected:
	float width, height;
public:
	void set_data (float a, float b)
	{
		width = a;
		height = b;
	}
};
 
class Rectangle: public Shape
{
public:
	float area ()
	{
		return (width * height);
	}
};
 
class Triangle: public Shape
{
public:
	float area ()
	{
		return (width * height / 2);
	}
};
 
int main (){
 
	Rectangle rect;
	Triangle tri;
	rect.set_data (5,3);
	tri.set_data (2,5);
	cout << rect.area() << endl;
	cout << tri.area() << endl;
	return 0;
}

J'étais personnellement parti avec l'idée d'utiliser un constructeur au lieu d'utiliser une fonction "void set_data (float a, float b)" , comme ci-dessous :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include <iostream>
using namespace std;
 
class Shape {        // The class
protected:
int hauteur;
int largeur;
  public:          // Access specifier
 
    Shape(int x, int y) {  // Constructor with parameters
      hauteur = x;
      largeur = y;
    }
};
class Rectangle : public Shape {
public:
 
float area (){
return (hauteur * largeur);
}
};
 
 class Triangle : public Shape {
 public:
 
float area() {
return (hauteur * largeur/2.0);
};
 };
 
int main() {
  Rectangle r(10, 99);
  Triangle t(3, 5);
 
 
  // Print values
  cout << "Aire du rectangle : " <<r.area() << "\n";
  cout << "Aire du triangle : " << t.area() << "\n";
  return 0;
}

Or, ça ne fonctionne pas et je ne vois pas comment corriger l'erreur ?

[Bousk]

Les constructeurs ne sont pas propagés aux classes filles.
Pour les exposer identiques, il y a la syntaxe using BaseClassName::BaseClassName;.

[dalfab]

Bonjour,

Pour construire une Shape, il faut lui fournir 2 arguments.

Les Rectangle et Triangle sont des Shape. Mais tu n'indiques pas comment les construire. Ton idée est surement que ça soit comme une Shape. Pour cela tu dois définir les constructeurs ou dire que c'est comme Shape.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
class Shape
{
public:
	Shape( double width, double height ) : width_{width}, height_{height} {}  // fabriquer les membres au plus tôt
        virtual  area()const = 0;                 // indiquer que toutes les Forme ont une aire et que la lire ne modifie aucune Forme
        double  width()const { return  width_; }   // lire la largeur, ne modifie pas la Forme
        void  width( double w ) { width_ = w ; }
        double  height()const { return  height_; } // pour écrire/lire la hauteur
        void  height( double h ) { height_ = h; }
private:                              // Ne jamais définir d'attribut protected, c'est public ou private
	double  width_, height_;      // éviter d'utiliser le type float, double est à préférer sauf rares exceptions
};
class Rectangle : public Shape
{
public:
	using Shape::Shape;     // Rectangle se construit en utilisant directement les constructeurs de Shape
	double  area ()const override {
		return  width() * height();
	}
};
// ou
class Triangle : public Shape
{
public:
	Triangle( double width, double height ) : Shape{width, height} {}
	double  area()const override {
		return  width() * height() / 2;
	}
};

[hary66]

Merci pour ces réponses.

@dalfab
A la ligne 4 :

Shape( double width, double height ) : width_{width}, height_{height} {} // fabriquer les membres au plus tôt

Je ne comprend pas cette partie :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 : width_{width}, height_{height}

D'autre part, je ne comprend pas ce que fait ce constructeur, il n'y a rien dans le corps de la fonction ??

[JolyLoic]

Il initialise la donnée membre width_ à l'aide de l'argument width passé au constructeur, idem pour heigth. Du coup (et comme c'est très souvent le cas), il ne reste plus rien à faire dans le corps du constructeur, tout est déjà initialisé comme il faut.

[Bousk]

Ça s'appelle la liste d'initialisation https://cpp.developpez.com/faq/cpp/?...-l-affectation