Discussion :

[Core8]

Bonjour à tous,

Voilà j'ai la classe suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class Objet {
 
	private :
		int attribut;
		int x;                
 
	public :
		Objet(int attribut, int x){
                       this->attribut = attribut;
                       this->x = x;
                }
};

Dans une autre classe je mets en attribut un tableau d'Objet.

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
class Test {
 
	private :
		Objet *tab;
                int taille;
 
	public :
		Test(int taille) {
                     this->taille = taille;
                     tab = new Objet[taille];
 
 	             for (int i = 0; i < taille; i++)
		           tab[i] = Objet();
		}
 
		~Test(){
                      delete [] tab;
                }
		...
};

Ce que je n'arrive pas à faire c'est tester si pour une case donnée du tableau un Objet a été affecté ou pas.
J'ai essayé dans le constructeur de la classe Test d'initialiser les cases du tableau à null mais ça ne marche pas.
C'est surement un truc tout bête mais en C++ je ne sais pas le faire.
Ce que je veux faire c'est quelque chose dans le genre :

Code :

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

1
2
"if (tab[x] == NULL)"
       // Alors la case est vide

Merci d'avance.

[Laurent Gomila]

Code :

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

1
2
3
Objet tab[10];
...
delete tab;

La règle est pourtant simple :
- new --> delete
- new [] --> delete []
Là en l'occurence tu n'as utilisé aucun des deux pour allouer ton tableau, donc tu n'as pas besoin de le libérer avec delete.

Ce que je n'arrive pas à faire c'est tester si pour une case donnée du tableau un Objet a été affecté ou pas.

Lorsque ton tableau est initialisé tous ses éléments sont de toute façon déjà correctement construits (par défaut -- ce qui fait que le code que tu nous donnes ne compilerait pas). En gros la boucle de ton constructeur ne sert à rien.
Deux solutions pour avoir un état "non initialisé" :
- Stocker des pointeurs (intelligents de préférence) : NULL sera une case non remplie
- Prévoir une instance spéciale de ta classe Objet qui représentera un objet non défini -- pas toujours utilisable, ça dépend de la classe