インスタンスを返すためのRubyのベストプラクティスは、定義されていない可能性があります

Question

Rubyでは、の2つのの4つのケースで違いがありますか？もしそうなら、どちらがより良い方法ですか？

class OptionOne 
    def initialize(arr) 
    @arr = arr 
    end 
    def arr 
    @arr || [] 
    end 
end 

class OptionTwo 
    def initialize(arr) 
    @arr = arr 
    end 
    def arr 
    (defined? @arr) ? @arr : [] 
    end 
end 

第三の選択肢、イリヤが言及したのRubyスタイルガイドにしたがって：

class OptionThree 
    def initialize(arr) 
    @arr = arr 
    end 
    def arr 
    @arr ||= [] 
    end 
end 

そしてキース・ベネットからの回答から4番目のオプション：

class OptionFour 
    attr_accessor :arr 
    def initialize(arr = []) 
    @arr = arr 
    end 
end 

Answer 1

注：元の質問に対する答えです。

def arr 
    @arr ||= [] 
end 

a ||= ba || a = bのためだけの速記：ルビーstyle guide、より有効に活用memoizationによると

。さらに詳しくhere。

Answer 2

結果のオブジェクトで何をしたいかによって異なります。

@arrがどこかの値に設定されていないと、あなたのオプションは常に新しいオブジェクトを返します。これにより、この値がOptionOneインスタンスの状態に影響しないことが保証されます。 OptionThreeの主な違いは、この場合の動作である：OptionThreeため

option_one = OptionOne.new(nil) 
arr = option_one.arr 
arr[0] = "Hello" 

arr 
# => ["Hello"] 
option_one.arr 
# => [] 

対：OptionThreeの場合

option_three = OptionThree.new(nil) 
arr = option_three.arr 
arr[0] = "Hello" 

arr 
# => ["Hello"] 
option_three.arr 
# => ["Hello"] 

、単一の配列オブジェクトは、オプションのオブジェクトに保存されているであろう後のコードで変更可能です。 OptionOneの場合、常に新しいArrayオブジェクトが返されます。

@arrがまだtruethy値（たとえば配列）で初期化されていない場合にのみ、この区別が成立することに注意してください。その場合、両方の変種はOptionThreeの動作を示します。多くの場合、同じ動作を保つためにOptionThreeが実装されていることがよくあります。

オプションオブジェクトの内部状態を常に保持することがわかっている場合は、毎回新しいオブジェクトを返すのが理にかなっています。

OptionTwoになりましたが、これは機能しません。イニシャライザで常に@arrを設定するので、常に設定され、したがって返されます。したがって、else部分は決して実行されません。インスタンス変数とdefined?を使用すると、ほとんど常に悪い考えであることを

def OptionTwo 
    def initialize(arr) 
    @arr = arr 
    end 

    attr_reader :arr 
end 

注：お使いのOptionTwoは、このように短縮することができます。 Rubyでは、インスタンス変数は暗黙的に多くの場合に初期化されます。どこにでもそれを読んでいるとき（前に明示的な値に設定されていたかどうかは関係ありません）。したがって、defined?の振る舞いに頼っているときは、間違って定義しないように注意する必要があります。しばしば、それが定義されていると仮定し、その値に基づいて振る舞いを処理する方がずっと便利です。

nilとfalseが@arrの有効な値でない場合は、いつでも安全に||=を使用してアレイにそれを初期化することができます。これらのどれかが有効な値である場合、正しい初期化を選択するために追加のロジックを追加する必要があります。