Get an Error as soon as the program runs.

Hi,
I am getting an error as soon as I run my program. The error is main.exe has caused a system error and needs to be closed. I know there is something wrong with my Container class but I can't find out the problem.

My Container class

Code:

---

#include <iostream.h> 

//file SetOfPersons.h
class SetOfPersons {
public:
        SetOfPersons(int initial_size = 4) {
    //default constructor allocate appropriate heap storage
    //store elements on heap array declared like this:
    //new Person*[initial_size];

    cout << "SetOfPersons(int)\n";
                numberOfElements = initial_size;
                elements = new  Person*[initial_size];}



        ~SetOfPersons(void) {delete [] elements;}


    void add(Person & element) {
        if(capacity == size()) {  //grow the container
            cout << "growing\n";
            Person **temp = elements;
            *elements = new Person[capacity*2];
            for(int i=0; i<capacity; i++) *elements[i] = *temp[i];
            numberOfElements = numberOfElements *2;
            delete [] *temp;
            }
      *elements[capacity++] = element;
    }
        Person & someElement(){
                capacity++;
        capacity = capacity % numberOfElements;
        return *elements[capacity]; }

        Person & removeSomeElement() {
        // answer some (any) element from the set and remove it from the set
    // if the set is more than half empty release some memory
     
        capacity--;
        if(capacity < numberOfElements/2) {  //shrink the container
            cout << "shrinking\n";
            Person ** temp = elements;
            *elements = new Person[numberOfElements/2];
            for(int i=0; i<capacity; i++) elements[i] = temp[i];
            numberOfElements = numberOfElements/2;
            delete [] temp;
        }
      return *elements[capacity];
    }


        int size(){
                //answer the number of elements in the set.
                //not just the size of the array
                return numberOfElements;
        }
       
        void printOn(ostream & ostr){
        int i ;
        for( i = 0; i<size(); i++) {
                cout<<*elements[i]<<"\n";
    }
 
        }

private:
        Person ** elements;
        int numberOfElements; //number of elements in the set
        int capacity; //size of the available array memory
};

//you will need the following global operator to print Sets.
//Put this in the same file as class Set, but not inside the Set class.
ostream & operator<<(ostream & ostr, SetOfPersons & s) {
    s.printOn(ostr);
    return ostr;
}

---

Well you don't do a very good job of initialising or updating capacity at appropriate times.

thanks. Can you point me where should I make the changes.

Well like your constructor should have
capacity = 0;

you mean like this:

Code:

---

SetOfPersons(int initial_size = 4) {
    //default constructor allocate appropriate heap storage
    //store elements on heap array declared like this:
    //new Person*[initial_size];
    capacity = 0;
    cout << "SetOfPersons(int)\n";
                numberOfElements = initial_size;
                elements = new  Person*[initial_size];}

---

I still have the same problem.

Well carry on with the analysis into the next function - it's your code, you debug it.

Run the code inside a debugger and step through it line by line, and examine the variables affected by each statement.
When it does something you didn't expect, you've found a bug.

delete [] *temp;
delete [] temp;
Which one of these is correct?