インスタンス変数でそれぞれを使用するとエラーが発生する

Question

私のすべての配列をループして、私のjson idの値を真のメソッドを渡すものと比較しようとしています。私はそれをしようとする場合でも、私はエラーを取得する：

undefined method 'each' for nil:NilClass

heroe.rb

require 'json' 

class Hero 
    attr_accessor :hp 

    json = File.read('skill_list.json') 
    @skills = JSON.parse(json) 

    def initialize(name, level, job, hp, strength) 
    @name = name 
    @level = level.to_i 
    @job = job 
    @hp = hp.to_i 
    @strength = strength.to_i 
    end 

    def get_name 
    @name 
    end 

    def profile 
    puts "#{@name} level is #{@level} and hp: #{@hp}" 
    end 

    def attack(id) 
    @skills.each do |s| 
     if s['id'] == id 
     puts true 
     else 
     puts false 
     end 
    end 
    end 

    def get_hp 
    puts @hp 
    end 
end 

skill_list.json

[ 
    { 
    "id": 0, 
    "name": "Double Slash", 
    "element": "fighter", 
    "amp": 1.1, 
    "mp": 1 
    }, { 
    "id": 1, 
    "name": "Slash", 
    "element": "fighter", 
    "amp": 1, 
    "mp": 1 
    }, { 
    "id": 2, 
    "name": "Falling Ashes", 
    "element": "psychic", 
    "amp": 1, 
    "mp": 1 
    } 
] 

その後、私は攻撃を持つオブジェクトと呼ばれます方法：hero.attack(1)

私はちょっとここに詰まっていますpは高く評価されます〜

Answer 1

主な問題は、@skillsがない任意のインスタンスで、そのクラスのプロパティになり、クラスのコンテキスト内で宣言されていることです。インスタンスメソッドには、これとは無関係の独自のコンテキストがあります。 MyClass.aとMyClass#aのようなものは異なるものですが、ここでも同じです。これを修正

は、単に怠惰な初期化子をクラスメソッドに一度、そのデータをロードする場合です：

class Hero 
    attr_accessor :name 
    attr_accessor :hp 
    attr_accessor :level 
    attr_accessor :strength 

    def self.skills 
    @skills ||= JSON.parse(File.open('skill_list.json')) 
    end 

    def initialize(name, level, job, hp, strength) 
    @name = name 
    @level = level.to_i 
    @job = job 
    @hp = hp.to_i 
    @strength = strength.to_i 
    end 

    def profile 
    puts "#{name} level is #{level} and hp: #{hp}" 
    end 

    def attack(id) 
    self.class.skills.each do |s| 
     if s['id'] == id 
     puts true 
     else 
     puts false 
     end 
    end 
    end 
end 

ます。またhp正しくattr_accessorを使用したが、その後は非常に非ルビーを行くと定義しています完全に無関係なget_hpの方法である。あなたは、すべてのプロパティのアクセサを使用して、それと他のget_メソッドを取り除くことができます。

Answer 2

インスタンス変数は、クラス内だけでなく、initializeメソッドまたは他のインスタンスメソッドで宣言する必要があるためです。 私はの初期化メソッドの中に次のものを置くべきだと思います。それは一度だけ呼び出されるように

json = File.read('skill_list.json') 
    @skills = JSON.parse(json) 

または動きが代わりに一定のスキルに@skills。

class Hero 
    attr_accessor :hp 

    SKILLS = JSON.parse(File.read('skill_list.json')) 

    # ... 
end