Primero hay un error al manejar el caso base de la recursividad, pues cuando n sea 0, vas a llamar a la misma función usando como parámetro n=0
, lo que hace que nuevamente vuelvas a llamar a la función usando como parámetro n=0
, .... y así infinitamente.
El caso base debería terminar la recursividad cuando n == 0:
if (n == 0) return;
Adicionalmente, la condición para imprimir la respuesta sería solamente si n es impar, o sea, si n % 2 == 1
(n deja resto 1 cuando se divide por 2):
if (n % 2 == 1) cout << n << endl;
Además siempre se debe llamar a la recursividad, así que tiene sentido hacerlo fuera de todos los if
.
La función quedaría entonces así:
void impares(unsigned int n) {
if(n == 0) return;
if(n % 2 == 1) cout << n << endl;
impares(n-1);
}
Espero ayude.
PD: Para que los números sean impresos de menor a mayor, solo hay que cambiar el orden de las lineas 3 y 4:
void impares(unsigned int n) {
if(n == 0) return;
impares(n-1);
if(n % 2 == 1) cout << n << endl;
}
if (n==0) impares(n);
, te recomiendo revisar el código con calma y una hojita de papel para ir anotando los valores de las cosas en cada iteración.