Discussion :

[khadijadadou]

bonjour à tous!

Code :

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

1
2
3
4
5
6
7
8
9
#include "sommet.h"
using namespace std;
#include<string>
 
Sommet::Sommet(string col,string nom)
{couleur=col;
Nom=nom;
 tab=new Sommet[100];
}

j'ai voulu créé un tableau dynamique dont les éléments sont de type "Sommet"
le compilateur me signale l'erreur suivant:

[Error] no matching function for call to 'Sommet::Sommet()

quelqu'un peut m'aider SVP !

[nikau6]

As tu créé un constructeur par défaut ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class Sommet
{
public:
   Sommet() : Couleur(""), Nom(""), tab(nullptr) {} // constructeur par defaut
   Sommet(std::string col, std::string nom);
 
private:
    std::string Nom;
    std::string Couleur;
    Sommet* tab;
};

le message d'erreur :" [Error] no matching function for call to 'Sommet::Sommet() ", te spécifie que la fonction membre "Sommet::Sommet()", que tu appelles en initialisant ton tableau, est introuvable. Le message est assez clair...Ça n'est pas toujours le cas mais là, si...

ps : Et n'oublies pas de désallouer la mémoire de ton tableau, dans le destructeur par exemple, pour éviter les fuites de mémoire.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class Sommet
{
public:
    Sommet() : Couleur(""), Nom(""), tab(nullptr) {}
    Sommet(std::string col, std::string nom);
    ~Sommet() { if(tab) { delete[] tab; tab = nullptr; } }
private:
    std::string Nom;
    std::string Couleur;
    Sommet* tab;
};

[JolyLoic]

Déjà, en C++, tableau dynamique se dit par défaut vector.
Ensuite, à chaque fois que tu construis un sommet, tu en construis 100 autres. Et chacun de ces 100 va-t-il en construire 100 autres,... Tu vas vite épuiser ta mémoire.
Par ailleurs, on préfère initialiser dans la liste d'initialisation, pas dans le corps du constructeur.

Enfin, ton erreur de compilation vient du fait que tu ne fournis pas de moyen de construire un sommet par défaut. Mais je pense que la bonne correction ne consiste pas à ajouter un constructeur pas défaut à sommet, mais à comprendre mieux ce que tu veux faire. J'ai l'impression que tu veux juste un conteneur pour stocker les sommets voisins du sommet courant. Dans ce cas, tu ne devras pas contenir des objets directement (car leur durée de vie serait lié à celle du sommet les contenant), mais des pointeurs (éventuellement intelligents) sur ces sommets.

A la construction, cette liste doit probablement être vide, donc pas besoin d'en faire quoi que ce soit de spécial. Éventuellement, si tu souhaites pré-réserver de l'espace pour ces éléments (ce qui est une optimisation (ou pas, selon ce que tu en fais) mais ne change rien au fonctionnel), tu peux utiliser reserve.

En résumé, je partirais plus dans la direction :

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
class Sommet
{
public:
   Sommet(std::string col, std::string nom);
 
private:
    std::string Nom;
    std::string Couleur;
    std::vector<Sommet*> tab;
};
 
 
Sommet::Sommet(string col,string nom) : couleur(col), Nom(nom)
{
  // Optionnel, probablement inutile voire néfaste :
  tab.reserve(100);
}

[khadijadadou]

MERCI à tous merci JolyLoic vous avez raison mon but était de créer un conteneur pour les sommets voisins du sommet courant,mais je n'ai pas procédé correctement!