How to create new variable in function to return?

[zensunni]

Ok, this is my code for my function:

// Puke - new pointers only point to mTemp
Object &Coordinate::stripObject(std::string name) {
    for ( int i=0; i < mItems.size(); i++ ) {
    if ( mItems.at(i)->getName() == name ) {
        mTemp = *mItems.at(i);
        mItems.erase(mItems.begin() + i);
        return mTemp;
    }
    }
    Object item("Nothing");
    mTemp = item;
    return mTemp;
}

This, of course, is not what I want to do. I want to create a brand new variable and return it. Every time I do this, however, g++ gives an error (probably because the created variable falls out of scope)

BTW - Any variable with an 'm' as the fist character is a member variable.

Any help would be greatly appreciated :)

[gpraghuram]

Ok, this is my code for my function:

// Puke - new pointers only point to mTemp
Object &Coordinate::stripObject(std::string name) {
    for ( int i=0; i < mItems.size(); i++ ) {
    if ( mItems.at(i)->getName() == name ) {
        mTemp = *mItems.at(i);
        mItems.erase(mItems.begin() + i);
        return mTemp;
    }
    }
    Object item("Nothing");
    mTemp = item;
    return mTemp;
}

This, of course, is not what I want to do. I want to create a brand new variable and return it. Every time I do this, however, g++ gives an error (probably because the created variable falls out of scope)

BTW - Any variable with an 'm' as the fist character is a member variable.

Any help would be greatly appreciated :)

Instead of returning the local object pass a variable as an argument(by reference) and then initialize it

void Coordinate::stripObject(std::string name,Object &retObj) {
Object item("Nothing");
retObj = item;
}
 

Raghuram

[zensunni]

Thanks for input, but I found a way..

Object &Character::stripObject(std::string name) {
    for ( int i=0; i < mItems.size(); i++ ) {
    if ( mItems.at(i)->getName() == name ) {
        Object *item = mItems.at(i);
        mItems.erase(mItems.begin() + i);
        return *item;
    }
    }
    std::cout << "Couldn't find " << name << "\n\n";
    Object *item = new Object("Nothing");
 
    return *item;
}

thanks again, though.

[gpraghuram]

Thanks for input, but I found a way..

Object &Character::stripObject(std::string name) {
    for ( int i=0; i < mItems.size(); i++ ) {
    if ( mItems.at(i)->getName() == name ) {
        Object *item = mItems.at(i);
        mItems.erase(mItems.begin() + i);
        return *item;
    }
    }
    std::cout << "Couldn't find " << name << "\n\n";
    Object *item = new Object("Nothing");
 
    return *item;
}

thanks again, though.

Using pointers is another option but the idea i told you will avoid the overhead of deleting the pointer in the calling part.

Raghuram