2012-08-12 63 views
1

我具有指示的坐标值,像这样数组的数组:红宝石:结合数组值

cells = [ [0,0], [0,1] ] 

阵列中的每个阵列是X和Y值。所以,如果我想改变这个权利,那么在每个单元格上将会是X+1。我可以表达这种像这样的小区:

delta = [1,0] 

现在,我想要做的是合并该值到每个细胞,使每个单元的X值与增量值相加,所以在这种情况下,最终的输出应该是:

new_cells = [ [1,0], [1,1] ] 

下面是我能想到的迄今为止最好的,看来真的是沉重的:

cells = [[0,0],[0,1]] 
delta = [1,0] 
cells.each do |cell| 
    cell[0] = cell[0] + delta[0] 
    cell[1] = cell[1] + delta[1] 
end 
# Now cells = [[1,0],[1,1]] 

是否有一个更清洁的一行样的方法,将总和一个ARR ay到数组链中的每个数组上,还是上述问题的最佳解决方案?

+0

需要它是就地更新?你这样做会失去一些美好的东西...... – tokland 2012-08-13 08:05:34

回答

3

我觉得你最好的解决办法是要认识到细胞和三角洲是如果它不是一个数组,你可以更清楚的操作在不同的数据类型:

Cell = Struct.new(:x, :y) do 
    def + other 
    Cell.new(self.x + other.x, self.y + other.y) 
    end 
end 

# cells is some array of Cell objects 
# delta is some Cell object 

cells.map! {|cell| cell + delta} 
+1

这是一个有趣的事情。我认为它可能是最干净的实现,并且从Struct构建允许一个非常简洁的定义。感谢所有的投入,我喜欢这个最好的! – Andrew 2012-08-13 03:13:51

2

这里:

cells = cells.map {|c| [c[0] + delta[0], c[1] + delta[1]] } 
1
cells.map! {|x, y| [x + delta[0], y + delta[1]] } 

注意Linuxios的答案,我的答案,你的解决方案都有不同的效果,如果别人已经包含在对细胞阵列或细胞的一个参考阵列。只有您的答案会修改原始单元格,因此使用我的解决方案或Linuxios的解决方案引用可能仍然指向旧数据。

+0

在这个答案中,如果你使用#map!不会修改单元格吗?假设没有任何组成单元格作为变量存储在其他地方,一旦“单元格”被爆炸映射,我不确定我会看到旧值如何仍然存在...... – Andrew 2012-08-12 23:10:41

+0

@Andrew:如果其中一个单元格在别处被引用,我的解决方案不会更新该参考,但您的将会。如果在其他地方引用'cells'数组,Linuxios的解决方案将不会更新该引用,但您的解决方案和我的解决方案将会。 – 2012-08-13 01:44:47

1
cells.map {|cell| cell.zip(delta).map{|x, y| x + y }} 

我不认为这比其他解决方案更清洁。

+0

+1。这就是我写的东西,它可以工作,不管矢量大小。 – tokland 2012-08-12 22:50:46

+0

我对#zip并不熟悉。我看到[docs](http://www.ruby-doc.org/core-1.9.3/Array.html#method-i-zip),但对我来说还是有点不清楚。你介意在这种情况下详细阐述一下#zip在做什么? – Andrew 2012-08-12 23:08:26

+0

@Andrew:'[1,2,3] .zip([4,5,6])'给出结果'[[1,4],[2,5],[3,6]]'。 – 2012-08-13 01:45:22

1

这里是你怎么不担心匹配与三角洲的细胞键:

cells = cells.map {|cell| [cell,delta].transpose.map {|value| value.reduce(:+)}} 

一步一步:

cells = cells.map { |cell|   # => [0,0] 

    combined = [cell, delta]   # => [[0,0], [1,0]] 

    transposed = combined.transpose # => [[0, 1], [0, 0]] 

    new_c = transposed.map { |value| # => [0, 1] 

    value.reduce(:+)    # => 1, => 0 

    } 

    new_c       # => [1,0] As expected for first cell. 
} 

cells        # => [[1,0],[1,1]] Final result 

与另一个样本数据:

cells = [[0,0],[1,1],[2,2]] 
delta = [1,1] 
plug = Proc.new { 
    cells = cells.map { |cell| [cell, delta].transpose.map { |value| value.reduce(:+) } } 
} 

plug.call # => [[1, 1], [2, 2], [3, 3]] 
plug.call # => [[2, 2], [3, 3], [4, 4]] 
plug.call # => [[3, 3], [4, 4], [5, 5]] 
plug.call # => [[4, 4], [5, 5], [6, 6]] 
plug.call # => [[5, 5], [6, 6], [7, 7]] 

另一种:

cells = [[0,0,0],[0,1,2],[1,2,3],[2,3,4]] 
delta = [3,2,1] 

plug.call # => [[3, 2, 1], [3, 3, 3], [4, 4, 4], [5, 5, 5]] 
plug.call # => [[6, 4, 2], [6, 5, 4], [7, 6, 5], [8, 7, 6]] 
plug.call # => [[9, 6, 3], [9, 7, 5], [10, 8, 6], [11, 9, 7]] 
plug.call # => [[12, 8, 4], [12, 9, 6], [13, 10, 7], [14, 11, 8]] 
plug.call # => [[15, 10, 5], [15, 11, 7], [16, 12, 8], [17, 13, 9]] 

希望这能更好地回答你的问题。

+1

1)我会写'[cell,delta] .transpose' - >'cell.zip(delta)'。 2)我认为减少是没有必要的,当你压缩三角形的向量时,你总是有2个值,所以不需要减少,只需要一对。这就是肯布卢姆的答案。 – tokland 2012-08-12 22:50:02

+1

非常有趣的方法,感谢彻底的一步一步! – Andrew 2012-08-12 23:04:55

+0

@tokland:同意 – 2012-08-13 18:08:07