problema al crear funcion

[fenixelian]

Iniciado por javidotnet

Hola fenixelian

Te cuento que tuve que conseguir un IDE, para probar el código, y puedo decirte que no te preocupes porque ya encontré cuál es el error.
Fijate la sentencia "for", la definición para esta sentencia en C dice que el ciclo se ejecuta siempre que se cumpla la condición que se encuentra encerrada por los punto y comas:

Código:

for (n = a - 1; n == 1; n--)

Por lo tanto, nuestro programita nunca va a entrar dentro del ciclo porque de entrada n vale a-1, es decir que sólo entra cuando a = 2

Entonces la solución es colocar:

Código C:

for (n = a - 1; n > 0; n--)
 

En cuanto al "if" lo podemos quitar, teniendo en cuenta esa característica de la sentencia "for" que recién te expliqué. Por consiguiente, el código quedaría así:

Código C:

public static int MD(int a)
        {
            int n;
            
            for (n = a - 1; n > 0; n--) // n-- es lo mismo que n = n - 1
            {
                if (a % n == 0)
                {
                    return n;
                } // if
            } // for
    
            return 1; // si se llega hasta aquí es porque no se encontró divisor
        } // MD
 

Con respecto a lo otro que mencionas:



Permíteme corregirte, es 36 en lugar de 32; y te digo que es sencilla la solución,
sólo agregas la función Math.Abs() y listo, así:

Código C:

for (n = Math.Abs(a) - 1; n > 0; n--)
 

Además me tomé la libertad, ayudado por las fotos, de agregar unas cositas al main, para que te sea más fácil probar los números en la consola.

Por lo que nuestro código completo quedaría así:

Código C:

public static void Main(string[] args)
        {
            string valor;
            int a, n;
            while (true)
            {
                Console.WriteLine("escriba un numero para ver el primero q lo divide (ENTER = Salir)");
                valor = Console.ReadLine();
                if (valor.Trim().Length == 0) break;
                a = Convert.ToInt32(valor);
                n = MD(a);
                Console.WriteLine("{0}\n", n);              
            }
            
        }
        
        public static int MD(int a)
        {
            int n;
            
            for (n = Math.Abs(a) - 1; n > 0; n--) // n-- es lo mismo que n = n - 1
            {
                if (a % n == 0)
                {
                    return n;
                } // if
            } // for
    
            return 1; // si se llega hasta aquí es porque no se encontró divisor
        } // MD
 

Espero que te haya aclarado un poco más el panorama.

Cualquier duda no dudes en preguntar.

Saludos cordiales.

--------------------------------------------------------------------------------------------------------

muchas gracias =) anda perfecto =). no sabia de esa funcion para el valor absoluto. se me habia ocurrido elevar al cuadrado la variable y sacarle la raiz cuadrada pero asi con esa funcion es mas facil. y con respecto al for, la verdad que me da verguenza jaja, hace mucho que no programo en este lenguaje, estube aprendiendo haskell y es muy diferente y deje mucho de lado este lenguaje. ahora lo que quiero es usar esa funcion para que me saque el MCD de dos numeros. se me complico un poco recien pero ya lo podre hacer jeje. de nuevo te agradesco mucho por la ayuda. un saludo