Discussion :

[Invité]

Bonjour,
A la vue de cette discussion : http://www.clubic.com/forum/programmation/creer-variable-dynamique-en-c-id622914-page1.html, je me pose la question si l'on peut avec la classe QVector, instancier une objet dans le ^parmaètre de la fonction push_back, en analogie avec la classe std::vector en c++ ?
Je peux, par exemple instancier un objet dynamiquement en faisant :

Code :

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

1
2
3
4
5
 
int n;
QVector<maclasse> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(maclasse); // remplit le premier élément du tableau avec un objet de type ma classe instancié dans le paramètre de la fonction push_back() ???... Ici j'aurais pu utiliser la fonction append()

ou bien :

Code :

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

1
2
3
4
int n;
QVector<maclasse> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(new maclasse);

Je vous demande car cette écriture me choque un peu.
Dans le cas où c'est possible, pouvez-vous expliquer comment ? Merci d'avance.

[LittleWhite]

Bonjour,

Je dois dire que cela ne me choque pas.
Sauf que, si on lit bien le code du deuxième exemple:

Code :

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

1
2
3
4
5
 
int n;
QVector<maclasse> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(new maclasse);

IL y a une faute (que le compilateur sera heureux de vous narguer avec :p)

Code :

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

1
2
3
4
5
int n;
QVector<maclasse*> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(new maclasse);

Maintenant, pour ce code avec pointeurs, il faudre penser à libérer les pointeurs :p

[yan]

salut

Code :

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

1
2
3
4
int n;
QVector<maclasse> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(maclasse(...) );

tu remplace les ... par les patametres du constructeur.

[Invité]

ah d'accord je ne savais pas. En fait ici on instancie un objet sans lui donner de nom...
après chacun fait ce qu'il veut mais personnellement je me sens plus à l'aise avec ce code :

Code :

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

1
2
3
4
5
int n;
maclasse *obj=new maclasse;
QVector<maclasse*> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(obj);

Je ne sais pas... Pour moi ce code est bien plus clair : on donne un nom à l'objet instancié.
Instancier des objets sans nom n'est-il pas risqué ?

[ymoreau]

Une instance n'est pas un nom mais une adresse mémoire, le nom que tu lui donnes sert juste à être utilisé dans le code, pour être lisible. Mais dans le code de yan ça n'est pas ça qui est important, c'est qu'il déclare des objets sans new, qui sont donc copiés dans le vecteur, et seront automatiquement nettoyés quand le vecteur sera détruit.

Ce qui n'est pas le cas dans ton code, qui en plus est faux puisque tu ne crées un objet qu'une seule fois et tu vas copier dans ton vecteur la même adresse mémoire à chaque tour de boucle. Il faudrait faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
int n=12;
maclasse *obj;
QVector<maclasse*> listobj;
for (int i=0;i<n;i++)
{
     obj=new maclasse;
     listobj.push_back(obj);
}

Mais tu pourrais le faire sans nom, ça serait la même chose

Code :

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

1
2
3
4
int n=12;
QVector<maclasse*> listobj;
for (int i=0;i<n;i++)
     listobj.push_back(new maclasse);

L'important à retenir c'est qu'en utilisant new, il faudra penser à détruire tous les objets créés, avec delete.

[Invité]

D'accord merci.