C / C++ / MFC

BOOL CTCC::InitInstance()
{  
    m_pLogAE         = new A;  
    m_pXmlAEInternal = new B;  
    m_pXmlConfig     = new C;  
 
    CTCCDlg dlg;    
    m_pMainWnd = &dlg;      
    dlg.DoModal();    
    if(m_pLogAE)          delete m_pLogAE; // put breakpoint here
    if(m_pXmlAEInternal)  delete m_pXmlAEInternal;    
    if(m_pXmlConfig)      delete m_pXmlConfig;  
 
    return FALSE;
}

Joan Murt wrote:

...when I reach the bracket "}" and then I press F10 again then I get the application finished suddenly.

InitInstance
{
  if (A)
  {
  }
  else
  {
   dlg.DoModal();
<code><big>  }</big></code>
  
  if/delete clauses
  return FALSE;
}

Joan Murt wrote:

If I remove the code that is before that bracket and after POINT 1, and execute again the code I can see how if the code reach that bracket it gets stopped again and don't go further the POINT 2.

template<typename T>
class Ptr {
	public:
		explicit Ptr(T *pT = 0)
			: m_pT(pT) {
			}
		virtual ~Ptr() {
			erase(); }
	public:
		operator const T *() const {
			return (const T *)m_pT; }
		operator T *() {
			return m_pT; }
		T* operator->() {
			return m_pT; }
		T** operator&() {
			return &m_pT; }
		Ptr& operator=(T *pT) {
			m_pT = pT;
			return *this; }
		operator bool() const {
			return (m_pT != 0) ? true : false; }
		bool operator !() const {
			return (m_pT == 0) ? true : false; }
	protected:
		virtual void erase() {
			T *pT = m_pT;
			m_pT = 0;
			if(pT) {
				delete pT;
			} }
	protected:
		T *m_pT;
};

Ptr<A> m_pLogAE = new A;  
Ptr<B> m_pXmlAEInternal = new B;  
Ptr<C> m_pXmlConfig = new C;

Cristian Amarie wrote:

(or perhaps in dialog)

if(a)
{
}
else
{
CDialog a;
...
}
if/delete clauses

BOOL CTCC::InitInstance(){
  m_pLogAE = new A;
  m_pXmlAEInternal = new B;
  m_pXmlConfig = new C;
  if (!FunctionX() ||
      !m_pControl->FunctionY() ||
      !m_pXmlFile->LoadFile(csPath, csReason))  {
    DoSomething();
  }
  else  {
    CTCCDlg dlg;
    m_pMainWnd = &dlg;
    dlg.DoModal();
    m_pControl->RestoreSettings();
  }
  // POINT 2
  delete m_pLogAE;
  delete m_pXmlAEInternal;
  delete m_pXmlConfig;
  // Since the dialog has been closed, return FALSE so that we exit the
  //  application, rather than start the application's message pump.
  return FALSE;
}

Joan Murt wrote:

I could delete something that could be deleted before without any problem

Thing *pointer = new Thing;
// use pointer
// ...
// conditionally destroy it
if (SomethingHappens()) {
  delete pointer;
  pointer = NULL;
}
// do more stuff
// ...
// final, unconditional cleanup
delete pointer;
pointer = NULL;