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?