Rubyクラスを初期化するための慣習的な方法

Question

Personオブジェクトを初期化するための慣用方法は何ですか？私は個人的にはperson2を好むが、私はRubyを初めて使っている。

class Person 
    attr_reader :first, :middle, :last, :phone_number 

    def initialize(first, middle, last, phone_number) 
     @first = first 
     @middle = middle 
     @last = last 
     @phone_number = phone_number 
    end 
end 

if caller.length == 0 
    person1 = Person.new("John", "T", "Smith", "555-5555") 
    person2 = Person.new(
     first=  "John", 
     middle=  "T", 
     last=  "Smith", 
     phone_number="555-5555", 
    ) 
end 

Answer 1

通常ルビーでは、オブジェクトは、いずれかのようなperson1か、以下の方法で作成されます。

class Person 
    attr_reader :first, :middle, :last, :phone_number 

    def initialize(options) 
     @first = options[:first] 
     @middle = options[:middle] 
     @last = options[:last] 
     @phone_number = options[:phone_number] 
    end 
end 

person = Person.new(first: "John", middle: "T", last: "Smith", phone_number"555-5555") 

この代替アプローチについての素晴らしい事は、オプションで含めることができるということであるとして、多くのかなどの属性のように、いくつかのあなたは好きなだけでなく、任意の順序を許可します。

Answer 2

person1 = Person.new("John", "T", "Smith", "555-5555") 

私の意見でより良いオプションです。同意しないものもありますが、ここで私はそれを考えています。

まず、コードをできるだけ読みやすくする必要があります。そのようにして、人々は何かが書かれている/構築されている方法を理解するために多くの質問をする必要はありません。それで、もしあなたがそれを第二のやり方でやるなら、あなたは説明を終わります。しかし、それを第2の方法でやってみると、あなたのコードを読んでいる人達には、それが初めてではなく書かれていないことを認めているようなものです。あなたのソフトウェアの開発者は、コンストラクタのパラメータ名を見ることができることを知っているか、IDEは既に自動ドキュメントの形式で指定しています。その後、上記の表記法の両方に代わる

person1 = Person.new("John Smith") 

またはより良いを次のようにコンストラクタで人物名を初期化するだろう...

person1 = Person.new() 

そして、何を説明するために、必要に応じてフィールドを設定します起こっている。二つ目は良く見えるかもしれませんなぜ私は完全に理解しながら

person1.address = "Blah street" 
person1.phoneNumber = "555-5555" 

さて、それは長期的には痛いです。不要なコードや、よりリーンなデザインや強力なツールを使用して削除する方が効果的です。