Ruby 类变量实例变量错误

问题描述

Game 有一个包含十个 Frame 实例的数组

class Frame
  attr_accessor :rolls
  def initialize
    @rolls = ""
  end
  
end

class Game
  attr_accessor :frames
  def initialize
    @frames = Array.new(10,Frame.new) 
  end
  
  def print_frames
    @frames.each_with_index do |frame,idx|
      p "Frame ##{idx+1}: #{frame.rolls}"
    end
  end
end

game = Game.new

rolls = [5,5,3,7,4,2,2]

curr = 0
rolls.each_with_index do |roll|
  game.frames[curr].rolls << roll.to_s
  if game.frames[curr].rolls.size == 2
    curr += 1
  end
end

p "Total rolls: #{rolls.size}"
p game.print_frames

我希望打印 10 行，每行都有一个带有 2 个数字的字符串的帧 ID 但是，返回以下内容

"Total rolls: 20"
"Frame #1: 55374000000000000022"
"Frame #2: 55374000000000000022"
"Frame #3: 55374000000000000022"
"Frame #4: 55374000000000000022"
"Frame #5: 55374000000000000022"
"Frame #6: 55374000000000000022"
"Frame #7: 55374000000000000022"
"Frame #8: 55374000000000000022"
"Frame #9: 55374000000000000022"
"Frame #10: 55374000000000000022"
[#<Frame:0x0000565402e40528 @rolls="55374000000000000022">,#<Frame:0x0000565402e40528 @rolls="55374000000000000022">,#<Frame:0x0000565402e40528 @rolls="55374000000000000022">]

实例变量的作用类似于在帧之间共享的类变量。所有 20 个卷号都连接在一起，而不是每个帧拥有一对。
代码有什么问题？是不是因为 Game 或 Frame 对象没有正确实例化？

解决方法

Array.new(10,Frame.new)

创建一个包含 10 个元素的数组，所有元素都指向单个 Frame 实例。要创建具有 10 个单独 Frame 实例的数组，您应该使用块形式。

Array.new(10) { Frame.new }

这将执行该块 10 次并将每次执行的结果分配给共同发起索引。

见：Creating Arrays

一个数组也可以通过显式调用 ::new 来创建，零， 一个（Array 的初始大小）或两个参数（初始大小 和一个默认对象）。

ary = Array.new    #=> []
Array.new(3)       #=> [nil,nil,nil]
Array.new(3,true) #=> [true,true,true]

请注意，第二个参数使用对 因此，仅在以下情况下才推荐使用 需要使用本机不可变对象实例化数组，例如 符号、数字、真假。

要创建具有单独对象的数组，可以传递一个块 反而。此方法可安全用于可变对象，例如 散列、字符串或其他数组：

Array.new(4) {Hash.new}    #=> [{},{},{}]
Array.new(4) {|i| i.to_s } #=> ["0","1","2","3"]

,

方法 Array.new 它被定义为 new(size=0,default=nil) 这意味着第二个参数将是数组内所有对象的值，默认值。

这就是为什么对于您的情况，所有 10 个对象都相同。

https://ruby-doc.org/core-2.7.0/Array.html#method-c-new

这段代码可以帮你解决

@frames = []
10.times {|i| @frames.push(rand) }
=> [0.7053319996471655,0.34131818323294594,0.4084836724883256,0.20452172335941388,0.5124065818560665,0.4203474973940552,0.6719502264788891,0.7453268015406016,0.09500886225101768,0.9053707563920769]

# try this for your case
# 10.times {|i| @frames.push(Frame.new) } 

# using array creation with block
# ty tadman for the advice
# @frames = Array.new(10) { Frame.new }