memory leak / destructor

Du skal kalde delete for hver gang du kalde new.

Ellers så leaker du memeory.

Der er ikke garbage collection som i Java og C#.

jeg benytter stort set ikke Obj* o = new Obj(),  men Obj o; men jeg går ud fra at det ikke spiller nogen rolle, da Obj o også skaber et objekt som fylder.

For lige at være sikker på at jeg har forstået det korrekt: Når jeg er "færdig" med objektet kalder jeg delete. delete kalder objektets destruktor.

mht spørgsmål 1) Kalder list.remove(e), så min destruktor?

Nej.

Obj o;

allokerer plads på stack ikke på heap of forsvinder automatisk
når objektet ryger ud af scope.

Og destructoen bliver kaldt.

delete og new er "koblet"

Jeg vender tilbage i aften med nogle lidt mere uddybende spørgsmål.....

Har du lavet nogle fornuftige copy constructor?

mener du = overload? (og svaret er nej)

jeg mener noget som:

class A {
public:
  int *a;
};

int main() {
  A a,b;
  a.a = new int;
  b = a; // nu er der en pointer til den nye int 2 steder og når de a og b frigives kan der opstå fejl hvis du ikke har en copy constructor.

  return EXIT_SUCCESS;
}

Prøv at lave en privat copy constructor så finder du hurtigt ud af om du skulle bruge sådan en.

jeg har prøvet at lave en private copy constructor erklæring i min .h fil hvilket kompiler uden fejl. Hvis jeg tilføjer en erklæring i .cpp filen får jeg følgende:

In file included from dsr/ACKMonitor.cc:154:
/usr/include/g++-3/stl_pair.h: In method `pair<_T1, _T2>::pair (const
_T1 &, const _T2 &) [with _T1 = const int, _T2 = ACKData]':
/usr/include/g++-3/stl_map.h:164:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_pair.h:44: within this context
/usr/include/g++-3/stl_construct.h: In method `pair<const int,
ACKData>::pair (const pair<const int, ACKData> &)':
/usr/include/g++-3/stl_construct.h:48:  instantiated from `construct (_T1 *, const _T2 &) [with _T1 = pair<const int, ACKData>, _T2 = pair<const int, ACKData>]'
/usr/include/g++-3/stl_tree.h:556:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::_M_insert (_Rb_tree_node_base *, _Rb_tree_node_base *, const _Value &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_tree.h:908:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::insert_unique
(_Rb_tree_iterator<_Value, _Value &, _Value *>, const _Val &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_map.h:164:  instantiated from `map<_Key, _Tp, _Compare, _Alloc>::insert (typename _Rb_tree<_Key, pair<const _Key, _Tp>, _Select1st<pair<const _Key, _Tp> >, _Compare, _Alloc>::iterator, const pair<const _Key, _Tp> &) [with _Key = int, _Tp = ACKData, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
dsr/ACKMonitor.cc:154:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_construct.h:48: within this contextIn file included from dsr/ACKMonitor.cc:154:
/usr/include/g++-3/stl_pair.h: In method `pair<_T1, _T2>::pair (const
_T1 &, const _T2 &) [with _T1 = const int, _T2 = ACKData]':
/usr/include/g++-3/stl_map.h:164:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_pair.h:44: within this context
/usr/include/g++-3/stl_construct.h: In method `pair<const int,
ACKData>::pair (const pair<const int, ACKData> &)':
/usr/include/g++-3/stl_construct.h:48:  instantiated from `construct (_T1 *, const _T2 &) [with _T1 = pair<const int, ACKData>, _T2 = pair<const int, ACKData>]'
/usr/include/g++-3/stl_tree.h:556:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::_M_insert (_Rb_tree_node_base *, _Rb_tree_node_base *, const _Value &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_tree.h:908:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::insert_unique
(_Rb_tree_iterator<_Value, _Value &, _Value *>, const _Val &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_map.h:164:  instantiated from `map<_Key, _Tp, _Compare, _Alloc>::insert (typename _Rb_tree<_Key, pair<const _Key, _Tp>, _Select1st<pair<const _Key, _Tp> >, _Compare, _Alloc>::iterator, const pair<const _Key, _Tp> &) [with _Key = int, _Tp = ACKData, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
dsr/ACKMonitor.cc:154:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_construct.h:48: within this contextIn file included from dsr/ACKMonitor.cc:154:
/usr/include/g++-3/stl_pair.h: In method `pair<_T1, _T2>::pair (const
_T1 &, const _T2 &) [with _T1 = const int, _T2 = ACKData]':
/usr/include/g++-3/stl_map.h:164:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_pair.h:44: within this context
/usr/include/g++-3/stl_construct.h: In method `pair<const int,
ACKData>::pair (const pair<const int, ACKData> &)':
/usr/include/g++-3/stl_construct.h:48:  instantiated from `construct (_T1 *, const _T2 &) [with _T1 = pair<const int, ACKData>, _T2 = pair<const int, ACKData>]'
/usr/include/g++-3/stl_tree.h:556:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::_M_insert (_Rb_tree_node_base *, _Rb_tree_node_base *, const _Value &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_tree.h:908:  instantiated from `_Rb_tree<_Key, _Value, _KeyOfValue, _Compare, _Alloc>::insert_unique
(_Rb_tree_iterator<_Value, _Value &, _Value *>, const _Val &) [with _Key = int, _Value = pair<const int, ACKData>, _KeyOfValue = _Select1st<pair<const int, ACKData> >, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
/usr/include/g++-3/stl_map.h:164:  instantiated from `map<_Key, _Tp, _Compare, _Alloc>::insert (typename _Rb_tree<_Key, pair<const _Key, _Tp>, _Select1st<pair<const _Key, _Tp> >, _Compare, _Alloc>::iterator, const pair<const _Key, _Tp> &) [with _Key = int, _Tp = ACKData, _Compare = less<nsaddr_t>, _Alloc = allocator<ACKData>]'
dsr/ACKMonitor.cc:154:  instantiated from here
dsr/ACKMonitor.cc:25: `ACKData::ACKData (const ACKData &)' is private
/usr/include/g++-3/stl_construct.h:48: within this context

Er det det du mener med jeg kan se jeg mangler den?

Det kan jeg ikke overskue, kunne du vise noget af den kode der ligger omkring de ovenstående linier?

Hej segmose

Jeg har "løst" problemet, dvs jeg har omskrevet noget af koden og copy pastet noget nogen andre havde skrevet. Jeg fandt dog aldrig ud af hvor problemet var:) Hvis du læser http://www.eksperten.dk/spm/423285 kan du se hvorfor det nok ikke var en memory leak.

Den fejl der kom (ovenstående) opstod når jeg prøvede at lave en copy constructor. Fejlen opstod da jeg oprettede en iterator i metoden og prøvede at kopiere elementer fra en STL list til en anden.

Status: problemet er løst. Du og Arne må gerne lægge nogen svar da jeres hjælp har fået mig til at indse at jeg havde nogle gevaldige forståelses mæssige huller (det har jeg stadig, bare lidt mindre :) ). Og tak for hjælpen iøvrigt!

Så gerne.

svar