Discussion :

[StingerBell]

Bonjour,

je voudrais déclarer un tableau dans une classe dont la taille dépends d'un autre paramètre de la classe. Comment puis-je faire cela ? Par exemple si je fais simplement comme paramètre de la classe :

Code :

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

1
2
int taille;
double tableau[];

et après dans mon code je ne sais pas comment définir le tableau pour lui dire quand fait tableau est un tableau de taille "taille" ? J'ai aussi essaye de déclarer dans les paramètres de la classe :

Code :

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

1
2
int taille;
double tableau[taille];

sans succès

Merci d'avance pour votre aide !

[ternel]

la taille du tableau doit être une constante à la déclaration de la classe (pas seulement à l'appel du constructeur).

Tu peux faire appel à std::vector.

Code :

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

1
2
3
4
5
6
7
class Bidule {
    vector<int> tableau;
 
public:
    Bidule(int taille) : tableau(taille, 0) {}
 
};

[Jimmy91]

Sinon, à l'ancienne on aurait ça :

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
 
// truc.h
class truc
{
	int* tab;
	int taille;
	truc();
};
 
//truc.cpp
//#include "truc.h"
truc::truc()
{
	tab = new int[taille];
}

Mais je suis d'accord avec leternel, c'est mieux d'utiliser les vectors.

[ternel]

et encore, ton code n'est pas très correct, il manque un destructeur, et une garantie de validité

ca serait plutot comme ceci. (en C++11, sinon, enlever le =delete et ne pas définir les fonctions)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Bidule {
private:
    int taille;
    int* data;
 
public:
    Bidule(int n) : taille(n), data(new int[n]) {}
    ~Bidule() {delete[] data;}
 
// non copiable, mais on pourrait faire une copie propre (faq: copy and swap)
private:
    Bidule(Bidule const&) = delete;
    Bidule& operator=(Bidule const&) = delete;
};

Une meilleure version C++11 serait:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class Bidule {
private:
    int taille;
    std::unique_ptr<int[]> data;
 
public:
    Bidule(int n) : taille(n), data(new int[n]) {}
 
private:
    Bidule(Bidule const&) = delete;
    Bidule& operator=(Bidule const&) = delete;
};

[Bousk]

Et surtout en écrivant new int(taille) j'espère que tu t'attends pas à avoir plus que 1 entier.

[Jimmy91]

Et surtout en écrivant new int(taille) j'espère que tu t'attends pas à avoir plus que 1 entier.

Tu as raison, il fallait écrire new int[taille] ? Merci pour cette correction !