如何创建一个附加空数组的函数？

Question

// I am attempting to answer the following question:

如果我们列出所有小于 10 且是 3 或 5 倍数的自然数，我们会得到 3、5、6 和 9。这些倍数之和是 23。

//Find the sum of all the multiples of 3 or 5 below 1000.



// function creating both integers, max number, and filtering 
func sumOfMultiples(x: Int, y: Int, n: Int)  {
    var numberList: [Int] = []
    for i in 0..<y {
        if x < n && y < n && x % n == 0 || y % n == 0 {
            return numberList.append(i)
        }
    
    } 
}


sumOfMultiples(x: 3, y: 5, n: 1000)

Answer 1

我不确定您的代码试图实现什么，但由于多种原因它无法工作。

下面是一些实现此目的的工作代码：

func sumOfMultiples(x: Int, y: Int, n: Int)->Int {
    var returnValue: Int = 0

    if x < n && y < n {
        for i in 0..<n {
            if i % x == 0 || i % y == 0 {
                returnValue += i
            }
        }
    }

    return returnValue
}

print(sumOfMultiples(x: 3, y: 5, n: 1000))

输出：

233168

您发布的代码中的问题是：

• 该函数没有返回类型（它是一个 void 函数）。我添加了->Int 以返回一个整数。
• 该函数存储整数列表 - 为什么？如果只需要总和，请将值存储在 Int 中并返回。
• for 循环一直循环到y。其实应该是n。
• 在每个循环迭代中执行x 和y 低于n 的比较。无需浪费资源 - 在循环之前完成。
• 模运算（x % n 和y % n）对用例无效。实际上是i % x 和i % y，因为您在每种情况下都在测试i。

Answer 2

如果你想要的结果是所有数字的总和：

func sumOfMultiples(x: Int, y: Int, n: Int) -> Int {
    var result = 0
    for number in 1..<n {
        if number % x == 0 || number % y == 0 {
            result += number
        }
    }
    return result
}

如果你想要的结果是一个包含所有数字的数组：

func arrayOfMultiples(x: Int, y: Int, n: Int) -> [Int] {
    var result: [Int] =  []
    for number in 1..<n {
        if number % x == 0 || number % y == 0 {
            result.append(number)
        }
    }
    return result
}

您也可以使用自定义范围作为参数，如下所示：

func sumOfMultiples(x: Int, y: Int, n: ClosedRange<Int>) -> Int {
    var result = 0
    for number in n {         //Notice the change from 1..<n to just n
...
}

func arrayOfMultiples(x: Int, y: Int, n: ClosedRange<Int>) -> [Int] {
    var result: [Int] =  []
    for number in n {         //Notice the change from 1..<n to just n
...
}

然后这样称呼它：

sumOfMultiples(x: 3, y: 5, n: 10...20)
//Result is 75

arrayOfMultiples(x: 3, y: 5, n: 10...20)
//Result is [10, 12, 15, 18, 20]