Question

我有当前的编码，我觉得它似乎接近我需要但我似乎无法让它为我想要的工作。我试图让它输出输入的两个数字的最高公因数。

            i = myInt;

            {
                if (myInt % i == 0 && myInt2 % i == 0)
                {
                    Console.Write("Your GCF is...");
                    Console.Write("{0} ", i);
                    Console.ReadLine();

                }
                else
                    i--;
                goto;
            }

Answer 1

正如其他人在评论中所说，你应该真的避免goto陈述，因为它们是不好的做法，特别是当你正在学习大学的编程课程时（通常应该符合结构编程）。而是使用while循环（或任何其他）与两个条件，如示例中所示。此外，我认为您应该从较小的数字开始搜索（第一次输入不需要更小），这在性能方面有一点改进。这是代码：

static void Main(string[] args)
{    
    string myInput;
    int myInt;
    string myInput2;
    int myInt2;
    int i;

    Console.Write("Please enter a number: ");
    myInput = Console.ReadLine();
    myInt = Int32.Parse(myInput);

    Console.Write("Please enter another number: ");
    myInput2 = Console.ReadLine();
    myInt2 = Int32.Parse(myInput2);

    i = myInt > myInt2 ? myInt2 : myInt;
    bool found = false;
    while(!found && i>0)
    {
        if (myInt % i == 0 && myInt2 % i == 0)
        {
            Console.Write("Your GCF is...");
            Console.Write("{0} ", i);
            Console.ReadLine();
            found = true;
        }
        else
            i--;
    }
}

编辑：由于@Servy

，我包含了其他可能的解决方案

bool found = false;
for( i = Math.Min(myInt, myInt2); !found && i>0; i--)
{
    if (myInt % i == 0 && myInt2 % i == 0)
    {
        Console.Write("Your GCF is...");
        Console.Write("{0} ", i);
        Console.ReadLine();
        found = true;
    }
}

Answer 2

        {
label:
            if (myInt % i == 0 && myInt2 % i == 0)
            {
                Console.Write("Your GCF is...");
                Console.Write("{0} ", i);
                Console.ReadLine();

            }
            else
                i--;
            goto label;
        }

会做的。但是，这是一个非常糟糕的主意。而是学习如何使用while。

最高的共同因素C＃

2 个答案: