problème allocation tableau dynamique

Bonjour à tous

comme vous l'aurez deviné, je sèche sur un problème d'allocation d'un tableau dynamique dans un constructeur.

Pour vous situer le contexte, je dois construire une File d'attente avec priorité pour gérer des patients, j'ai donc entre autres les classes:
Patient
FileAt -> une file d'attente de patients
FAP -> une File d'attente avec priorité de patients, gérée à l'aide d'un tableau de FileAt

Mon problème se situe au niveau du constructeur de FAP, dont voici le code

Code:

---

1 2 3 4 5

FAP::FAP() { t = new FileAt[MAXI]; }

---

MAXI étant une constante correspondnant à la priorité maximale que je puisse avoir.

Dans mon main j'ai une seule et unique instruction:

Code:

---

1 2 3 4 5 6

int main() { FAP f; return 0; }

---

et à l'exécution j'ai droit, à

Citation:

---

segmentation fault(core dumped)

---

Ce qui m'étonne c'est que si dans le main, à la place de la déclaration de l'objet, je construit "manuellement" un tableau du même type, cela ne pose aucun problème.

Si quelqu'un a une idée, merci d'avance

Salut.

Code:

---

t = new FileAt[MAXI];

---

Tu définis pas de type pour ta variable.
Même si apparemment sa compile, c'est pas vraiment propre...
Normalement tu devrais faire :

Code:

---

FileAt* t = new FileAt[MAXI];

---

En définissant ta variable comme étant un pointeur tu peux l'allouer dynamiquement.

si si le type de la variable est bien défini c'est un attribut de ma classe, j'aurais peut être du préciser un peu plus:

voilà la déclaration de la classe:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

const int MAXI = 20; class FAP { public: FAP(); FAP(const FAP& F); ~FAP();   FAP& add(Patient elem, int priorite); FAP& operator--(); FAP& operator=(const FAP& F); bool operator==(const FAP& F); bool operator!=(const FAP& F);   friend ostream& operator<<(ostream& flux, const FAP& f); Patient& get_element();   private: FileAt* t; int getPrioriteMax() const; };

---

Salut,

Comment défini tu ta fille d'attente (FileAt) :question:

Pourquoi ne pas, tout simplement, si tu as un constructeur par défaut pour ta file d'attente, utiliser la classe std::vector<FileAt> (ou, selon le cas, un std::vector<FileAt*> ):question:

Tu pourrais alors le déclarer sous la forme de

Code:

---

std::vector<FileAt> t;

---

et le définir sous la forme de

Code:

---

FAP::FAP():t(MAXI){}

---

et très facilement, pourquoi pas, prévoir de récupérer les différentes files d'attente sous une forme proche de

Code:

---

1 2 3 4 5 6 7 8 9 10

FileAt& FAP::operator[](size_t ind) { assert(ind<=MAXI); return t[ind]; //avec déréférencement si tu utilise un std::vector<FileAt*> } const FileAt& FAP::operator[size_t ind] const { assert(ind<=MAXI); return t[ind]; //avec déréférencement si tu utilise un std::vector<FileAt*> }

---

J'ai pas fais attention mais dans ton main() tu appelle pas ton constructeur.

On dois faire :

FAP t();

Peut être que ton erreur vien de là... (j'aime me sentir utile...)

Citation:

---

Envoyé par Namtar

JOn dois faire :
FAP t();
Peut être que ton erreur vien de là... (j'aime me sentir utile...)

---

En fait il l'appelle correctement... pour le compilo FAP t(); ça sera une définition de fonction.

Donc c'est bien FAP t;