[ 命名建議 ] LeetCode 1588. Sum of All Odd Length Subarrays

[LPenny-github]

原 issue:
#89

LeetCode:
https://leetcode.com/problems/sum-of-all-odd-length-subarrays/

---

再麻煩 @twy30 給予命名建議 orz 感激不盡

namespace Sandbox
{
    public class NamSumOfAllOddLengthSubarrayse
    {
        public static int SumOddLengthSubarrays(int[] arr)
        {
            int[] inputNumbers = arr;
            int output = 0;
            int count = 1;

            while (count <= inputNumbers.Length)
            {
                for (int i = 0; i < inputNumbers.Length; ++i)
                {   
                    // 判斷 取出個數 是否超出 inputNumbers 總個數
                    if (i + count > inputNumbers.Length)
                    {
                        continue;
                    }
                    for (int k = 0; k < count; ++k)
                    {
                        output += inputNumbers[ i + k ];
                    }
                }
                count += 2;
            }
            return output;
        }
    }
}

---

這次寫了測試，大部分是 複製貼上 的成果 😅 <--因為上次問了大大關於測試的問題，所以測試也來練一下 😆

小的會好好學習，感激不盡 orz

using Sandbox;
using Xunit;

namespace Tests
{
    public class UnitTestNamSumOfAllOddLengthSubarrayse
    {
        [Fact]
        public void Test1()
        {
        Assert.Equal(NamSumOfAllOddLengthSubarrayse
                    .SumOddLengthSubarrays(new int[]{1,4,2,5,3}), 58);
        }

        [Fact]
        public void Test2()
        {
        Assert.Equal(NamSumOfAllOddLengthSubarrayse
                    .SumOddLengthSubarrays(new int[]{1,2}), 3);
        }

        [Fact]
        public void Test3()
        {
        Assert.Equal(NamSumOfAllOddLengthSubarrayse
                    .SumOddLengthSubarrays(new int[]{10,11,12}), 66);
        }
    }
}

[twy30]

@LPenny-github

你好 😊

---

            int count = 1;

這裡可以考慮用 subarrayLength ，呼應 LeetCode 原題中提到的 "subarray" 。

---

namespace Sandbox
{
    public class NamSumOfAllOddLengthSubarrayse

我猜想 "SumOfAllOddLengthSubarrays" 是取自 LeetCode 第 1588 題的標題；而這 class 名稱最前面的 "Nam", 最後面的 "e" 或許是 typo? 🤔 (可能是手誤而沒有把暫時性的名字 "Name" 取代乾淨？)

---

        public void Test1()
        public void Test2()
        public void Test3()

可以考慮以下兩種命名方式：

• 測試案例來源
  • LeetCodeExample1
  • LeetCodeExample2
  • LeetCodeExample3
• 測試案例性質
  • OddLengthInputArray
  • EvenLengthInputArray

可以參考看看 😊

[LPenny-github]

@twy30

可能是手誤而沒有把暫時性的名字 "Name" 取代乾淨？

真的非常抱歉（大大的通靈功力一流） orz

PS. 聖誕快樂

[LPenny-github]

@twy30 不好意思 orz
我誤把 expected 和 actual 放反了 orz，以下是更正後的版本：

using Sandbox;
using Xunit;

namespace Tests
{
    public class UnitTestNamSumOfAllOddLengthSubarrayse
    {
        [Fact]
        public void LeetCodeExample1()
        {
            Assert.Equal(58, NamSumOfAllOddLengthSubarrayse
                        .SumOddLengthSubarrays(new int[] { 1, 4, 2, 5, 3 }));
        }

        [Fact]
        public void LeetCodeExample2()
        {
            Assert.Equal(3, NamSumOfAllOddLengthSubarrayse
                        .SumOddLengthSubarrays(new int[] { 1, 2 }));
        }

        [Fact]
        public void LeetCodeExample3()
        {
            Assert.Equal(66, NamSumOfAllOddLengthSubarrayse
                        .SumOddLengthSubarrays(new int[] { 10, 11, 12 }));
        }
    }
}