vector of objects to use with a vector of pointers

Question

The proxy objects generated by gSoap indicate that I should use a vector of pointers:

class SOAP_CMAC ota__RoomStayTypeRoomRates
{
public:
    std::vector<class ota__RoomRateType * >RoomRate;
    //....
};

Rather than using:

vector.push_back(new Object());

and then having to delete the objects, I thought I might create a vector of objects and then use the address of those objects as they will be destroyed when the vector goes out of scope, thereby avoiding memory leaks:

OTARoomRates roomRates;

std::vector<ota__RoomRateType> rateObjectList;

rateObjectList.reserve(7);
for (int i = 0; i < 7; i++)
{
    rateObjectList[i].RoomTypeCode = &roomTypeCode;
    rateObjectList[i].RatePlanID = &ratePlanID;
    //...
    roomRates.RoomRate.push_back(&rateObjectList[i]);
}

I get a segfault. I suppose it's a bad idea. Can you explain why?

score 3 · Accepted Answer · answered Feb 08 '12 at 09:44

3

rateObjectList.reserve(7) doesn't actually allocate or construct any ota__RoomRateType objects; it simply requests that the vector expand its capacity enough to hold 7 objects.

Potentially, you wanted rateObjectList.resize(7). Or std::vector<ota__RoomRateType> rateObjectList(7); if you know the number at vector creation time.

answered Feb 08 '12 at 09:44

Chowlett

45,935
20
116
150

Thanks! That was the problem... (waiting 3 minutes to accept the answer.) – d-_-b Feb 08 '12 at 09:51

score 0 · Answer 2 · answered Feb 08 '12 at 09:47

0

Can you explain why?

Sure. If someone holds roomRates when rateObjectList destroyed then any attempt to use a pointer from roomRates can cause SEG_FAULT. That's a bad idea, anyway.

This is better in this case

vector.push_back(new Object());

Even better is to use smart-pointers, like boost::shared_ptr

answered Feb 08 '12 at 09:47

maverik

5,508
3
35
55

boost is nice and all, but I think it might not be possible to use with the gSoap generated classes. It's only for generating an SOAP request. So both vectors are in the same scope. – d-_-b Feb 08 '12 at 09:55

vector of objects to use with a vector of pointers

2 Answers2