Remove object from list based on object property

Question

my code is:

class Room {
public:
    int id;
    string name;
    int ownerFd;

    Room(int id, string name, int ownerFd)
    {
        this->id = id;
        this->name = name;
        this->ownerFd = ownerFd;
    }
};

void RemoveUserRooms(int ownerFd) {
    for(auto& room : rooms) {
        if (room.ownerFd == ownerFd) {
            //remove room from list
        }
    }
}

What I want to do is to remove object from list. I already tried with remove and erase but that seems not to work in this way. Is is possible to do with list?

Possible duplicate of Can you remove elements from a std::list while iterating through it? — 1201ProgramAlarm
– 1201ProgramAlarm, Commented Jan 4, 2018 at 5:03

Gaurav Sehgal · Accepted Answer · 2018-01-04 05:15:27Z

2

Use iterator and erase while properly updating the iterator.

    for(auto i=rooms.begin();i!=rooms.end();)
    {
        if((*i).ownerFd == ownerFd)
        i=rooms.erase(i);
        else
        i++;
    }

Or better , you can use remove_if

rooms.remove_if([ownerFd](Room i){return i.ownerFd == ownerFd;});

edited Jan 4, 2018 at 5:15

answered Jan 4, 2018 at 5:05

Gaurav Sehgal

7,4972 gold badges20 silver badges35 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

David Konn Over a year ago

using for I am getting ‘room’ was not declared in this scope, and using remove_if: ‘ownerFd’ is not captured

Collectives™ on Stack Overflow

Remove object from list based on object property

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related