Нулевой класс при попытке инициализации колоды карт

Question

Следующий код приводит к нулю, и я не могу понять, почему. Что-то не так с моей инициализацией?

class Card 
    VALUES = %w(2 3 4 5 6 7 8 9 10 J Q K A) 
    SUITS = %w(S H D C) 

    def initialize(suit, value) 
     @suit = suit 
     @value = value 
    end 
end 

class Deck 
    attr_accessor :cards 

    def initialize 
     @cards = [] 
     Card::SUITS.each do |suit| 
      Card::VALUES.each do |value| 
       @cards << Card.new(suit, value) 
      end 
     end 
    end 
end 

Deck.new 

p @cards 

Answer 1

Вы создаете новый объект Deck.new, но не напечатает значение его cards - печатать переменную @cards экземпляра, который в этом контексте является nil.

Возможно, вы хотели что-то вроде p Deck.new.cards.

Answer 2

@cards не известен за пределами объекта. Вне класса Deck это переменная экземпляра области верхнего уровня в Ruby.

Вы должны использовать метод доступа, чтобы получить содержание:

class Card 
    VALUES = %w(2 3 4 5 6 7 8 9 10 J Q K A) 
    SUITS = %w(S H D C) 

    def initialize(suit, value) 
     @suit = suit 
     @value = value 
    end 
end 

class Deck 
    attr_accessor :cards 

    def initialize 
     @cards = [] 
     Card::SUITS.each do |suit| 
      Card::VALUES.each do |value| 
       @cards << Card.new(suit, value) 
      end 
     end 
    end 
end 

deck = Deck.new #<--- Store object in a variable 

p deck.cards #<--- Use accessor 

Answer 3

или просто:

my_deck = Deck.new 

p my_deck.cards 

Answer 4

Короче говоря, переменных экземпляра можно увидеть только с помощью других методов из внутри тот же класс. Я считаю, что вы пытаетесь сделать, это:

new_deck = Deck.new 
p new_deck.cards 

Вызов метода cards на new_deck возвращается @cards.

Answer 5

В настоящее время ваш выход для массива @cards трудно прочитать и содержит информацию о объекте. Я думал, что предложить альтернативу я просто conjoured вверх, надеюсь, что это помогает:

class Deck 

    attr_writer :suits, :values 
    attr_accessor :deck 

    def initialize 
    suits 
    values 
    generate_deck 
    shuffle 
    end 

    def generate_deck 
    @deck = [] 
    @suits.each do |suit| 
     @values.each { |value| @deck << [suit, value] } 
    end 
    end 

    def suits 
    @suits = ['Clubs', 'Diamonds', 'Hearts', 'Spades' ] 
    end 

    def values 
    @values = ('2'..'10').to_a + ['J','Q','K','A'] 
    end 

    #just in case you want to shuffle your deck 
    def shuffle 
    @deck.shuffle! 
    end 

end 

require 'pp' #this makes the output prettier 
new_deck = Deck.new 
pp new_deck.deck #calls your current deck so you can see it 

Выходной пример:

$ ruby yourfilename.rb 
[["Hearts", "K"], 
["Spades", "5"], 
["Clubs", "7"], 

#code omitted... (the rest of your cards would be here) 

["Clubs", "K"], 
["Hearts", "5"], 
["Diamonds", "J"], 
["Hearts", "7"]]