Duda con programa que funciona bien

[picman]

Buenas!!

Una duda, he hecho un programa que me diga el numero de cifras que tiene un numero introducido por teclado a través de una función. La cosa es que me ha salido perfecto pero el compilador me saca este error:

Untitled.c:26:1: warning: control may reach end of non-void function [-Wreturn-type]

Mirad, este es el programa que he hecho:

Código:

#include <stdio.h>

int num_cifras (int numero);

int main() {
	int num, cifras;
	
	printf("Introduce un número:\n");
	scanf("%d", &num);
	cifras=num_cifras(num);
	printf("El número tiene %d\n", cifras);
	return 0;
}

int num_cifras (int numero){
	int cont=1;
	while (numero%10!=0) {
		numero=(numero/10);
		cont++;
		if (numero<=10) {
			return cont;
		}
	}
}

¿Qué ocurre con el return?

Gracias!

[Malenko]

El return está dentro de un "if". Por lo tanto, si no se llegase a cumplir la condición nunca se devolvería el valor. Yo lo evito poniendo el returno como última linea de las funciones.

Yo reescribiria la funcion de dos formas, llevando un booleando sabiendo cuando has de salir del bucle:

Código C++:

int num_cifras (int numero){
    int cont=1;
        int salir = 0;
    while ((numero%10!=0)&&(salir==0){
        numero=(numero/10);
        cont++;
        if (numero<=10) {
            salir = 1;
        }
    }
        return cont;
}
 

O usando un break:

Código C++:

int num_cifras (int numero){
    int cont=1;
    while (numero%10!=0){
        numero=(numero/10);
        cont++;
        if (numero<=10) {
            break;
        }
    }
        return cont;
}
 

El break creo que es más óptimo, pero a nivel de diseño, quizás es más limpia la versión con el booleano.

[amchacon]

Una nota, no es un error sino un warning, una advertencia ;)

[starfix]

como dice Malenko, si no entra en el if, no vas a tener valor de retorno....... proba ejecutar la funcion asi:

Código C++:

...
int Cifras= num_cifras(100);
...
 

con eso nunca va a entrar en el while, por lo tanto nunca va a alcanzar el return