Thread: vector<float*> push_back() problem

Hello,

I have a problem with the vector<vector<float*>>. Here is the code:

Code:

	std::vector<float*> one_vector;
	for(int i=0; i< 49; i++)
	{
		float one_float[3]={(float)i, (float)i, (float)i};
		one_vector.push_back(one_float);
	}

With breadpoint, I saw that, after each push_back, one_vector changes unexpectly.
when i=0, one_vector={0.0};
when i=1, one_vector={1.0, 1.0};
when i=2, one_vector={2.0,2.0, 2.0};
...

Have no idea, why the old elements are all changed after push_back().
Does anyone know what's going on there?
Thanks in advance.

You are storing pointers to a local array that goes out of scope after each iteration (the pointer becomes invalid almost immediately) . Essentially you are storing a pointer to the same memory location over and over, overwriting the contents of that memory each time.

It is somewhat hard to tell from this example what you want to achieve.

Indeed, to store a fixed-sized array in a vector, make it copyable by wrapping it up in a struct. To maximize reusability, you can make the struct a template:

Code:

#include <vector>

template <class T, unsigned Size>
struct ArrayWrapper
{
    T array[Size];
};

int main()
{
    std::vector<ArrayWrapper<float, 3> > one_vector;
    for(int i=0; i< 49; i++) {
        ArrayWrapper<float, 3> arr = {{ float(i), float(i + 1), float(i + 2) }};
        one_vector.push_back(arr);
    }
}

In fact you might find a more convenient version of the above in boost: Boost.Array. You may also check if your SC++L implements tr1 (planned library additions in C++0x).