Rubyクラス、メソッドの前後で引数が渡される、インスタンス変数？

Question

私はコードの匂いかもしれないと思うこの問題を抱えています。私はその初期設定で引数を受け取り、1つの公開といくつかのプライベートメソッドを含むクラスを持っています。例：

class Foo 
    def initialize(a) 
    @a = a 
    end 

    def apply?(examples) 
    foo_1(examples) 
    end 

    private 

    def foo_1(examples) 
    foo_2(examples) ? 1 : 2 
    end 

    def foo_2(examples) 
    examples.size > @a 
    end 
end 

ここに私の問題は、パブリックメソッドが周りを超えるとプライベートメソッドを介して実行されることによって受信された「例」であり、何、それはかなり見ていないと、それはコードのにおいのように思えますここに最善のアプローチ？パブリックメソッド内のインスタンス変数にしますか？

おかげ

Answer 1

examplesを受け入れて、プライベートメソッドの数が1〜2よりも大きい場合は、[はい、これはコードのにおいと考えることができます。

ここではruleを表すクラスを抽出することを考慮する必要があります。例えば

：それはそのオブジェクトへの呼び出しの間メモリに固執しまう危険性がありますよう

class Foo 
    def initialize(a) 
    @a = a 
    end 

    def apply?(examples) 
    size_rule_applies?(examples) ? 1 : 2 
    end 

    private 

    def size_rule_applies?(examples) 
    SizeRule.new(@a, examples).apply? 
    end 

    class SizeRule 
    def initialize(a, examples) 
     @a  = a 
     @examples = examples 
    end 

    def apply? 
     @examples.size > @a 
    end 
    end 
end 

私はFooクラスのインスタンス変数examplesをすることはないだろう。私はそのようなバグを見た。

Answer 2

examplesが動的に変更された場合、そのインスタンス変数を変更することはオプションではありません。 examplesごとにFooをインスタンス化する必要があります。examples自体が変更されている可能性があります。Fooが変更される可能性があります。

コードは正常です。私に関係する唯一のことは、ある方法は別の方法に依存するということです。これは通常、大したことではないのですが、これは私には良くなります

def apply?(examples) 
    foo_1(examples) && foo_2(examples) 
    end 

Answer 3

別のオプションは、ブロックを使用することです：

class Foo 
    def initialize(a) 
    @a = a 
    end 

    def apply?(examples) 
    foo_1 { examples } 
    end 

    private 

    def foo_1 
    v = foo_2 { yield.size } 
    v ? 1 : 2 
    end 

    def foo_2 
    yield > @a 
    end 
end 

Answer 4

クラスFooが現在ちょうど私が意志.apply?(examples)を評価するよりも、他には何もしませんので、アドバイスの例をイニシャライザに追加し、インスタンス変数を作成します。これは、より簡単で、より効率的で、より明白です。

class Foo 
    def initialize(a, examples) 
    @a = a 
    @examples = examples 
    end 

    def apply? 
    foo_1 
    end 

    private 

    def foo_1 
    foo_2 ? 1 : 2 
    end 

    def foo_2 
    @examples.size > @a 
    end 
end