Thread: Safe to declare an array of structures that contain arrays?

Is it safe to declare an array of structures, if the strucure itself contains dynamically allocated arrays? Below is some sample code.

The program I'm working on seems to work, but I'm worried that the array of structures will get me in trouble later.

Code:

#include <stdio.h>


struct MyStruct{

	double V1;     
	double V2;      
	double * A1;  
	double * A2; 		             

};


int main() {


	int n = 10; 

	struct MyStruct *x;
	x = (struct MyStruct*)malloc((n)*sizeof(struct MyStruct));


	for( i = 0; i < n; i++ ){			
		x[i].A1 = (double*)malloc(60*sizeof(double));
		x[i].A2 = (double*)malloc(60*sizeof(double));
	}


	//
	//
	// insert useful code here
	//
	//
	    
	    
	for( i = 0; i < n; i++ ){
		free(x[i].A1);
		free(x[i].A2);
	}	  	
	
	free(x);     	      


return 0;       
}

As long as you're fastidious in your memory cleanup you should be fine. However, you should not need to cast the return value of malloc if you're compiling this as C code, and doing so can hide problems.

Originally Posted by rags_to_riches

As long as you're fastidious in your memory cleanup you should be fine. However, you should not need to cast the return value of malloc if you're compiling this as C code, and doing so can hide problems.

Like including the proper header file.

Originally Posted by rags_to_riches

As long as you're fastidious in your memory cleanup you should be fine. However, you should not need to cast the return value of malloc if you're compiling this as C code, and doing so can hide problems.

I think I'm actually compiling this as C++. I'm using mkoctfile to compile, and the file I'm using ends in .cc.

When I tried to compile without the casting, I got some message about conversion from MyStruct* to void*.

Originally Posted by quzah

Technically, you don't have a structure which has an array. You have a structure which has a pointer. There's a big difference. If it were an actual array, you wouldn't have to free the array (in fact, you could not free it implicitly, rather, you would free the actual structure--if you had allocated it dynamically).

Quzah.

That makes sense. It would explain why my program hasn't crashed.