Discussion :

[ofinot]

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 :

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

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 :

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

1
2
3
4
5
6
 
int main()
{
   FAP f;
   return 0;
}

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

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

[Namtar]

Salut.

Code :

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

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 :

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

FileAt* t = new FileAt[MAXI];

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

[ofinot]

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 :

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
 
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;
};

[koala01]

Salut,

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

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*> )

Tu pourrais alors le déclarer sous la forme de

Code :

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

std::vector<FileAt> t;

et le définir sous la forme de

Code :

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

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 :

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

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*>
}

[Namtar]

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...)

[nikko34]

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;