[Solucionado] [C#] Problema función factorial

Dogod · 27-ene-2010, 09:53

Hola a todos, qué hay de malo en el siguiente código?:

Código:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication8
{
    class Program
    {
        int n;
        int factorial;

        public static void Main(string[] args)
        {
            Console.WriteLine("Ingrese un número para calcular su factorial");
            n = int.Parse(Console.ReadLine());
            Console.ReadLine();
            int resultado = sumar(n);

        }

            double sumar(int n)

{

 if (n == 0)
            {
                factorial = 1;
            }
            else
            {
                factorial = n*factorial(n - 1);
                return factorial;

        }

}

    }
    }

Intento calcular el factorial recursivamente pero no me funciona,

Muchas gracias

Fashion · 27-ene-2010, 11:07

Cita:

Iniciado por Dogod

Hola a todos, qué hay de malo en el siguiente código?:

Código:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication8
{
    class Program
    {
        int n;
        int factorial;

        public static void Main(string[] args)
        {
            Console.WriteLine("Ingrese un número para calcular su factorial");
            n = int.Parse(Console.ReadLine());
            Console.ReadLine();
            int resultado = sumar(n);

        }

            double sumar(int n)

{

 if (n == 0)
            {
                factorial = 1;
            }
            else
            {
                factorial = n*factorial(n - 1);
                return factorial;

        }

}

    }
    }

Intento calcular el factorial recursivamente pero no me funciona,

Muchas gracias

Se ha comentado los errores:

Código:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication8

{
    class Program
    {
        //int n;
        //int factorial;  

        //public double sumar(int n)
        public static int sumar(int n)
        {            
            if (n == 0)         
                //factorial = 1;
                return 1;            
            else            
                //factorial = n*factorial(n - 1);
                return n * sumar(n - 1);
        }

        static void Main(string[] args)
        {
            int n;
            Console.WriteLine("Ingrese un número para calcular su factorial");
            n = int.Parse(Console.ReadLine());            
            //int resultado = sumar(n);
            Console.WriteLine("Factorial: " + sumar(n));
            Console.ReadLine();
        }
    }
}

ajr784 · 27-ene-2010, 19:33

Primer error así al sondeo rápido.
¿Por qué si calcula factorial, la función se llama sumar?, es más... en la operación factorial no interviene para nada la suma.....

Ahora, lee bien el manual de tu lenguaje. Lo que tu haz hecho es crear un método para la clase Program, es decir que necesariamente debes crear un objeto Program y enviarle el mensaje.

Código:

using ... // ganzadas que le gusta a C#

namspace UnNamespace  // ganzada que pone Visual Studio
{
    class Program
    {
         // Variables de instancia y toda la bola  
         // .............

          public static void Main()
          {
               Program p = new Program(); // crear el objeto program
               p.factorial(10);  // Enviar el mensaje factorial al objeto p
          }

           public int factorial(int nro)
           {
                  // codigo de factorial.. creo que está bien y no tengo ganas de pensar
           }
    }
}

Así es cómo tu planteas la solución. Ahora, como no es necesario que sea un método de objeto (un mensaje) sino que puede ser un método de clase, es lo que Fashion te marca como corregir el error:

Código:

// bla bla bla toda habladuría anterior
    public static int factorial(int factoria)
    {
    }

Dogod · 30-ene-2010, 18:10

Lo he resuelto. Pero gracias.

Herramientas
Mostrar Versión Imprimible Enviar por Correo
Desplegado
Mode Lineal Cambiar a Modo Hibrido Cambiar a Modo Hilado

27-ene-2010, 19:33	#3
ajr784 Senior Member Fecha de Ingreso: marzo-2007 Mensajes: 474	Primer error así al sondeo rápido. ¿Por qué si calcula factorial, la función se llama sumar?, es más... en la operación factorial no interviene para nada la suma..... Ahora, lee bien el manual de tu lenguaje. Lo que tu haz hecho es crear un método para la clase Program, es decir que necesariamente debes crear un objeto Program y enviarle el mensaje. Código: using ... // ganzadas que le gusta a C# namspace UnNamespace // ganzada que pone Visual Studio { class Program { // Variables de instancia y toda la bola // ............. public static void Main() { Program p = new Program(); // crear el objeto program p.factorial(10); // Enviar el mensaje factorial al objeto p } public int factorial(int nro) { // codigo de factorial.. creo que está bien y no tengo ganas de pensar } } } Así es cómo tu planteas la solución. Ahora, como no es necesario que sea un método de objeto (un mensaje) sino que puede ser un método de clase, es lo que Fashion te marca como corregir el error: Código: // bla bla bla toda habladuría anterior public static int factorial(int factoria) { }

30-ene-2010, 18:10	#4
Dogod Member Fecha de Ingreso: marzo-2009 Mensajes: 44	Lo he resuelto. Pero gracias.