How to initialize an array of objects of a parameterized constructor

Question

I need to initialize an array of objects of a parametrized constructor . How can i do it in best possible way ?

   # include<iostream>
   # include<conio.h>
   # include<stdio.h>

   using namespace std;

   class A
   {
    public:
    int a;       
    A();
    A(int x)
    {
     a=x;
    }      
   };
   int main()
   {
    A *a1,a2(1);
    a1 = (A*)malloc(sizeof(A)*10); // equivalent to A[10]. 
    for(int i=0;i<10;i++) a1[i]=a2; // Initialization is important in any program.
    for(int i=0;i<10;i++) cout<<a1[i].a; 
    getch(); 
    return 0;   
   }

This does work but is there some other way better than this ?

Why C's malloc and getch() instead of C++'s new and using iostream? — Cameron
– Cameron, Commented Feb 14, 2012 at 18:24
"This does work" No it doesn't; it invokes UB and only appears to work. — ildjarn
– ildjarn, Commented Feb 14, 2012 at 19:42
@ildjarn What do you mean by it invokes only UB ? I did not get what you mean to say by UB ? — Invictus
– Invictus, Commented Feb 14, 2012 at 19:45

Bo Persson · Accepted Answer · 2012-02-14 18:22:38Z

4

The C++ way would be to use a std::vector.

std::vector<A>   a1(10, 1);

creates 10 A's initialized by 1.

answered Feb 14, 2012 at 18:22

Bo Persson

92.9k31 gold badges153 silver badges211 bronze badges

Sign up to request clarification or add additional context in comments.

8 Comments

Joel Falcou Over a year ago

This requires A(int) to be not explicit I think ?

Bo Persson Over a year ago

Probably, but it isn't in the example.

Joel Falcou Over a year ago

sure, wanted to be sure as I wanted to answer the very same thing but thought something was maybe amiss :)

Bo Persson Over a year ago

The vector will destroy itself when it goes out of scope (at the end of main). Or you can call clear if you want to empty it earlier.

Bo Persson Over a year ago

@Ritesh - You cannot just declare a destructor, you have to define it as well. But the above class doesn't need one.

|

Joel Falcou · Accepted Answer · 2012-02-14 18:22:30Z

0

This is solved using std::vector constructor taking size and base element :

A a2(1);
std::vector<A> tab(10, a2);

answered Feb 14, 2012 at 18:22

Joel Falcou

6,3871 gold badge19 silver badges35 bronze badges

Comments

Mooing Duck · Accepted Answer · 2012-02-14 19:49:44Z

-1

Note that malloc does not construct objects, and so calling a1[i]=a2 is bad form. It probably seems to work fine since they are POD-ish objects, but this is not the proper way to do C++. It is undefined behavior, which is completely unpredictable. It may work ten thousand times in a row, and then erase your bank account. You should use new instead, which also constructs. Or better yet, use a vector, like the other answers suggest. Also, be sure that the default constructor initializes the data, and initialization will be less of a worry.

If you really must use malloc, the "correct way" to initialize is this:

std::uninitialized_copy(a1, a1+10, a2); //constructs and assigns

which is roughly equivalent to:

{
    int i=0;
    try {
        for(i=0; i<10; ++i)
           new(a1+i)A(a2); //constructs and initializes in the buffer
    } catch(...) { 
        try {
            for(; i>=0; --i)
               (a1+i)->~A(); //destroy if an error occured
        } catch(...) { 
            std::terminate();
        }
        throw;
    }
}

edited Feb 14, 2012 at 19:49

answered Feb 14, 2012 at 18:27

Mooing Duck

67.6k19 gold badges107 silver badges168 bronze badges

5 Comments

Bo Persson Over a year ago

I wouldn't expect the A(int) constructor to throw. Or the default copy constructor.

Mooing Duck Over a year ago

@BoPersson: true, but when he does this again with std::string, and is surprised that his program crashes? I'll teach the generic correct (and easy!) way.

ildjarn Over a year ago

"and I'm pretty sure it's undefined behavior" It definitely is.

Mooing Duck Over a year ago

@ildjarn: I wasn't 100% sure, since the definition of POD has changed, and PODs are exceptions to some relevant rules.

Invictus Over a year ago

@MooingDuck +1 For Genric Correct way :)

Collectives™ on Stack Overflow

How to initialize an array of objects of a parameterized constructor

3 Answers 3

8 Comments

Comments

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

8 Comments

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related