[C++] errore allocazione memoria

[effe_89]

progetto442.cpp|33|error: 'y' cannot appear in a constant-expression|

#include <iostream>
#include <string>
using namespace std;
struct matrix
{
    char nome;
    int r;
    int c;
    int** data;
};
//N MATRICI,UNA LETTERA CHE INDICA UNA MATRICE,INTERO R,INTERO C
void specificaN(int& n)
{
    cin>>n;
}
int main()
{
    int n;
    specificaN(n);
    matrix m[n];
    int index=0;
    char l;
    int x,y;
    while(n!=0)
    {
        cout<<"inserisci nome A,B,C,..:"<<endl;
        cin>>l;
        m[index].nome=l;
        cin>>x;
        m[index].r=x;
        cin>>y;
        m[index].c=y;
        m[index].data=new int[x][y];
        n--;
        index++;
    }

}

devo per forza inserirle da tastiera! come faccio??

[oregon]

Queste righe

int n;
specificaN(n);
matrix m[n];

che senso hanno?

Se n non ha alcun valore, come pensi che possa essere usata nei vettori?

E comunque, che compilatore usi? Se non usi un compilatore C99 dovrai usare l'allocazione dinamica, come ti ho già detto in un'altra discussione.

[Dark867]

Cieco...non vedi che lo usa nella funzione specifica n come parametro d'uscita?

Il tuo problema è che non puoi assegnare a un vettore/matrice una dimensione decisa in fase di esecuzione, cioè

int n;
cin >> n;
int m[n]; //errore!

questo è sbagliato! Con l'allocazione statica, cioè quella che hai appena fatto, n dovrebbe essere noto in fase di compilazione, cosa che non è!

Se vuoi farlo decidere all'utente tramite il cin devi obbligatoriamente usare l'allocazione dinamica, cioè fare una cosa tipo:

int n;
cin >> n;
int* m = new int[n];
//..il tuo codice..

delete []m; //non dimenticare mai di deallocare!!

[Scara95]

Originariamente inviato da effe_89
progetto442.cpp|33|error: 'y' cannot appear in a constant-expression|
...
m[index].data=new int[x][y];
...[/CODE]

Vi faccio notare l'errore...
...e due possibili soluzioni:

#include <iostream>
#include <cstdio>
using namespace std;

int main() {
	int *array, x, y;
	cin>>x;
	cin>>y;
	array = new int [x*y];
	for(int i = 0; i < x; i++)
		for(int j = 0; j < y; j++)
			array[i*x+j] = (i*10) + j;
	for(int i = 0; i < x; i++) {
		for(int j = 0; j < y; j++)
			printf("%0.2d ", array[i*x+j]);
		cout<<endl;
	}		
	delete[] array;
	int **array2 = new int*[x];
	for(int i = 0; i < x; i++)
		array2[i] = new int[y];
	for(int i = 0; i < x; i++)
		for(int j = 0; j < y; j++)
			array2[i][j] = (i*10) + j;
	for(int i = 0; i < x; i++) {
		for(int j = 0; j < y; j++)
			printf("%0.2d ", array2[i][j]);
		cout<<endl;
	}
	for(int i = 0; i < x; i++)
		delete[] array2[i];
	delete[] array2;
	return 0;
}

[Scara95]

A parte che, permettendo il compilatore quell'istruzione (matrix m[n] su cui tanto discutete, va benissimo anche:

int x, y;
cin>>x;
cin>>y;
int array[x][y];

[shodan]

Originariamente inviato da Scara95
A parte che, permettendo il compilatore quell'istruzione (matrix m[n] su cui tanto discutete, va benissimo anche:

No. Non va bene per niente. Il C++ vieta l'uso degli variable length array che il C99 invece permette. In C++ standard la variabile dimensionale di un array dev'essere nota a compile time. Viceversa è obbligatorio ricorrere all'allocazione dinamica. Se un compilatore permette i variable length array in C++ la cosa dev'essere considerata un bug.

[oregon]

Originariamente inviato da Dark867
Cieco...non vedi che lo usa nella funzione specifica n come parametro d'uscita?

Ops ... veramente addormentato ...

Ma in effetti in C++ niente VLA ... alloacazione dinamica forever ...

[Scara95]

Originariamente inviato da shodan
No. Non va bene per niente. Il C++ vieta l'uso degli variable length array che il C99 invece permette. In C++ standard la variabile dimensionale di un array dev'essere nota a compile time. Viceversa è obbligatorio ricorrere all'allocazione dinamica. Se un compilatore permette i variable length array in C++ la cosa dev'essere considerata un bug.

Non ho detto che sia giusto farlo, ho solo detto che se il compilatore lo permette si può fare...

[shodan]

Ci sono molte cose che il compilatore permette, ma non per questo si debbono fare.
Ad esempio:

int qualcosa = 0;
delete &qualcosa;

E comunque solo il Gcc / G++ (per quel che ne so) supporta il C99 (e lo abilità per default in barba allo standard C++).

[Dark867]

Se vuoi fare codice immondizia fa pure, ma secondo me ti conviene seguire lo standard...e poi c'è da dire che così il codice sarà compilabile solo usando quel compilatore, ma darà errori con altri...meglio scrivere una volta sola e bene no?