Discussion :

[progfou]

Bonsoir tout le monde !
J'alloue un tableau de template (je ne sais pas si ça se dit...) ainsi :

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
23
24
25
26
27
template<class T>
class DynBuf
{
public:
  DynBuf(): m_uiBufferSize(0), m_pT(NULL) {}
 
  ErrVal init( UInt uiBufferSize )
  {
    ROT( NULL != m_pT );
    m_pT = new T [ uiBufferSize ];
    ROT( NULL == m_pT );
    m_uiBufferSize = uiBufferSize;
    return Err::m_nOK;
  }
 
  ErrVal uninit()
  {
    if(NULL != m_pT )
    {
    delete [] m_pT;
    m_pT = NULL;
    }
    m_uiBufferSize = 0;
 
    return Err::m_nOK;
  }
etc.

J'ai utilisé un détecteur de fuites mémoire, et il me dit que l'allocation faite avec le new pose problème (il ne me dit pas ce qui ne va pas ).
Comme je dois être bigleux, est-ce que l'un de vous voit l'erreur ?

Merci d'avance !

[roulious]

Tu as une fuite de mémoire possible si tu appelles init deux fois de suite : dans ce cas, l'adresse du bloc précédemment alloué est perdue.

[loufoque]

acquiert les ressources dans le constructeur et libère les dans le destructeur.

[progfou]

Peux-tu détailler ?
Je ne comprends pas trop...

[roulious]

Il pense à l'idiome du RAII. La mémoire est allouée dans le constructeur et libérée dans le destructeur, ce qui te garantit une gestion correcte et pas de fuite.

[progfou]

Ca suppose que je change tout le code...
Effectivement, ce serait nettement meilleur !
Je vais regarder ça.

Merci !