0

我正在尝试解决 Projecteuler #11，但是当我尝试创建一个函数来计算一列中每 4 个数字的乘法时遇到了错误。我收到一个错误：

    Project11.rb:59:in `sumvertical': undefined method `[]' for nil:NilClass (NoMeth
odError)

我觉得这里有一些我很容易忽视的东西。感谢您的帮助！

#project #11 http://projecteuler.net/problem=11
grid="08 02 22 97 38 15 00 40 00 75 04 05 07 78 52 12 50 77 91 08
49 49 99 40 17 81 18 57 60 87 17 40 98 43 69 48 04 56 62 00
81 49 31 73 55 79 14 29 93 71 40 67 53 88 30 03 49 13 36 65
52 70 95 23 04 60 11 42 69 24 68 56 01 32 56 71 37 02 36 91
22 31 16 71 51 67 63 89 41 92 36 54 22 40 40 28 66 33 13 80
24 47 32 60 99 03 45 02 44 75 33 53 78 36 84 20 35 17 12 50
32 98 81 28 64 23 67 10 26 38 40 67 59 54 70 66 18 38 64 70
67 26 20 68 02 62 12 20 95 63 94 39 63 08 40 91 66 49 94 21
24 55 58 05 66 73 99 26 97 17 78 78 96 83 14 88 34 89 63 72
21 36 23 09 75 00 76 44 20 45 35 14 00 61 33 97 34 31 33 95
78 17 53 28 22 75 31 67 15 94 03 80 04 62 16 14 09 53 56 92
16 39 05 42 96 35 31 47 55 58 88 24 00 17 54 24 36 29 85 57
86 56 00 48 35 71 89 07 05 44 44 37 44 60 21 58 51 54 17 58
19 80 81 68 05 94 47 69 28 73 92 13 86 52 17 77 04 89 55 40
04 52 08 83 97 35 99 16 07 97 57 32 16 26 26 79 33 27 98 66
88 36 68 87 57 62 20 72 03 46 33 67 46 55 12 32 63 93 53 69
04 42 16 73 38 25 39 11 24 94 72 18 08 46 29 32 40 62 76 36
20 69 36 41 72 30 23 88 34 62 99 69 82 67 59 85 74 04 36 16
20 73 35 29 78 31 90 01 74 31 49 71 48 86 81 16 23 57 05 54
01 70 54 71 83 51 54 69 16 92 33 48 61 43 52 01 89 19 67 48"

grid=grid.split()
grid=grid.collect {|s| s.to_i}
multiarray=[]
i = 0
e = 19
until e > 400
    multiarray << grid[i..e]
    i+= 20
    e+= 20
end

def sumhorizontal(x) #checks sum of all horizontal 4 elements
sum = 0
    x.each {|a|
        i=0
        e=3
        while e < a.length
            if a[i..e].inject(:*) > sum
                sum = a[i..e].inject(:*)
                i += 1
                e += 1
            else
                i += 1
                e += 1
            end
        end
    }
    return sum
end

def sumvertical(x)
sum = 0
    i=0
    e=0
    while e < x.length  #Will break once the end point is longer than the length of an array
        until i > 20   #Checks the first column      
            if x[i][e]*x[i+1][e]*x[i+2][e]*x[i+3][e] > sum #Error is here
                sum = x[i][e]*x[i+1][e]*x[i+2][e]*x[i+3][e]
                i += 1
            else
                i += 1
            end
        end
        e += 1 #once you are out of the until statement, it increases e by 1 to check the next column
        i = 0  #resets i so it can go back to the zero
    end
return sum
end

print sumvertical(multiarray)
4

1 回答 1

2

网格有 20 行。您的循环实际上是在尝试一直到达第 24 行；那是因为它经历了 21 次迭代（i从 0 开始，一直到等于 21），并且每次迭代都超过了 3 的当前值i（当您调用时x[i+3]）。当i是 17 时，您的代码将中断，因为x[i+3][e]试图索引到x. i+3是 20，但最高可用索引是 19。所以会发生什么，x[20]返回nil，然后[]调用方法 on nil，这会产生错误。

此外，标准库有一个转置方法，您可以在数组上调用它。如果你使用它，你只需要一种方法（sumhorizontal）。您可以使用 获取列总和sumhorizontal(multiarray.transpose)。

还有一件事......看起来你来自一种程序语言。Ruby 有一个广泛的标准库和编码结构，可以为您节省大量时间和击键。在 Ruby中，通常不需要迭代while循环和索引变量。sumhorizontal，例如，可以这样写（它确实应该被称为producthorizontal，但如果你试图解决Project Euler #11：

def sumhorizontal(x)
  x.map { |r| r.each_slice(4).map { |s| s.reduce(:*) }.max }.max
end

祝您在接下来的 Ruby 学习之旅中好运！

于 2013-10-22T06:51:50.070 回答