Pass Vector of pointers to function expe

Forum

Forum
Beginners
Pass Vector of pointers to function expe

Pass Vector of pointers to function expecting an array

Mar 20, 2013 at 7:12pm

Hi,

I have a vector of pointers to objects, I need to pass this into a third party function which is expecting an array of objects, the function prototype goes like this:

void func(int *data, int size);

Which updates everything in data.

Here's an example:

#include <stdio.h>
#include <vector>

void func(int *data, int size);

int main()
{
	std::vector <int*> things;
	int a[10];

	for(int i = 0; i < 10; i++)
	{
		int *d = new int;
		*d = i + 100;

		things.push_back(d);

		a[i] = i + 100;
	}

	func(things[0], 10);
	printf("\n");

	func(a, 10);
}

void func(int *data, int size)
{
        //in real world this function does something to modify the data
	for(int i = 0; i < size; i++)
	{
		printf("%d\n", data[i]);
	}
}

Edit & run on cpp.sh

You'll see I've tried passing an array (a) to func which shows the correct output. The output is:


100
-33686019
98893109
-2013260835
18001016
17987184
1540103372
556
4
2

100
101
102
103
104
105
106
107
108
109

My question is why doesn't func produce the expected output when passing in my vector of pointers? It appears to handle the first item correctly but then it all goes wrong. I thought vectors stored data like an array (the next item immediately after the previous). How do I put this right?

-------------------------------------------------------------------------------
Bit of background:

I'm using a vector of pointers as I have a base class (gameObject) and a few derived classes (enemy, player, etc) so I can have a single list of all objects and call methods at certain times (e.g update function for each object etc) and I can add and remove new gameObjects when I like. In my example int* is just a replacement for gameObject*.

Any thoughts or feedback appreciated.

Thanks!

Mar 20, 2013 at 7:28pm

Fransje (435)

A std::vector is of another type than an array. But since it uses an array internally and it provides the data() function which returns that array, you can solve it like this:

func( things.data(), things.size() );

Last edited on Mar 20, 2013 at 7:28pm

Mar 20, 2013 at 7:34pm

closed account (28poGNh0)

# include <stdio.h>
# include <vector>

void func(int *data, int size);

int main()
{
	std::vector <int> things;

	for(int i = 0; i < 10; i++)
	{
		int d = i + 100;

		things.push_back(d);
	}

	func(&things[0], 10);
	printf("\n");
}

void func(int *data, int size)
{
    // call elements of things this way
    for(int i = 0; i < size; i++)
	{
		printf("%d\n", data[i]);
	}

	// Or this way
	/*for(int i = 0; i < size; i++,data++)
	{
		printf("%d\n", *data);
	}*/
}

Mar 20, 2013 at 8:31pm

pazza (8)

Hi,

Thanks for the replies.

Unfortunately things.data() doesn't work as data() returns a pointer so I end up trying to pass a pointer to a pointer (int**) when an int* is required. So this causes an error.

I want to keep the vector as a vector of pointers rather than a vector of ints.

Thanks,

Mar 20, 2013 at 8:52pm

Warnis (230)

so just dereference it like such:
*things.data();

Mar 20, 2013 at 9:12pm

pazza (8)

That results in the same output as originally listed, I don't understand why?

Mar 20, 2013 at 9:50pm

Warnis (230)

Hmm, testing it out myself I had similar results.
Changed the printing line to
printf("%d\n", *data+i);
and it works just fine

Mar 20, 2013 at 11:29pm

cire (8284)

If func is expecting an array of ints and you have a vector of pointers to int, there is no way to feed that to func and get the expected output. The same would be true if you had an array of pointers to int. There would be no way to pass that to func, either. If func expects an array of int, you must give it an array of int.

#include <stdio.h>
#include <vector>

void func(int *data, int size);

int main()
{
	std::vector <int> things;
	int a[10];

	for(int i = 0; i < 10; i++)
	{
        a[i] = i+100 ;
        things.push_back(a[i]) ;
	}

	func(things.data(), things.size());
	printf("\n");

	func(a, 10);
}

void func(int *data, int size)
{
        //in real world this function does something to modify the data
	for(int i = 0; i < size; i++)
	{
		printf("%d\n", data[i]);
	}
}

Edit & run on cpp.sh

Hmm, testing it out myself I had similar results.
Changed the printing line to

printf("%d\n", *data+i);

and it works just fine

changing things randomly until they seem to work is not a recipe for success. The only reason this output is what you expect is because the singular int at *data is 100. All you're doing is adding i to that number each iteration of the loop.

*data+i is the same as (*data)+i

Last edited on Mar 20, 2013 at 11:33pm

Mar 21, 2013 at 12:16am

Warnis (230)

You're absolutely right, my bad.

Mar 21, 2013 at 6:27pm

pazza (8)

As mentioned, a different approach required then. Thanks for explanation about this.

Topic archived. No new replies allowed.

C++

Forum

Pass Vector of pointers to function expecting an array