通过使用因子函数方法组合

时间:2014-12-21 14:42:53

标签: c# class combinations factorial

我无法组合20和17,程序说结果是1.为什么?我确定我的代码是正确的,但我无法将大数字组合在一起。

using System;
namespace question
{
    class beat_That
    {
        static int Factorial(int m)
        {
            int result = 1;
            for (int i = 1; i <= m; i++)
            {
                result *= i;
            }
            return result;
        }
        static void Main(string[] args)
        {
            Console.WriteLine("enter number of objects in the set: ");
            int n = Convert.ToInt32(Console.ReadLine());

            Console.WriteLine("enter number to be chosen: ");
            int k = Convert.ToInt32(Console.ReadLine());

            int combination = Factorial(n) / (Factorial(n - k) * Factorial(k));
            Console.WriteLine("C(" + n + ", " + k + ") = " + combination);

            Console.ReadKey();

        }
    }
}

1 个答案:

答案 0 :(得分:0)

我猜这是家庭作业?以下是一些有助于您朝着正确方向前进的提示:

(1)通常,.NET类为Pascal Case,例如:comb应为Comb。此外,最好分配描述性的类名而不是简短的缩写。为清楚起见,我假设您至少将comb重命名为Comb,以免与变量名称混淆,但另一个选项可能是Calculator

(2)检查语法和编译器错误。例如,编译器应该抱怨这行代码: Console.WriteLine("the combination of {0} and {1} is {2}. "),a1,b1,;

(3)您的方法FactorialCombinationstatic方法(与实例方法相对)。这会改变您调用这些方法的方式。在没有实例的情况下调用静态方法,例如:Comb.Combination(..)

(4)确保测试各种输入!你对Combination的实现并不完全正确,但我会将其作为练习来解决原因。