Tienes una serie de problemas de concepto y diseño en tu código que dan lugar al error que describes. Empecemos por lo principal:
La memoria.
En c++ puedes pedir memoria para una formación1 de manera automática o de manera dinámica.
La manera automática es especificando un tamaño concreto en el momento de definir la variable:
int formacion[10]; // Una formación de '10' enteros.
El espacio que puedes pedir para una formación1 en memoria automática debe ser conocido en tiempo de compilación y está limitada al tamaño máximo de la pila, en cambio pidiendo memoria dinámica:
int *formacion = new int[tamanyo]; // Una formación de 'tamanyo' enteros
El espacio no necesita ser conocido en tiempo de compilación y no tiene la limitación de la pila.
En cualquiera de los dos casos, antes de rellenar una formación1 hay que reservar espacio para sus elementos.
Los parámetros.
Puedes pasar formaciones1 a funciones como referencia o como puntero, al pasarla como referencia debes especificar también el tamaño:
// Una función que recibe una formación de 10 enteros por referencia
void funcion(int (&formacion)[10])
{
...
}
Si pasas la formación1 como puntero, pierdes la información del tamaño:
// ¿Cuántos elementos tiene 'formacion'? No se sabe
void funcion(int *formacion)
{
...
}
Pasar formaciones1 como puntero es compatible tanto con formaciones1 en memoria automática como en memoria dinámica, sin embargo, pasarlas por referencia sólo es compatible con formaciones1 en memoria automática; pero en ambos casos debería haber memoria reservada para los elementos de la formación1.
El desastre.
En tu código estás pasando una formación1 como puntero y una variable para definir su tamaño, pero en ninguna parte de la función ingresarNumeros
pides memoria para la formación1 que recibes en el primer parámetro, en consecuencia cuando asignas valores a los elementos de la formación:
arr[i]=num2;
Estás asignando datos a memoria que no te pertenece y el programa reacciona dando un fallo en tiempo de ejecución (se cierra), si quieres que el tamaño de la formación1 sea arbitrario tendrás que usar memoria dinámica:
void ingresarNumeros(int *arr, int &tam)
{
using namespace std;
int num2;
bool band1;
do {
cout << "\t-> Ingrese el tamanio del vector: ";
cin >> tam;
if (tam <= 0) {
cout << "\tCantidad no permitida.\n\n";
band1 = false;
}
else {
band1 = true;
}
} while(band1 == false);
arr = new int[tam]; // Pedir tamaño para la formación
printf("\n");
for (int i = 0; i < tam; ++i) {
cout << "\t-> Ingrese un entero: ";
cin >> num2;
arr[i] = num2;
}
}
Sólo con añadir la petición de memoria en la función ya se arregla tu error, pero la función sigue siendo un desastre:
- Es mala práctica pedir memoria en un lugar diferente al que se va a usar, el hecho de pasar la formación1 por puntero (y su tamaño por referencia) nos indica que pretende usarse fuera de la función.
- Mezclas impresión por consola estilo C++ (
cout
) con estilo C (printf
) usa sólo uno de ellos (o sea, usa el estilo C++).
- El código es innecesariamente complicado y extenso. Para lo poco que hace, podría hacerlo con menos complicaciones.
- Olvida las formaciones1 usa algún contenedor de C++.
Propuesta.
Abordando los problemas listados en el apartado anterior, tu código podría parecerse a:
#include <iostream>
#include <list>
// Devuelve una lista de números, el propio objeto lista almacena su tamaño.
std::list<int> ingresarNumeros()
{
using namespace std;
std::list<int> numeros;
// Si el valor es sin signo, no puede contener valores negativos.
unsigned tam = 0;
// Repetimos mientras 'tam' sea cero (cero se interpreta como 'false').
while (!tam)
{
cout << "\t-> Ingrese el tamanio del vector: ";
cin >> tam;
if (!tam)
cout << "\tCantidad no permitida.\n\n";
}
for (int i = 0; i != tam; ++i)
{
int numero;
cout<<"\t-> Ingrese un entero: ";
cin >> numero;
numeros.push_back(numero);
}
return numeros;
}
int main()
{
std::cout << "Los numeros son:\n";
for (const auto &numero : ingresarNumeros())
std::cout << numero << ' ';
return 0;
}
Puedes ver el código funcionando en Try it online!.
1También conocida como arreglo o en inglés array.