Thread: problem with classes in a list

Hi I'm new to this forum so if anything I do is a problem I'm quite sorry.

I am programming a game at the moment to consolidate my c++ skills, improve and have something to extend when I decide to learn graphical programming. However I have come across a brick wall and although i could come up with a work-around I would like to solve my problem rather than run away from it.

On to the problem!
I'm trying to create a function that can check if any sprite classes in a list are located at the position passed into the function and then return the address of the class so that it can be edited.

If I have found a 'sprite' which has the position desired I will return true and return the address of the 'sprite' class by using the pointer.

My problem is assigning the address of the 'sprite' class to the pointer sent in the parameters.

Code:

bool sprite::checkSpritePos(std::list<sprite*> spriteList, int xpos, int ypos, sprite *spriteAddress)
{
    std::list<sprite *>::iterator Iter;

    for (Iter = spriteList.begin(); Iter != spriteList.end(); Iter++)
    {
        if ((int)((*Iter)->getX()) == xpos && (int)((*Iter)->getY()) == ypos)
        {
            // need to copy the address of the desired class to a pointer
            return true;
        }
    }
    return false;
}

I'm am not sure what code needs to go there. My ideas have just caused compile errors.

Any help would be very appreciated and for any time spent trying to help me.

Thank you, it was so easy to overlook the two asterixes.

I have constructed this function this way as it is an overloaded function depending on if I want to return the sprite or not.

Hi again, this function is causing me errors at runtime now.

Code:

 bool sprite::checkSpritePos(std::list<sprite*> spriteList, int xpos, int ypos, sprite **spriteAddress)
{
    std::list<sprite *>::iterator Iter;
    spriteAddress = 0;

    for (Iter = spriteList.begin(); Iter != spriteList.end(); Iter++)
    {
        if ((int)((*Iter)->getX()) == xpos && (int)((*Iter)->getY()) == ypos)
        {
            *spriteAddress = *Iter; // error occurs here
            return true;
        }
    }
    return false;
}

The line *spriteAddress = *Iter; is causing an unhandled exception.

Here is the message that shows up when I run the program in degugging mode.

Unhandled exception at 0x0041f4a0 in snake game.exe: 0xC0000005: Access violation writing location 0x00000000.

All these fancy numbers are making that message very cryptic for me right now. Thanks for any help.

I am not sure what to do. What can I do to initialize anything without using a NULL pointer?
Should I just create a new sprite class pointer for the spriteAddress

Thanks for that, figuring out that would have taken me quite a while to nut out myself.

If you are going to use grumpy's code you must remove line 4 from tha function

Code:

spriteAddress = 0;

Local spriteAddress is set to point to the_address but if you set it to 0 you'll get the same problem as before: dereferencing pointer that doesn't exist.

Also, you should declare your function as

Code:

bool sprite::checkSpritePos(const std::list<sprite*>& spriteList, int xpos, int ypos, sprite **spriteAddress)

to avoid creating local list upon every call.
In that case, local Iter must be const iterator:

Code:

std::list<sprite *>::const_iterator Iter;