Rubyで引数をkind_ofなどを使って型チェックしたほうが良いのでしょうか。

Question

Rubyで引数をkind_ofなどを使って型チェックしたほうが良いのでしょうか。
Hashの場合は更にその中身も不明です。
呼び出し側で調整したほうがいいのでしょうか。

よろしくお願い致します。

def initialize keys={}
    keys = {} if keys.nil?
    #@coll = self.class.collection

    #hashがRailsのPlaceクラスから来た場合:mongodbから来た場合
    @id = keys[:_id].nil? ? keys[:id] : keys[:_id].to_s

    @address_components = []
    acs = keys[:address_components] ||= []
    acs.each do |ac|
      @address_components << AddressComponent.new(ac)
    end
end

Answer 1

ケースバイケースだと思います。

メソッド呼び出しが未定義エラーにならないようにという観点では、kind_of?ではなくて、respond_to?で、オブジェクトがそのメソッドを呼び出し可能か調べます。

案1:
x.upcase! if x.kind_of? String
案2:
x.upcase! if x.respond_to? :upcase!

ただ、これだと、同名で無関係なメソッドを呼び出してしまう可能性があるので、確かに文字列であると判断したいときには、案1を使うことになります。

逆に、respond_to?で判断しておけば、継承関係にない別のクラスに同名の類似機能のメソッドを定義してそのクラスでも対応できることになります。この場合、kind_of?でチェックしていると修正しないと別クラスに対応できません。
例：IOクラスとStringIOクラスは無関係なクラスですが、どちらも類似のwriteメソッドを持っているので、引数にIOクラスオブジェクトを取るつもりで書いたメソッドでも、kind_of? IOというチェックをしてなければ、StringIOクラスオブジェクトを渡しても多分うまく動きます。
参考：ダックタイピング

Comments

[Yussa]

有難うございます。ダックタイピングという言葉を初めて知りました。この言語を業務で使うのは未だ怖いです。都度チェックするしか無いですかね。

Answer 2

一案です。
0. 作ったスクリプトは自分以外使わない。->どんな書き方でも良い。メンテナンス性も含めてHave Your Own Risk
0. クラスは自分以外の人も使う。->型チェック，エラー処理をクラス側に書いておく。なぜなら，自分以外の人がどんな引数で呼び出すかをコントロールできない。

Comments

[Yussa]

有難うございます。人のプログラムも見て見るようにします。