Discussion :

[Linschn]

Bonjour à tous,

Je suis face à un problème que je ne parviens pas à résoudre.
Soit la ligne suivante, où up et tangage sont des vecteurs (au sens géométrique, pas au sens de structure de stockage) :

Code :

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

1
2
 
      tangage->add(up->times(0.1));

La fonction add(Vecteur*) ajoute l'argument à son receveur, et la fonction times(double) renvoie le receveur multiplié par le scalaire passé en argument.

La méthode times(double) crée un nouveau Vecteur, il y a donc un toto=new Vecteur(blabla), et lui donne comme coordonnées les valeurs qui vont bien.
Est-ce que cet appel à new Vecteur(blabla) est à l'origine d'une fuite de mémoire ?

Je me pose la question car je ne vois pas à quel moment le resultat de up->times(0.1) sera détruit.

Si il y a effectuvement une fuite de mémoire, comment faut-il faire ?
Il faut avouer que

Code :

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

1
2
3
4
 
Vecteur* toto = up->times(0.1);
tangage->add(toto);
delete toto;

est un peu lourd, et encore, il n'y a qu'un niveau d'imbrication.

[Médinoc]

Ça dépend si add() "prend possession" du vecteur pointé ou non (ou s'il y a possibilité de récupérer le pointeur par la suite).

• Si oui, il n'y a pas de fuite.
• Si non, il y a fuite: Cela peut peut-être être résolu en utilisant des pointeurs intelligents (par exemple, faire retourner un pointeur intelligent par times()).

[Linschn]

Merci pour ces précisions. J'ignorais jusqu'à l'existence des pointeurs intelligents.

[koala01]

Salut,

Ceci dit, si les données du vecteur sont copiables, pourquoi ne pas utiliser les conteneurs de la STL, sans allocation dynamique de la mémoire de ta main

Tu modifie ton code pour que up->times() renvoie une référence constante sur le conteneur, et pour que add prenne... une référence constante vers le conteneur en tant qu'argument, "et le tour est joué"

Mieux, encore, tu n'expose pas une fonction times qui renvoie l'ensemble du vecteur, mais deux fonction (timeBegin() et timeEnd() ) qui renvoie... un itérateur constant, respectivement sur le début et sur l'itérateur qui suit le dernier élément de ton conteneur, et tu modifie la add en conséquence sous une forme proche de

Code :

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

1
2
3
4
5
 
retour LeTypeDeTangage::add(timeIterator & begin, timeIterator & end)
{
    /* ajout de la plage allant entre begin et end au membre */
}

qui sera appelée sous une forme proche de

Code :

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

tangage->add(uptime->timeBegin(1.0),uptime->timeEnd(1.0));

(le tout, avec un typedef définissant timeIterator comme... l'itérateur adequat)

[JulienDuSud]

Koala, l'auteur du post original a pourtant bien précisé que c'était des vecteurs au sens mathématique du terme et non au sens conteneur

Je dirais que t'es un peu HS

[koala01]

Effectivement, du moins, pour la deuxième partie de la réponse...

Cependant, je reste convaincu que le fait de passer par des pointeurs vers des Vecteurs reste une erreur monumentale :p

[3DArchi]

Merci pour ces précisions. J'ignorais jusqu'à l'existence des pointeurs intelligents.

Salut,
Un peu de lecture alors :
Présentation des pointeurs intelligents en C++ par Loïc Joly
Gérer ses ressources de manière robuste en C++ par Aurélien Regat-Barrel
Boost.SmartPtr : Les pointeurs intelligents de Boost par Matthieu Brucher

[Linschn]

Merci pour les liens. J'étais tombé sur les deux premiers en cherchant ce que sont les pointeurs intelligents.

Il y a toujours quelque chose que je me demande. Si la méthode times(double) (qui ne modifie pas son receveur) crée un nouvel objet avec new, et le renvoie cette fois-ci par référence, celui ci sera-t-il détruit proprement à la fin du bloc si :

• Il est stocké dans une variable ?

  Code :

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

  1
  2
  Vecteur tmp = toto->times(2.);
  titi->add(&tmp);

• Il est passé directement en argument de add(Vecteur&) ?

  Code :

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

  1
  2
   
  titi->add(toto->times(2.));

Mon utilisation forcenée des pointeurs vient du fait que je n'ai fait qu'une fois du C++, et je croyais connaitre le langage, mais j'ai surtout fait du C...

[Médinoc]

Il y a forcément l'un de ces codes qui est mauvais:

• Si add() prend possession du pointeur pour le détruire avec delete lors de la destruction de la liste, le premier code tentera de deleter un objet sur la pile lors de la destruction de la liste --> Aie!
• Si add() ne prend pas possession du pointeur, alors le pointeur retourné par times() sera sans doute "perdu" dans une fuite de mémoire, à moins qu'on ne le détruise explicitement (genre, s'il est possible de faire delete titi[index]).

Il faut plus de contexte pour dire si quelque chose est une fuite ou non, car il n'y a pas réellement de fuite tant qu'il existe un pointeur accessible sur les données...