Memory håndtering

Jamen, du behøver vel ikke delete det? Det behøver du kun hvis du explicit har brugt new eller malloc. Eller har jeg misset noget?!?!?

Så vidt jeg ved slettes Block structen inklusiv array automatisk efter destructoren er blevet kørt. Det er kun hvis du har oprettet Block ved hjælp af new at det er nødvendigt at frigøre hukommelsen manuelt.

Jeg er rimelig sikker på at det er der fejlen ligger. Mit program løber i ca 10 sek. og suger 100MB RAM...

En variabel har normalt samme levetid som den funktion eller klasse den er blevet erklæret i. Det kan kun omgås ved brug af new, og da du ikke har brugt new har du ikke noget problem.

Kan du så ikke prøve at poste noget mere af din kode?

Du bliver nok nødt til at poste lidt mere kode...

Hvordan ser din cpy metode ud? Det lyder som om det kunne være der den er gal!

void cpy(int a[][8], int b[][8]){
    for(int i=0; i<8; i++){
        for(int j=0; j<8; j++){
            b[i][j]=a[i][j];
        }
    }
}
int sol[8][8];
//fylder sol op med data her vha dobbelt for
MyClass c = new MyClass(sol);

Memoryproblemet må ligge omkring new operatoren du bruger i linien
MyClass c = new MyClass(sol);
Det er også muligt at problemet kan ligge der hvor du fylder sol op med data vha dobbelt for.
Lidt mere kode ville være fint.

Ikke fordi det vil give stor mening, men sådan fylder jeg sol op:
            for (i=0; i<queens.size(); i++) {
                q = queens[i];
                pl = q->placering;
                sol[pl->getX()][pl->getY()]=1;
                delete q;
            }
Jeg ville gå ud fra at når jeg er færdig med min instans af MyClass (der er en MyClass *c) kunne jeg slette den med delete, og den burde kalde deconstructoren, hvor jeg kunne slette min struct

Er der ikke noget grundlæggende forkert i den måde du laver ovenstående på ? Hvis du VED at din int matrix er på 8 x 8 elementer (Faste indeks længder i din cpy() metode), burde du så ikke lave en dedikeret klasse eller struct typedef til det ?

Hvad mener du med at returnere en int[][] fra en metode ? Det kan man da sagtens - men det er jo en pointer. Hvis det er indholdet du vil ændre i, kan du bare bruge reference (&):

const int xDim = 8;
const int yDim = 8;
typedef int[yDim][xDim] SIntArray;

void danArray(SIntArray& a, int b[yDim][xDim])
{
  for (int i=0; i<yDim; i++)
    for (int j=0; j<xDim; j++)
      a[i][j] = b[i][j];
}

Din Block struct slettes automatisk når du deleter instansen af MyClass

Anadan >> Ooops, hvad er det for en delete q; ? Du skal KUN -gentager KUN- lave delete af variabler du har lavet vha. new operatoren - og kun i samme antal som du laver new !

int[] q;

for (i=0; i<queens.size(); i++) {
  q = queens[i];
  pl = q->placering;
  sol[pl->getX()][pl->getY()]=1;
  // delete q; NEJ, NEJ - er bare en pointer som forsvinder når
  // q går out of scope. HER sletter du faktisk data-bidden til
  // queens[i] !!!
}

For at svare på dit allereførste spørgsmål, så skal du IKKE LAVE DELETE nogen steder, sådan som du har defineret din klasse. (Du bruger jo ikke new til at lave Block, så den forsvinder af sig selv, når block går out of scope, dvs. når din klasse ikke bruges mere.

class MyClass{
  private: // Vi skal ikke have nogen til at pille direkte i dataene.
  Block block;

  public: // Ellers kan du ikke tilgå metoderne uden for klassen.
  MyClass(int data[8][8]) {
    cpy(data, block.data)
  };
  ~MyClass(){
    // Default destructor er nok.
  }
}

Når du så skal bruge den, skal du ENTEN lave en direkte instans:

MyClass varA;
:
:

ELLER dynamisk allokere OG slette:

// Skal til at bruge den
MyClass *pVarA = new MyClass();
:
:
:
// Færdig med at bruge den.
delete pVarA;

Du har garanteret ret soepro. Det kan godt være at programmet bare skal suge så meget memory :'-(