###Problema. Utilizas Arreglos de Longitud Variable (VLA en inglés ), los cuáles no pertenecen al C++ estándar si no que son una extensión del compilador (Lee sobre la extensión en el compilador GCC ). Cada compilador tiene sus propias extensiones de manera que es posible que este código te funcione en determinados compiladores.
Según el estándar de C++ (traducción y resaltado míos):
###8.3.4 Arreglos
- En una declaración
T D
en queD
tiene la forma
D1 [
expresión-constanteopcional]
secuencia-especificación-atributoopcional
y el tipo del identificador en la declaración
T D1
es "declarador-tipo-listaT
", entonces el tipo del identificador deD
es arreglo; [...]
Este galimatías viene a decir que para crear un arreglo de la manera en que lo estás haciendo es imposible (sea de las dimensiones que sea), ya que los tamaños en la definición del array deben ser constantes conocidas en tiempo de compilación, pero el valor de piezas
y procesos
sólo se conoce en tiempo de ejecución.
###Solución.
Usa memoria dinámica. La solución propuesta por Mario Rodríguez en la primera parte de su respuesta es correcta y funcional (si se corrigen los errores en los tipos de variables):
int piezas = 0, procesos=0;
std::cout<< "Digite cantidad de pizas a procesar\n";
std::cin >> piezas;
std::cout << "Digite cantidad de procesos a evaluar\n";
std::cin >> procesos;
double **steel = new double*[piezas];
// ^^ Puntero a puntero!
for (int i = 0; i < piezas; i++)
steel[i] = new double[procesos];
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
steel[x][y] = x + (y * piezas);
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
std::cout << steel[x][y] << '\n';
Puedes ver el código funcionando en Wandbox 三へ( へ՞ਊ ՞)へ ハッハッ.
Mas la segunda no sigue el estándar y no compilará en compiladores sin soporte a VLA. El guardar una variable conocida en tiempo de ejecución en una variable constante no la transforma en una expresión constante (valor conocido en tiempo de compilación). ###Propuesta: arreglo 1D tratado como 2D. La solución anterior suele dispersar los datos por memoria, posibilitando una baja localidad de datos, lo cuál afectaría al rendimiento, para solucionarlo puedes crear el arreglo en una dimensión pero tratarlo como dos dimensiones:
int piezas = 0, procesos=0;
std::cout<< "Digite cantidad de pizas a procesar\n";
std::cin >> piezas;
std::cout << "Digite cantidad de procesos a evaluar\n";
std::cin >> procesos;
double *steel = new double[piezas * procesos];
// ^^ Puntero!
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
steel[x + (y * piezas)] = x + (y * piezas);
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
std::cout << steel[x + (y * piezas)] << '\n';
Esta aproximación, además de garantizar que todos los datos estarán anexos en memoria, te ahorra el proceso de inicializar cada sub-dimensión del arreglo.
###Propuesta: olvida new
y delete
.
Todas las soluciones anteriores tienen en común que debes solicitar manualmente la memoria y en consecuencia debes liberarla manualmente (ninguno de los ejemplos incluye liberación de memoria). Este proceso de liberar memoria puede ser tedioso y propenso a errores, así que mejor usemos std::vector
, que nos facilita todo, incluida la gestión de memoria:
int piezas = 0, procesos=0;
std::cout<< "Digite cantidad de pizas a procesar\n";
std::cin >> piezas;
std::cout << "Digite cantidad de procesos a evaluar\n";
std::cin >> procesos;
// Construye un vector de 'piezas * procesos' elementos con valor '0.'.
std::vector<double> Steel(piezas * procesos, 0.);
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
steel[x + (y * piezas)] = x + (y * piezas);
for (int y = 0; y < piezas; ++y)
for (int x = 0; x < procesos; ++x)
std::cout << steel[x + (y * piezas)] << '\n';
Usando std::vector
obtenemos:
- Buena localidad de datos.
- Gestión automática de memoria.
- Mismo tipo de acceso que con arreglos.