Hi, im pretty new at c++ and am taking a class right now, but ive run into a problem and i was hoping to see if someone could help me out.

Problem: "write a function that removes all odd integers from an array using pointers."

This is what i have so far:

the array before: 3,4,5,6,7,8,9,10,11,12
what it has to look like after: 4,6,8,10,12
what below actually does: 4,6,6,8,8,10,10,12,12

my function:
Thanks for any help!

Hey ionyka, welcome to the forums.

C/C++ isn't flexible with arrays at all. Once you've defined the size of an array, you cannot change it. So you cannot "delete" elements from an array. But you can mark elements as "removed", in your case as settings them as something wierd, maybe 0?

Secondly, I'm looking through your code. I've always found it handy for working with arrays to do out a horrible memory map, a simply table with a box for each element of the array. Then step through a a cycles of the loop, and see what happens.

If you do this, you'll see what's up. This chunk:
is in charge of what happens when an odd number is found.

Let i=0; and 3 is odd, the if statement will be true & so you do this: array2[0] = array2[1]. So, you've copied the value 4 over 3, so you're array will look like this:


Let i=1, the number 4 (the second element of the array) is even, so it's okay.

Let i=2, we're on the number 5, it's odd, so the array will look like this now:

You see now where it's going wrong?

You're not moving the values of the array to the left, you're simply copying even numbers over the odd, making more correct values.

What you want to do, it to shift all the values of the array to the left in one go, to prevent any values being copied. It's not veryhard, so I'll let you figure out that, but if you've problems, please ask!

Hope this helps
Garth Farley