yang686526

problems in std::map<oddbobjectid,...>
problems in std::map<oddbobjectid,...>
hi,
i'm using dwgdirect 1.12.4.
i have a strange problem in using oddbobjectid as a std::map key.
in a drawing with external reference blocks the drawing contains some layers from the external reference: for example drawing1|layer1, etc
i have overriden the member function
void odgsview::setlayer(oddbstub* pstub), where i create oddbobjectid from the given stub and write the layer into a std::map as a key. in the std::map i map the layer id to some information.
sometimes (not always) my program crashes on regen because of a problem in the operator < in oddbobjectid class. to simulate it to happen always, i call in a loop both reloading of the external references and then regen.
the problem was fixed when i replaced the key in the map with the layer name. i.e. std::map<odstring,...>
does it mean that it is dangerous to use oddbobjectids as keys when there are several drawing databases?
thanks in advance for any help.
best regards
chudomir

chudomir, could you try the following operator < for object id?
(std::map lets define external predicate):
code:
bool oddbobjectid:perator < (const oddbobjectid& id) const
{
// compare by using original database
if (!m_id || !id.m_id || m_id->database() == id.m_id->database())
{
return m_id < id.m_id;
}
// compare by using redirected id
oddbobjectid me(*this);
me.converttoredirectedid();
oddbobjectid it(id);
it.converttoredirectedid();
return me.m_id < it.m_id;
}
sincerely yours,
george udov
thank you very much for your reply. i'll try this asap. i suppose i don't have to recompile the dwgdirect files. i'll use this as an external operator in some namespace.
best regards
chudomir

you can define predicate for std::map (this is a third argument for template), that compares keys (oddbobjectids) using same way.
sincerely yours,
george udov