Estás usando una característica de C++ que no se considera estándar, el "Arreglo de Tamaño Variable" (ATV).
Los ATV son formaciones1 cuyo tamaño se ha definido mediante una variable en lugar de una constante, como es tu caso:
int lim, i;
float array[lim];
lim
es una variable, y usarla para definir array
hace que éste sea un ATV. Los ATV no forman parte del estándar de C++ pero algunos compiladores les dan soporte mediante una extensión de compilador, consulta estas preguntas para saber más del tema.
Dado que no es C++ estándar, no te aconsejaré una solución como la de Germán Martínez si no que te aconsejaré que rehagas el código para no usar ATV:
Primera propuesta: Memoria dinámica.
No puedes crear formaciones1 cuyo tamaño se defina mediante una variable a no ser que lo hagas en memoria dinámica (usando el operador new
):
int main() {
int lim;
cout << "Ingrese el limite de elementos del arreglo: "; cin >> lim;
float *array = new float[lim];
for (int i = 0; i < lim; ++i) {
cout << "Ingrese un numero para el espacio [" << i + 1 << "]: ";
cin >> array[i];
}
delete[] array;
return 0;
}
Al usar memoria dinámica (pedir memoria con new
) debes liberarla manualmente con delete
(en este caso, al ser memoria pedida para una formación, debes usar delete[]
); ésto es engorroso, las tendencias de C++ moderno aconsejan...
Segunda propuesta: Memoria dinámica con puntero inteligente.
... usar un puntero inteligente que libera al programador de las preocupaciones de la gestión manual de memoria:
int main() {
int lim;
cout << "Ingrese el limite de elementos del arreglo: "; cin >> lim;
auto array = make_unique<float[]>(lim);
for (int i = 0; i < lim; ++i) {
cout << "Ingrese un numero para el espacio [" << i + 1 << "]: ";
cin >> array[i];
}
return 0;
}
Al usar el puntero inteligente std::unique_ptr
, la gestión de memoria se hace automáticamente (no tienes que llamar a delete[]
manualmente). Pero sigue siendo más de lo que tu código realmente necesita:
Tercera propuesta: Usa un contenedor de datos.
Tu código podría ser mucho más sencillo usando un contenedor de datos como la plantilla std::vector
:
int main() {
int lim;
cout << "Ingrese el limite de elementos del arreglo: "; cin >> lim;
vector<float> array(lim);
for (int i = 0; i < lim; ++i) {
cout << "Ingrese un numero para el espacio [" << i + 1 << "]: ";
cin >> array[i];
}
return 0;
}
Uno de los constructores de std::vector
acepta un número como tamaño inicial del contenedor, lo cuál se ajusta perfectamente a tus necesidades.
Otras cosas a tener en cuenta.
- Se aconseja que el ámbito de las variables sea lo más pequeño posible; no es necesario que tu variable
i
para indexar el bucle for
sea declarada de antemano, puede declararse y definirse en el bucle sin problemas.
- Favorece el pre-incremento frente al post-incremento (lee este artículo para saber por qué).
- También conocidas como arreglos o en inglés array.