1
int piezas = 0, procesos=0;
cout<< "Digite cantidad de pizas a procesar\n";
cin >> piezas;
cout << "Digite cantidad de procesos a evaluar\n";
cin >> procesos;



double steel[piezas][procesos];
getTimes(steel);

El compilador me genera error, int piezas & int procesos must have a constant value justo en esta parte double steel[piezas][procesos];mas alla de la solución no entiendo esta parte de c++ yo esto mas familiarizado a JAVA.

2 respuestas 2

2

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

  1. En una declaración T D en que D 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-lista T", entonces el tipo del identificador de D 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:

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];

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:

  1. Buena localidad de datos.
  2. Gestión automática de memoria.
  3. Mismo tipo de acceso que con arreglos.
1
  • Muy agradecido con tu ayuda, me gustaría instalar la extensión al compilador (visual studio) como lo descargo?
    – R.Ramirez
    Commented el 7 may. 2017 a las 0:49
1

Para poder inicializar un array de 2D "estático" el tamaño tanto de las filas como el las columnas debe ser constante. En este caso, una posible solución puede ser la de implementar la creación de un array dinámico a partir de las entradas, siendo piezas los datos de las filas y procesos los datos de las columnas:

double** steel = new double*[piezas];
for (int i = 0; i < piezas; i++)
    steel[i] = new double[procesos];

getTimes(steel);

Sin olvidar luego que hay que liberar memoria al finalizar al tratarse de memoria dinámica (importante).

Si por el contrario, no te importa la eficiencia y, además, prefieres olvidarte de liberar memoria al final puedes hacer un pequeño cambio en tu código original de esta forma:

int piezas = 0, procesos=0;
cout<< "Digite cantidad de pizas a procesar\n";
cin >> piezas;
cout << "Digite cantidad de procesos a evaluar\n";
cin >> procesos;

const int TAM_FILA = piezas ;
const int TAM_COL = procesos ;
double steel[TAM_FILA][TAM_COL];

getTimes(steel);

Todo depende del uso que posteriormente se le vaya a dar al array en cuestión (si se va a tener que modificar su longitud u otra gestión similar) y de los requerimientos de dicho programa. Sin embargo, esta opción no cumple los estándares ISO C++ sino que se utilizan las extensiones provenidas del compilador.

9
  • La primera parte, con memoria dinámica, OK. La segunda ... no se yo, pero me parece que el mensajito must have a constant value quiere decir otra cosa ...
    – Trauma
    Commented el 23 abr. 2017 a las 16:14
  • El mensaje quiere decir exactamente lo que he demostrado. Puedes compilar y probar el funcionamiento libre de errores y warnings en ambos casos. Commented el 23 abr. 2017 a las 16:29
  • Curioso. A mi me sigue dando 2 warnings: warning: ISO C++ forbids variable length array 'steel'
    – Trauma
    Commented el 23 abr. 2017 a las 16:37
  • Pues no sé @Trauma, mi compilador incluso con -Wallno imprime nada. De todas formas he depurado y probado el programa llenando el array y funciona bien igualmente. Commented el 23 abr. 2017 a las 22:15
  • 1
    Después de añadirle pedantic, ahora sí que me aparecen. Corregido, muchas gracias. Commented el 23 abr. 2017 a las 22:36

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.