Passing a reference array to template pointer array

Question

I have this class:

template<typename T>
class array_eraser
{
    T* array;
    int size_array;
public:

    array_eraser(T &a[], int s)
    {
        size_array = s;
        array = new T[s];
        array = a;
    }
}

I call the class with this line in main:

std::string langs[] = { "C++", "Haskell", "Ada", "Python", "Ada" };
array_eraser<std::string> le(langs, sizeof(langs) / sizeof(langs[0]));

and I got this error: cannot convert argument 1 from 'std::string [5]' to 'T *[] What did i do wrong?

array = new T[s]; array = a; Nice instant memory leak ;) allocating memory and immediately discarding it. — JHBonarius
– JHBonarius, Commented Dec 13, 2020 at 21:13
Does this answer your question? Passing an array by reference — JHBonarius
– JHBonarius, Commented Dec 13, 2020 at 21:16

Quimby · Accepted Answer · 2020-12-13 21:12:25Z

You are nicely demonstrating why not to use raw arrays.

T &a[] is not a reference to an array of Ts but an array a of references to T. Maybe you wanted T (&a)[] - a reference to an array. But that will not fly either. Because in this case you disable the decay mechanics and you have to specify the size. This can be done using the template trick:
```
template<std::size_t N>
array_eraser(T (&a)[N]) {}
```
If you really wanted T& a[], that is a no go, T a[] and T& a[] are never compatible.
If you use array_eraser(T a[], int s), the array is not passed by value, but decays into a pointer. There is no difference between T a[] and T* a. Which brings me to your second mistake. array = a; will not copy arrays but only assign the pointers leaking the newly allocated array. You can copy the array using std::memcpy(array,a,s);.

So, unless you really have to and know what you are doing, use std::array or better std::vector if you need run-time and modifiable length. More typing with std:: stuff is not a good enough reason.

Collectives™ on Stack Overflow

Passing a reference array to template pointer array

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related