variable-sized object `val' may not be initialized

[arashf]

I have problem compiling the following code:

class sample
{
public:
    static const unsigned int d = 5;
    static const unsigned long int M = 10000000;
    double** val;
	sample();
	~sample();
};

#include "test.h"
#include <fstream>


sample::sample(){
	val = new double* [d];
	for (int i = 0; i<d; ++i){
	double* val[i] = new double [M];
	}
}

sample::~sample(){
	delete val;
}

int main(){
sample smp;
}

I get the error

$g++ test.cpp -o test.exe
test.cpp: In constructor `sample::sample()':
test.cpp:7: error: variable-sized object `val' may not be initialized

I can't understand what the problem is.

[vlad from moscow]

The reason of the error is statement

double* val[i] = new double [M];

in the constructor. I think you meant

val[i] = new double [M];

Also you incorrectly delete val in the destructor.

[Catfish3]

As a C++ programmer, you are encouraged to use initialization lists for constructors.

Try this (code wasn't tested):

sample::sample(): val(new double* [d]){
	for (int i = 0; i<d; ++i){
	val[i] = new double [M];
	}
}

[vlad from moscow]

@Catfish3
As a C++ programmer, you are encouraged to use initialization lists for constructors.

Try this (code wasn't tested):

sample::sample(): val(new double* [d]){
	for (int i = 0; i<d; ++i){
	double* val[i] = new double [M];
	}
} 

As I pointed out above this code is invalid.:)

[Catfish3]

Indeed, vlad. I saw that in the last moment.