抛出异常时可以退出循环吗？

Question

我在Hackerrank上解决了一个任务。com，问题是这样的：

你有一个数组。这个数组包含数字。
现在输入两个数字：

• 第一个描述了一个总和
• 第二个描述了您添加在一起的索引量（序列长度）

最后你得到的序列的数量其总和就是你定义的数字

例如：

我为此写了这段代码：

static int GetSequences(List<int> s, int d, int m)
{
    //m = segment-length
    //d = sum

    int count = 0;
    int j = 0;
    int k = 0; 

    do
    {
        try
        {
            List<int> temp = new List<int>();

            for (int i = 0; i < m; i++)
            {
                temp.Add(s[i + k]);
            }
            if (temp.Sum() == d)
            {
                count++;
            }
            j++;
            k++;
        }
        catch (ArgumentOutOfRangeException)
        {
            break;
        }

    } while (true);

    return count;
}

由于我不知道需要多少次计数（例如，序列长度为3的6长度数组有4个序列（1,2,3,4 | 3,4,5 | 4,5,6）），当索引超出范围时，我停止循环。但我不确定这个解决方案是否可行。不仅仅是程序速度，还有代码清洁度。该代码是否可以接受，或者最好使用循环，例如，对于具有3个长度序列的6个长度数组，哪些循环正好4次？

Answer 1

通常，除了开关/机箱外，通常没有使用break的真正原因。此外，例外必须如名称所示，因此它不能成为您逻辑的一部分。正如Jeppe所说，您可以使用框架提供的方法和属性来执行您喜欢的操作。在这里，s.Count似乎是一条可行之路。

Answer 2

最好修复代码，使其不会经常引发异常：

您对这些段中的每一个进行求和：

0  1  2  3    start = 0
|  |          summing indexes: 0, 1
+--+

0  1  2  3    start = 1
   |  |       summing indexes: 1, 2
   +--+

0  1  2  3    start = 2
      |  |    summing indexes: 2, 3
      +--+

括号从索引开始，大小为m。s的长度由s.Count给出。因此，我们希望继续进行，直到开始m==s.Count。

（我总是发现画出这些东西并输入样本编号很有用，以确保您的数学正确。在上面的示例中，您可以看到我们在start（2）m（2）==数组大小（4）时停止）

static int GetSequences(List<int> s, int d, int m)
{
    //m = segment-length
    //d = sum

    int count = 0;

    for (int start = 0; start + m <= s.Count; start++)
    {
        List<int> temp = new List<int>();
        for (int i = 0; i < m; i++)
        {
            temp.Add(s[start + i]);
        }
        if (temp.Sum() == d)
        {
            count++;
        }
    }

    return count;
}

但是，您可以稍微改进代码：

1. 使用有意义的变量名
2. 不要每次都创建一个新的临时列表，只是为了求和
3. 检查您的输入

static int GetSequences(List<int> numbers, int targetSum, int segmentLength)
{
    if (numbers == null)
        throw new ArgumentNullException(nameof(numbers));
    if (segmentLength > numbers.Count)
        throw new ArgumentException("segmentLength must be <= numbers.Count");

    int count = 0;

    for (int start = 0; start + segmentLength <= numbers.Count; start++)
    {
        int sum = 0;
        for (int i = 0; i < segmentLength; i++)
        {
            sum += numbers[start + i];
        }
        if (sum == targetSum)
        {
            count++;
        }   
    }
}

Answer 3

不建议这样做。应该为不应该发生的事情保留例外，而不是流控制或验证。

您想要的是使用条件逻辑（if语句）和关键字。

此外，codereview。stackexchange。com更适合于这类问题。