Question

I recently started with C++ and i’m not entirely sure I grasp the concept of pointers and their connection to arrays. I have two classes, Term and Polynom. I have a main loop which allows the user to enter 2 numbers. Those numbers is then added to the “Term” object and that object is then added to the “Polynom” object. Everytime the loop is executed a new “Term” object is created.

 //These lines are executed until the user is done entering numbers             
 potens = new Term;
 potens->sattPotens(kinput, ninput);//Add values to "Term object"
 poly.addTerm(potens);//Add "Term" object to "Polynom" object

A “Polynom” object is only created once in the program. In the “Polynom” class I use a “Term” pointer to store all the “Term” objects that is added to the “Polynom” object. The “Term” pointer in the “Polynom” class is initiated once in the “Polynom” constructor.

 void Polynom::addTerm(Term *t){
      *(term+antal_termer) = *t;//This is were the program crashes
      antal_termer++;
}

I know I could use a vector instead of a pointer to store the “Term” objects but i’m trying to learn how pointers work. I am also unsure when I’m supposed to delete the objects created in the main loop. Since every time the loop is executed I create a new “Term” object but I never delete them.

EDIT: I used to allocate the “Term” object in the “Polynom” class this way: term = new Term[]; I then changed it to term = new Term[10]; but I still crashes when I execute term[antal_termer] = *t;

Answer 1

_{Pasting in outcome from comments.}

antal_termer was not initialised in the constructor, resulting in invalid memory access here:

*(term+antal_termer) = *t;

As the code is copying t, via assignment, you can delete potens; after the call to addTerm(). The code must prevent going beyond the end of the term array in addTerm(), otherwise another invalid memory access will occur:

void Polynom::addTerm(Term *t){
      if (antal_termer < 10)  // Use constant instead of literal 10
      {
          *(term+antal_termer) = *t;
          antal_termer++;
      }
}