I am alittle bit confused on topic of allocating objects on heap vs allocating on stack, and when and how delete() should be called.
For example I have class Vector.
delete [] v;
- it's an array notation of delete operator. You have to use it when deleting arrays, instead of deleting each element sequentially.Vector* v = Vector[100];
just won't compile. Write Vector v[100];
and it will allocate array of vectors on stack. You may not delete it manually.x, y, z
are on the heap as the whole object is on the heapitems[10]
is also allocated on the heap as is makes part of the object. To delete it just call delete v2;
. You don't need a special destructor unless you allocate anything in constructor.int* items;
is stored on the stack as it's part of the object allocated on stack. You don't have to delete it, it will be deleted automatically when comes out of scope.I'll start from the beginning...
Vector** v = new Vector*[100];
Allocates an array of 100 pointers to objects of type Vector on the heap It returns one pointer- v - the you can use to keep track of this array of pointers.
Delete this array of 100 points with:
delete[] v;
(Use the delete
operator- delete
for a single allocated object, delete[]
for an array)
Next case (I'm assuming you mean new Vector[100]
:
Vector* v = new Vector[100];
You allocated an array of 100 Vectors on the heap and got a pointer to its start location- v. Delete this array with:
delete[] v;
Next...
class Vector
{
int x, y, z;
}
Vector* v = new Vector();
This allocates an object of class Vector on the heap and gives you a pointer to keep track of it. Because you allocated the entire object on the heap, x, y, and z are all allocated on the heap.
Delete it with:
delete v;
class Vector2
{
int items[10];
}
Vector2* v2 = new Vector2();
This one is a bit trickier but I'm going to reason it out...
Classes are blueprints. You haven't allocated any memory at all until you instantiate the class somehow, in this case on the heap. Because the class is a blueprint, items
could not have been allocated until you created an object of class Vector2
on the heap. I think we can reasonably infer that items
is thus allocated on the heap.
Delete v2 with:
delete v2;
And finally:
class Vector3
{
int* items;
}
Vector3 v3 = Vector3();
You allocated all of class Vector3 on the stack, the pointer inside of it items
is also allocated thus. Nothing went on the heap, so don't delete it.
General rule:
when you use new
or new []
, you allocate on heap, in other case you allocate on stack.
every time you use new
you should use delete
(explicitly or not)
every time you use new[]
you should use delete[]
(explicitly or not)
The next code is UB since you use one new[] and 101 delete. Use one delete[]
.
Vector** v = new Vector*[100];
for (int i = 0; i < 100; ++i)
{
delete(v[i]);
}
delete(v);
Lets star off that you probably do not need to dynamically allocate anything.
Either a static array or vector will work much better.
But lets assuming you are doing this as a learning exercise.
Vector** v = new Vector*[100]; //create an array of 100 pointers to 100 Vector objects
//
// The above comment is misleading.
// The 100 pointers have not been initialized.
This allocated an area of memory that has space for 100 pointers (to Vector (Note they are uninitialized ie each pointer in random)). To delete this you need to do:
delete [] v;
If you allocate each of the members (which you should do if you want to use them:
for (int i = 0; i < 100; ++i)
{
v[i] = new Vector;
}
// Code
for (int i = 0; i < 100; ++i)
{
delete v[i];
}
So note that for each call to new there should be a corresponding call to delete.
Vector* v = Vector[100];
This is just wrong. If will not compile.
Unless the member is a pointer it is inside the object.
If the member is a pointer it must be separately allocated.
class Vector
{
int x, y, z;
}
Vector* v1 = new Vector();
Vector v2 = Vector(); // Yes the = Vector() is required
Here v1 points at a dynamically allocated object that contain x/y/z
Here v2 is an object that contain x/y/z
I know people are going to say the = Vector();
is a not needed or a copy construction. Both sort of true but both missing the point. 1) Its a copy construction but the compiler is always smart enough to remove it. 2) It is needed to make it equivalent to the line above. The difference is without it is default-initialized
(ie not initialized) with it the members are zero-initialized
(because Vector() has only a compiler generated constructor).
So what about array members.
They are no different from other members. Members are ALWAYS allocated inside the object. If the member is a pointer then it is inside the object but what it points at has to be explicitly set.
class Bob
{
int dataArray[10];
int* dataPtr;
};
Bob b1 = Bob();
Bob* b2 = new Bob();
b1.dataArray[0] = 1; // dataArray is inside the object.
// b1 is allocated locally
b1.dataPtr = new int [10]; // dataPtr is inside the object.
// But what it points at must be seprotally defined.
// Note you must call delete [] for each new []
b1.dataPtr[5] = 2;
b2->dataArray[0] = 1; // dataArray is inside the object.
// b2 is allocated dynamically
b2->dataPtr = new int [10];// dataPtr is inside the object.
// But what it points at must be aseptically defined.
// Note you must call delete [] for each new []
b2->dataPtr[5] = 2;