C / C++ / MFC

led mike wrote:

Prefer other patterns such as MVC and Factory creational patterns.

SRect sRect = {iX, iY, iW, iH};
RectMap.insert(make_pair(iID, sRect));

map<int, SRect *> RectMap;

map<int, SRect> RectMap;

RectMap[0] = new SRect();

SRect s;
	RectMap[0] = &s;

std::map<int, SRect *> RectMap;

[Delegates]      [Virtual Desktop]      [Tray Me !]

-Malli...!

Austrian_Programmer wrote:

void MakeParamRect(int iID, int iX, int iY, int iW, int iH)
{
RectMap[iID]->iX = iX;
RectMap[iID]->iY = iY;
RectMap[iID]->iW = iW;
RectMap[iID]->iH = iH;
}

std::map<key_type, SRect*> RectMap; <- bad
RectMap[key] = new SRect(iX, iY, iW, iH); <- this is why it is bad
void cleanup_before_destruction() <- and this is why it is bad
{
    std::map<key_type, SRect*>::iterator i;
    for( i = RectMap.begin(); i != RectMap.end(); ++i )
        delete *i;
}

std::map<key_type, SRect> RectMap; <- good
SRect sr(iX, iY, iW, iH);
RectMap[key] = sr;