Discussion :

[seco102]

Bonjour,

Je suis en train du code C++. Voici le morceau de code qui me génère l'erreur suivante: error: no matching function for call to 'ChauffeEau::ChauffeEau()':

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
#include "Noeud.h"
#include <queue>
#include "Message.h"
#include "ChauffeEau.h"
using namespace std;
Noeud::Noeud(ChauffeEau ch1,int inQuota1,std::queue<Message> rq1) {
	// TODO Auto-generated constructor stub
	ch = ch1;
	inQuota = inQuota1;
	for(int j = 0; (unsigned)j < rq.size(); j++){
		   rq.push(rq1.front());
		   rq1.pop();
	}
 
}

Chauffeau est une classe qui possède un constructeur avec des paramètre.

Je serais content si quelqu'un peut me dire ce qui ne va pas et comment y remédier.

Merci

[JolyLoic]

http://cpp.developpez.com/faq/cpp/?p...EUR_liste_init

[dragonjoker59]

Je suppose que ta classe Noeud possède un membrer de type ChauffeEau (ch ?).
Il faut savoir que dans un constructeur il vaut mieux utiliser les listes d'initialisation pour initialiser les membres de classe car sinon le constructeur fait appel au constructeur par défaut (donc sans argument) de chaque membre non initialisé dans ces listes d'initialisation.
Le code du constructeur que tu as écrit correspond plus ou moins à ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Noeud::Noeud(ChauffeEau ch1,int inQuota1,std::queue<Message> rq1)
	:	ch()
	,	inQuota()
	,	rq()
{
	ch = ch1;
	inQuota = inQuota1;
	for(int j = 0; (unsigned)j < rq.size(); j++)
	{
		rq.push(rq1.front());
		rq1.pop();
	}
 }

D'où ton problème de constructeur par défaut inexistant.
Tu peux résoudre ton problème avec les listes d'initialisation :

Code :

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

1
2
3
4
5
6
Noeud::Noeud(ChauffeEau ch1,int inQuota1,std::queue<Message> rq1)
	:	ch( ch1 )
	,	inQuota( inQuota1 )
	,	rq( rq1 )
{
 }

Je vois malgré tout un autre problème, c'est que tu utilises des passages d'argument par copie, ce qui fait que ta file de messages est copiée avant d'être passée à la fonction, ce qui peut être lourd en fonction de la taille de cette file. De même pour ch1.
Préfère les passages par référence (constante ou non), qui permettent de ne copier que l'adresse de la variable et non la variable complète.
J'aurais écrit ton constructeur ainsi :

Code :

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

1
2
3
4
5
6
Noeud::Noeud(ChauffeEau const & ch1, int inQuota1, std::queue<Message> const & rq1)
	:	ch( ch1 )
	,	inQuota( inQuota1 )
	,	rq( rq1 )
{
 }

EDIT :
Tu remarqueras que je ne passe pas inQuota1 par référence, une adresse étant au mieux de la même taille qu'un int (en 32 bits) au pire plus grande (en 64 bits)

[seco102]

Merci à vous tous. C'est résolu mon problème.