Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to properly static cast a vector in C++?

I have a code in which at the end of a function I need to cast from int to double all the elements of an array in order to being able to do a final push_back before exiting the function. The code I have right now is:

template <class T, size_t dims> class A {
    typedef typename std::array<int, dims> ArrayInt;
    typedef typename std::array<double, dims> ArrayDouble;
    typedef typename std::vector <ArrayDouble> VectorDouble;

/* ...*/

foo() {
   /*  ...*/

   ArrayInt myArrayInt;
   ArrayDouble myArrayDouble;
   VectorDouble myVectorDouble;

    /* Initialize myArrayInt 
    Do some other stuff */

    for (int i = 0; i < dims; ++i) 
        myArrayDouble[i] = static_cast<double>(myArrayInt[i]);

    myVectorDouble.push_back(myArrayDouble);
    }
}

It works properly, but I do not feel comfortable about the lines:

for (int i = 0; i < dims; ++i) 
    myArrayDouble[i] = static_cast<double>(myArrayInt[i]);

Is there any better way of doing this?

Thank you.

like image 797
Javi Avatar asked Feb 28 '14 13:02

Javi


2 Answers

You can use a function from algorithm.

With copy_n :

std::copy_n( myArrayInt.begin(), dims, myArrayDouble.begin() );

or with copy :

std::copy( myArrayInt.begin(), myArrayInt.end(), myArrayDouble.begin() );
like image 90
BЈовић Avatar answered Sep 21 '22 07:09

BЈовић


This can be written with less code, but it is explicit.

ArrayInt myArrayInt;
ArrayDouble myArrayDouble;
VectorDouble myVectorDouble;

/* Initialize myArrayInt 
Do some other stuff */

using std::transform;
using std::copy;
using std::begin;
using std::end;

// with explicit conversion
auto to_double = [](const int i) { return double{i}; };
transform(begin(myArrayInt), end(myArrayInt), begin(myArrayDouble),
    to_double);

// with implicit conversion
copy(begin(myArrayInt), end(myArrayInt), begin(myArrayDouble));
like image 44
utnapistim Avatar answered Sep 23 '22 07:09

utnapistim