NewLC - Simplify Two Phase Constructor in Symbian - Comments

Re: Simplify Two Phase Constructor in Symbian

RaviRawat — Wed, 28 May 2008 10:59:37 +0200

this information is enough to understand the 2phase construction.
thanks.

Re: Simplify Two Phase Constructor in Symbian

dennis_george — Wed, 20 Jun 2007 12:06:14 +0200

Something like...

#define declareNewLC(T) static T* NewLC()
#define defineNewLC(T) T* T::NewLC() \
        {\
        T* self = new (ELeave) T();\
        CleanupStack::PushL(self);\
        self->ConstructL();\
        return self;\
        }\

class CMyTest
        {
        public:
                IMPORT_C declareNewLC(CMyTest);
        
        private:
                CMyTest(){};
                void ConstructL(){};
        };

EXPORT_C defineNewLC(CMyTest);

Re: Simplify Two Phase Constructor in Symbian

dennis_george — Wed, 20 Jun 2007 12:02:18 +0200

You can simply make the constructor "protected".

But the provided solution will only be utilized for small subset of the problem. Because it cannot be used for
* Parametrized constructors.
* Exporting NewL and NewLC.
* These NewL functions are inlined. So will be slightly rigid in design (barrier for BC).

So I think a slightly better solution would be to use macros. Instead of creating the template class.

Simplify Two Phase Constructor in Symbian

vinay — Thu, 28 Dec 2006 11:17:25 +0100

This design forces the default constructor of the CUsePhase class to be public, which should be avoided as users should not be allowed to create objects of this class on stack. ('C' type classes).

Vink

Simplify Two Phase Constructor in Symbian

vinay — Fri, 22 Dec 2006 09:53:47 +0100

This design forces the default constructor of the CUsePhase class to be public, which should be avoided as users should not be allowed to create objects of this class on stack. ('C' type classes).

Simplify Two Phase Constructor in Symbian

Anonymous — Wed, 19 Jul 2006 17:10:33 +0200

Yeah, you would be overloading.

Simplify Two Phase Constructor in Symbian

[floyd] — Fri, 03 Mar 2006 18:24:16 +0100

And to be perfectly accurate, applying delete to a null pointer is valid in C++ as described in the ISO/IEC standard. It results in a no-op.

Simplify Two Phase Constructor in Symbian

dDast — Mon, 20 Feb 2006 13:25:22 +0100

if you want parameterized constructors then you must override NewL and NewLC and then from that has to call NewL of CPhaseBase.

This won't work if you have to initialise member references of your class, as they need to be passed in to the class's constructor.

Also you wouldn't be overriding NewL/NewLC from CPhaseBase as CPhaseBase functions take no parameters and therefore have different function signatures. And hey, they're not even virtual anyway.

Simplify Two Phase Constructor in Symbian

fwojtek — Thu, 16 Feb 2006 16:05:51 +0100

>>>> 1) In C++ you are allowed to delete NULL. To be more particulare: Not in C++. -----> In Symbian C++...:)

Simplify Two Phase Constructor in Symbian

pat — Thu, 16 Feb 2006 14:27:04 +0100

I want to comment a small detail in the code listing:

CUsePhase ::~CUsePhase() { if(iMyBuf) { delete iMyBuf; iMyBuf = NULL; } }

There are two things which don't make sense. All you need there is a single row like this:

CUsePhase ::~CUsePhase() { delete iMyBuf; }

...because:

1) In C++ you are allowed to delete NULL. Members of C-based classes are always initialized to NULL during construction. You don't need the IF.

2) There's no need to set the member to NULL after deletion in destructor.

Simplify Two Phase Constructor in Symbian

vijay_k_r — Mon, 13 Feb 2006 07:43:10 +0100

For every class that is derived from the class template, code for a new class type will be generated leading to increased executable size.

Another way of reducing redundant code with out increasing the executable size is to use Macros for the function declarations and definitions.

Simplify Two Phase Constructor in Symbian

Girish — Mon, 13 Feb 2006 07:28:32 +0100

Thanks for the valuable information. Using the trick does cuts down the development time . Girish

Simplify Two Phase Constructor in Symbian

joshibhuvnesh — Fri, 10 Feb 2006 23:49:25 +0100

As we know that every time when we declare a class with initial C and derive from CBase we have to provide second phase constructor (NewL and NewLC ) in genral for such classes. There is a way which will give some flexibility to reduce to write NewL and NewLC evey time when you derive a class from CBase and need second phase constructor.