Discussion of article "The Basics of Object-Oriented Programming"

New article The Basics of Object-Oriented Programming is published:

Author: Дмитрий

I have two examples of code based/copied from this article.  They both create an array of pointers to a class and then delete them upon exit.  However, one exits with a memory leak the other exits without a memory leak.  The only difference is the 2nd example has the 'Alert' in the destructor commented out.  When the alert in the destructor is removed there is a memory leak.  This is very weird.  Can anyone explain???  Please help, this is driving me CRAZY.

At the end of the examples of code from this article I have included an extremely simple example of code i wrote that also has a memory leak.  Again, why????  There is nothing complicated about this final example of code... 

 This code works without a memory leak: 

class CName
  {
private:
   int               m_arg; // Variable for the instance
public:
                     CName(int aArg)
     { // Constructor
      m_arg=aArg;
      //Alert("Constructor "+IntegerToString(m_arg));
     }
                    ~CName()
     { // Destructor
      Alert("Destructor "+IntegerToString(m_arg));
     }
  };
//---
CName *cname[]; // Array

void OnInit()
  {
// Prepare array to load ten instances of class
   ArrayResize(cname,10);

   for(int i=0;i<10;i++)
     { // Load instances
      cname[i]=new CName(i);
     }
  }
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
{
   for(int i=0;i<10;i++)
     { // At the end unload all instances from memory
      delete(cname[i]);
     }
}

Messages from expert log, note no leaked memory: 

PI 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 0

KP 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 1

JG 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 2

EN 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 3

LF 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 4

OM 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 5

FD 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 6

IK 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 7

HS 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 8

CJ 0 OOP_sConstDestr_2 (EURUSD,M1) 13:57:59 Destructor 9 

This code exits WITH a memory leak!!!!   Why?????????  The only difference is commenting out the 'Alert' in the destructor...

class CName
     {
private:
   int               m_arg; // Variable for the instance
public:
                     CName(int aArg)
     { // Constructor
      m_arg=aArg;
      //Alert("Constructor "+IntegerToString(m_arg));
     }
                    ~CName()
     { // Destructor
      //Alert("Destructor "+IntegerToString(m_arg));
     }
  };
//---

CName *cname[]; // Array

void OnInit()
  {
// Prepare array to load ten instances of class
   ArrayResize(cname,10);

   for(int i=0;i<10;i++)
     { // Load instances
      cname[i]=new CName(i);
     }
  }

//+------------------------------------------------------------------+
void OnDeinit(const int reason)
{
   for(int i=0;i<10;i++)
     { // At the end unload all instances from memory
      delete(cname[i]);
     }
}

 Messages from expert log, note leaked memory:

FM 1 OOP_sConstDestr_2 (EURUSD,M1) 13:51:19 10 undeleted objects left

EG 1 OOP_sConstDestr_2 (EURUSD,M1) 13:51:19 10 objects of type CName left

GO 1 OOP_sConstDestr_2 (EURUSD,M1) 13:51:19 200 bytes of leaked memory 

 Final example, my code, very simple but exits with a memory leak....

class CCandleStick
{
public:
   CCandleStick() { };
  ~CCandleStick() { };
};

CCandleStick *cCandleArray[];

//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+
void OnInit()
  {
   ArrayResize(cCandleArray, 10);
   for (int i = 0; i < 10; i++) {
      cCandleArray[i] = new CCandleStick();
   }   
  }
//+------------------------------------------------------------------+
//| Expert deinitialization function                                 |
//+------------------------------------------------------------------+
void OnDeinit(const int reason)
  {
   for (int i = 0; i < 10; i++) {
      delete(cCandleArray[i]);
   }   
   Print("Last Error- ", GetLastError(), "   Current time: ", TimeCurrent());
  }

Messages from expert log, note leaked memory: 

MR 0 tester (EURUSD,M1) 14:07:23 Last Error- 0   Current time: 2013.02.06 23:07:23

FL 1 tester (EURUSD,M1) 14:07:23 10 undeleted objects left

KN 1 tester (EURUSD,M1) 14:07:23 10 objects of type CCandleStick left

JS 1 tester (EURUSD,M1) 14:07:23 160 bytes of leaked memory

Checked all three varaints. All three are identical. All are normal. None gives leaks. In these examples, it can not be - we have the array, in each element of the array have instance, when finishing we delete all objects. Objects do not create copies of themselves, you can not miss to delete samething. If object creates the copy of himself, then we can have difficulties with deleting, very easy to miss something. Show variants that realy have leaking.

Use SRC button to insert code (better - attach files).

Sorry for my english:)

Here is the service desk message:

In the current build of the terminal (756) You can fix this by adding a string type member to the class CName.