Hvordan returneres et Int Array oprette på heapen?

Så vidt jeg kan tælle { } så kalder du delete inden i while
løkken d.v.s. at personPL bliver deallokeret inden du er færdig
med at bruge det.

Jeg forstår så heller ikke helt logikken med found, men ...

Hej Arne,

Tak for din hurtige reaktion.
Ja der mangler en slut } men den har jeg med i den "rigtige metode".
Found er for ikke at løbe alle tre pladser igennem - er ikke nødvendigt.
Jeg er nybebynder med C++, så der nok en hel del som kunne være skrevet bedre, men her er hele den modtagne metoden:

bool kontrollerKrav(Stol& s)
{
    bool found = true;
       
    Person& personKrav = s.getPerson().getPersonKrav();
    if (&personKrav == NULL) return true;

    int *personPL = s.getBord().findPladser(s);
       
    int i = 0;
    while( i < 3 && found)
    {
    if(s.getBord().getStol(personPL[i])->getPerson() == personKrav )
        {           
          found = false;
    }
    else
    {
      i++;
    }
    }
//Sletter int array som er blevet skabt på Heapen, i metoden findPladser(Stol& s)
  delete[] personPL;
       
return found;
}//End kontroller krav


Problemet er lige så snart i bliver talt en op personPL[i] - så er der ingen værdi på næste plads i arrayet - som jeg kan se det får jeg kun en pointer til første plads i arrayet og kan ikke komme til plads nr 2 - personPL[i=1]
Og lige så snart i antager værdien 1 - går programmet ned selv om jeg har lagt værdier ind på 3 pladser i arrayet ???

Og du er sikker på at det er personPL[1] der er problemet - altså ikke
indeholder 5 ?

Og ikke getStol(5) der returnerer NULL således at ->gerPerson() dør ?

Hej,

Det er jeg sikker på da jeg kun indsætter 3 værdier i arrayet i den medtoden som opretter arrayet, som du kan se i findPladser metoden.
Du "angriber" ikke min kode, jeg havde forvnetet at jeg havde skrevet noget forkert og derfor fik jeg ikke arrayet med over. Ser det umiddelbart "brugbart" ud den kode som er lavet??

Det her virker fint:

#include <iostream>

using namespace std;

int *al()
{
  int *p = new int[100];
  for(int i=0;i<100;i++) p[i]=i;
  return p;
}

void test()
{
  int *p = al();
  for(int i=0;i<100;i++) if(p[i]!=i) cout << "problem" << endl;
  delete[] p;
}

int main()
{
  for(int i=0;i<100;i++) test();
  return 0;
}

Det er derfor jeg tror at problemet ikke ligger i findPladser
med i at getStol returnerer NULL.

Prøv lige det her:

bool kontrollerKrav(Stol& s) {
  bool found = true;
     
  // se test nedenfor.
  Person& personKrav = s.getPerson().getPersonKrav();

  // hmm, &personKrav giver adressen på variablen personKrav så det giver aldrig NULL?!?
  if (&personKrav == NULL) return true;

  int *personPL = s.getBord().findPladser(s);
     
  int i = 0;
    while( i < 3 && found)    {
      // har vi hvad vi tror
      printf("PL[%d]=%d\n", i, personPL[i]);
      if (s.getBord().getStol(personPL[i]) == NULL)
        printf("getStol == NULL\n");
      if(s.getBord().getStol(personPL[i])->getPerson() == personKrav ) {         
          found = false;
    } else {
      i  ;
    }
    }
//Sletter int array som er blevet skabt på Heapen, i metoden findPladser(Stol& s)
  delete[] personPL;
     
return found;
}//End kontroller krav


// Hvor mange warnings for du for :
Person& test() {
  return NULL;
}

Tak begge to - ja I havde ret, min fejl - der var en fejl som gav en null pointer reference i person, nu virker det :-)
Jeg havde stirret mig blind på det int array fordi jeg forventede, jeg havde skrevet en forkert syntaks.
Tak for hjælpen!!
Det er første gang jeg har stillet et spørgsmål i dette forum.
Hvordan lukker jeg det korrekt?

Tja - du skulle jo nok have bedt os ligge et svar og så accepteret vores
svar i.s.f. selv at ligge et svar og acceptere det.

Åhh Undskyld :-/ - hvordan retter jeg det?
(tænkte nok det ikke var rigtigt)

Du kan oprette et spørgsmål med titel "point til arne_v og segmose"
og et link til dette spørgsmål.

Så svarer vi og du giver os point der.

Hej

Ok det vil jeg gøre - jeg sidder faktiske og prøver om jeg kan læse mig frem til hvordan det her foregår (savner en Q&A, om åbne og især lukke spørgsmål og point regnskab hvordan det hænger sammen) men det er måske meget logisk bygget op når man først kommer ind i det.

Prøv og læs:

http://expfaq.1go.dk/