Thread: error : variable-sized object 'largeArray2' may not be initialized

Hi, when i compile my code i get this error : "error : variable-sized object 'largeArray2' may not be initialized"
I just don't understand this error, any help would be appreciated, thanks in advance.

Code:

float give_coefficients_routh_table_and_fill_two_first_lines(int denominator_degree)
{
  float largeArray2[20][20] = {0};
  int l = 0;
  int c = 0;
  int e = denominator_degree ;
  for ( e = denominator_degree; e>=0; e--)
            {
            float array_of_coefficients[20];
            float d(e);
            cout << "what is the coefficient linked with degree" << e << "?" << endl;
            cin >> d;
            d = array_of_coefficients[e];
            if ( l==2 ){ l = 0; c++;}


                float largeArray2[l][c] = {d};
                l++;
            }
return largeArray2[20][20];
}

Originally Posted by funnydarkvador

Code:

float largeArray2[20][20] = {0};

you're obviously declaring and initializing your array here. on which line does that warning message acctually occur?

Originally Posted by funnydarkvador

Code:

cin >> d;
d = array_of_coefficients[e];

I don't know what you're trying to achieve here. you're accepting input into d, and then overwriting that input with the value already stored in array_of_coefficients[e], which is uninitialized.

Originally Posted by funnydarkvador

Code:

float largeArray2[l][c] = {d};

what you're doing here is declaring a new array called largeArray2, with dimensions l and c. this is a new array, and the compiler doesn't care because it's in a new scope, distinct from where the previous largeArray2 array was declared.

Originally Posted by funnydarkvador

Code:

return largeArray2[20][20];

this is wrong. this will return a single float value, when I suspect you actually want to return the whole array. unfortunately, the compiler knows that you only declared the array as a float[20][20], and you're trying to return the last element of a float[21][21].

The only error is at this line :

Code:

floatlargeArray2[l][c] = {d};

I just want to give a value d (which is different each time in the loop because it is typed in by the user) to each cell of the array largeArray2 but i think the error is due to the fact that variables l and c are incremented, that's why i get the error error : variable-sized object 'largeArray2' may not be initialized

Please tell me how to do

Originally Posted by laserlight

The thing is, that line looks like an attempt to declare and initialise a variable length array named largeArray2, which hides the other array of arrays named largeArray2.

What you probably want to do is to assign the value of d to each element of largeArray2[l].

Yes that's what i want to do, but i don't understand your idea to use a simple loop to assign the values, can you explain me a little bit please?

[QUOTE=laserlight;1151378]Consider this program:

Code:

    for (int i = 0; i < 10; ++i)
    {
        std::cout << numbers[i] << ' ';
    }
    std::cout << std::endl;
}

I just don't understand the use of this part of the code : why do i need to display the cells of an array on 1 line :

1	0 1 2 3 4 5 6 7 8 9 i don't say your idea is stupid, it may be right actually, but i don't understand how to use it

Thank you for your advice laserlight. I think i don't need to use std::cout because i did #include <cstdlib> at the beginning of my program, so that i use cout whether than std::cout
Anyway i've discovered some errors in my program, and i'll show you the final version that works