Re: delete, hash_map

From:

cbarron3@ix.netcom.com (Carl Barron)

Newsgroups:

comp.lang.c++.moderated

Date:

Sat, 5 May 2007 07:18:22 CST

Message-ID:

<1hxlqnw.1vbsvtm1f8l7j4N%cbarron3@ix.netcom.com>

SimpleCode <DragonXLong@gmail.com> wrote:

class COsgCar;
class moving_vechicle;
hash_map<moving_vechicle*, COsgCar*> m_hash;

hash_map<moving_vechicle*, COsgCar*>::iterator iter;

iter = m_hash.find( pMovingVechicle );
if ( iter != m_hash.end() )
{
COsgCar *p = m_hash[pMovingVechicle];
delete p;
}
///////////////////////////////////////
It always run error.
I don't know why.
I just test for some days.

   I don't know the inards of your hash_map but I might guees, that it
holds a pair<moving_vechicle *,COsqCar *> and it is deleting it twice.
Perhaps something like:

   iter = m_hash.find(pMovinfVechicle);
   if(iter != m_hash.end())
   {
      moving_vechicle *pv = iter->first;
      COsqCar * pc = iter->second;
      m.hash.erase(iter);
      delete pv;
      delete pc;
   }

assuming that a pair<const key_type,mapped_type> & is what *iterator
returns.

if hash_map is old enough you might need (*iter)....
instead of iter->...

--
      [ See http://www.gotw.ca/resources/clcm.htm for info about ]
      [ comp.lang.c++.moderated. First time posters: Do this! ]