[machoolah]

Hello,

I am a totally novice programmer. I want to define a dynamic array within a class (index is a pointer to the dynamic array I want to define). The code I use follows. My program crashes when run.
The question I have is if it is at all possible to use dynamice arrays within a class, if yes, I am on the right track? and again if I am not on the right track, what is an appropriate method to do that?

I appreciate any help.

in the meanwhile, don't hesitate to ask if you need further details.

Thank you all,

#include <iostream>
using namespace std;
class NODE
{
public:
	int * index;

	NODE (int dim)
	{
		int * index = new int [dim];
	};
};

int main()
{
	NODE sampleNode(5);
	sampleNode.index[0]=1;
	cout << sampleNode.index[0];

	return 0;
}

This post has been edited by machoolah: 27 May 2009 - 04:54 PM

[Mowgef]

#include <iostream>
using namespace std;
class NODE
{
public:
	int *index;

	NODE (int dim)
	{
		index = new int[dim];
	};
};

int main()
{
	NODE sampleNode(5);
	sampleNode.index[0]=1;
	cout << sampleNode.index[0];

	return 0;
}

Try this instead.

[KYA]

MEMORY LEAK!!!!!!!!

You need a destructor to free the memory:

~NODE()
{
	 if (index != NULL) delete [] index;
}

[machoolah]

MowGef, thank you,

that solved my problem.

KYA, thank you too, the node won't be deleted until the program ends. still thank you for reminding me that I need a destructor for safe computing!

[crazyjugglerdrummer]

Yes, you can definitely use dynamic arrays in classes. You could use a vector (but that's slightly different). I think whenever you declare an array, you're actually declaring a pointer to it, so you could get away with

int index[];

NODE (int dim)
{
index = new int[dim];
} <--- no semicolon here

I'm not 100% sure about the code, see if it compiles

[scooterfort]

crazyjugglerdrummer, on 27 May, 2009 - 05:02 PM, said:

Yes, you can definitely use dynamic arrays in classes. You could use a vector (but that's slightly different). I think whenever you declare an array, you're actually declaring a pointer to it, so you could get away with

int index[];

NODE (int dim)
{
index = new int[dim];
} <--- no semicolon here

I'm not 100% sure about the code, see if it compiles

you forgot your code braces!!!