brug af delete

1)

Nej - a allokeres på heap og deallokeres derfor pænt ved return.

2)

Det skal du ikke - kun delete ved new.

I funktion2 kan du overveje memcpy til hurtig kopiering.

arne> kan der kun være memory leak såfremt jeg har benyttet new ...so,ething og ikke benytte delete ...something?

Hvis vi ser bort fra obskuriteter såsom rekursive kald der ikke
returnerer som de skal, så kræver en memory leak dynamisk allokering
uden matchende deallokering.

I C++ er det meget typisk new uden delete. Man kan dog også i C++ bruge
C funktionerne malloc og free.

hmm, nu kommer der et langt ude spørgsmål:

Har du nogen ide til hvad der kan få et program til at gå ned efter alt mellem 3-15 minutter? Jeg sidder med en fejl jeg simpelt hen ikke kan finde

og hvor mange new uden tilhørende delete skal der til før det bliver et problem? Blot 1?

En memory leak er bare en memory leak d.v.s. at hvis du leaker 1000 bytes
per gang så kan du gøre det "antal fri memory i din PC"/1000 gange.

Du kan nemt detecte en memory leak. Du studerer bare hvordan memory
forbruget udvikler sig. Task Manager i windows. Hvis memory forbruget
vokser konstant indtil det nærmer sig total memory så er det en leak.

Fejl kan også skyldes andre ting. F.eks. noget der løber ud over
en array graænse og laver en overskrivning af noget memory.

Jeg lukker her, tak for din hjælp arne. Er du "til at få fat på" her i aften?

Ja.

arne_v >> du mener selvfølgelig a allokeres på stack'en, ikke heap'en...

conrad >> er der nogen speciel årsag til at du kopierer alt indholdet?
Det er jo netop kun lokalt for funktionerne, så jeg forstår ikke helt hvad du får ud af det..?

Ja selvfølgelig. Sorry.

Obj indeholder et array. Et nyt objekt af en anden type skal oprettes med en kopi af dette array. Det var lidt hurtigt skrevet eksempel- function2 er min constructor

er der en af jer der vil svare på om følgende er en legal måde at delete på:

map<int,SomeObject*> it;

if(it == mymap.end())
{
  delete mymap[id];
  mymap.erase(id)
}

Jeg kan evt åbne et nyt spørgsmål

Jamen det vil jo ikke virke da array'et i function2 er lokalt!
Når funktionen/konstruktøren er udført, er b ikke længere i memory...

jpk>> Jeg kan godt se det ikke er helt smart den eksempel kode jeg har fåt skrevet. Den rigtige kode, som svarer til function2 ser således ud:

ACKData::ACKData(int aid,nsaddr_t r[],int rl, Time sat)
{
    this->ackid = aid;//id of this ACK
    this->route_l = rl;//lenght of array
  //do some copying here
    for(int i = 0; i < rl; i++)
    {
      this->route.push_front(r[i]);
    }
    this->sendat=sat;//time we send this ACK
}

den er der vel ikke noget galt med?

Hvilken type er route?

STL list

Hvoefor gemmer du så størrelsen på array'et i route_l?
list har jo en size() metode...

Jeg tror jeg startede med at have et array, hvor jeg blev nødt til at gemme længden. Senere har jeg skiftet til list og så er det da rigtigt at det er unødvendigt.