输入后输出控制台冻结

Question

我一直在尝试编码来查找给定数组元素的LCM。我的代码如下

public long LcmOfArray(List<int> a)
{
    long LCM = 1;
    bool divisible = false;
    int divisor = 2, count = 0;

    while (count != a.Count)
    {
        for (int i = 0; i < a.Count; i++)
        {
            if (a[i] == 0)
            {
                return 0;
            }
            if (a[i] < 0)
            {
                a[i] *= -1;
            }
            if (a[i] % divisor == 0)
            {
                divisible = true;
                a[i] /= divisor;
            }
            if (a[i] == 1)
            {
                count++;
            }
        }

        if (divisible)
        {
            LCM *= divisor;
        }
        else
        {
            divisor++;
        }
    }
    
    return LCM;
}

我的问题是，当我输入时，输出控制台会冻结。我尝试了另一种没有while循环的方法。也就是说，消除了while循环，并在最后添加了一个if循环。

if (count == a.Count)
{
    return LCM;
}

但现在编译器抛出一个错误，指出并非所有代码路径都返回一个值。有人能帮我解决代码中的问题吗？我是一名编程初学者。提前谢谢！！

Answer 1

一个while循环将持续运行，直到你给它一些东西来打破循环。在您的情况下，它打破循环的唯一方式似乎是在列表中的每一项都是1或达到0的情况下，因此根据列表的不同，它可能会或可能不会陷入无限循环。例如，a[i]=2似乎创建了一个无限循环。另一个编译器错误是因为某些条件可能会跳过返回语句。

if(count==a.Count)
{
   return LCM;
}
return void; // because what if count does not equal a.Count?

Answer 2

你应该把bool divisible=false 和int count=0 在while循环中。既然你不能把int count=0在while循环之外，您应该使用while（true）而不是while（count！=a.count）并将下面的if语句放在while循环的最后一个。

if (count == a.Count)
{
    return LCM;
}

以下是完整的方法：

public static long LcmOfArray(List<int> a)
{
    long LCM = 1;
    int divisor = 2;

    while (true)
    {
        int count = 0;
        bool divisible = false;
        for (int i = 0; i < a.Count; i++)
        {
            if (a[i] == 0)
            {
                return 0;
            }
            if (a[i] < 0)
            {
                a[i] *= -1;
            }
            if (a[i] == 1)
            {
                count++;
            }
            if (a[i] % divisor == 0)
            {
                divisible = true;
                a[i] /= divisor;
            }
        }
        if (divisible)
        {
            LCM *= divisor;
        }
        else
        {
            divisor++;
        } 
        if (count == a.Count)
        {
            return LCM;
        }
    }
}