if foo.class == Hash && foo[:title] == "bar" を短く書く方法を教えてください

Question

if foo.class == Hash &&  foo[:title] == "bar"
 やりたい処理
end

上記のifの条件部分を短く書きたいです。
fooにはStringやHashやArrayなどどのclassが入ってくるかわからないため、 classのチェックと同時にkey: valueのチェックを行っています。

if foo[:title] == "bar"
 やりたい処理
end

このように書くと、もしfooにHash以外のclassが入っていた場合エラーになってしまいます。
ただ、keyを指定しているのであればHashを期待してることは明らかなので、Hash以外が入っていたらエラーになるのではなくただ一致しなかっただけとして処理をしてほしい、というのが理想です。

あまりうまく伝えられていないかもですが、とにかく、
Hashかどうか、:titleの値はbarか、という二段階のチェックをもっと短く書きたいです。

何か方法は無いでしょうか。

ちなみに、どんなclassが入ってくるかわからないという設計自体がまずいですかね、という疑念が。

Answer 1

すでに回答として上がっておりますが、私ならこうします。

def foo.has?(key, val)
  # Hashクラス かつ valのkeyをもつか？
  self.is_a?(Hash) && self[key] == val
end

if foo.has?(:title, "bar")
 やりたい処理
end

Comments

[退会済みユーザー]

実は取り出したい値がネストしている状況でして、それに対応するメソッドを書いていたのですが行き詰ってしまいました。。

def has_value?(item, key, value)

if key.is_a?(Array)

keys = "'"+key.join("','")+"'"

p eval("#{item}.dig(#{keys}) == '#{value}'")
end
item.is_a?(Hash) && item[key] == value
end

res = { 'error':
{'title': 'Unauthorized'}
}
has_value?(res,[:error,:title],'Unauthorized')

[JUNJUNJUN]

行き詰まること、私もよくあります（笑）
いい解法がでることをお祈りしています。

[退会済みユーザー]

怪しいですけど動きました！

def has_value?(item, key, value)

if key.is_a?(Array)

keys = ":"+key.join(",:")
eval("#{item}.dig(#{keys}) == '#{value}'")
end
item.is_a?(Hash) && item[key] == value
end

res = { "error":
{"title": "Unauthorized"}
}
has_value?(res,['error','title'],'Unauthorized')

https://paiza.io/projects/MgfsZ9rmXn5gowg6u1oS8Q

[JUNJUNJUN]

おめでとうございます～

Answer 2

if ( foo[:title] == "bar" rescue false )

のような荒技を使わない限り、2条件をandで繋がざるを得ないと思います。
短く書くなら、そういうメソッドを定義してifではそれを呼び出すようにするか。

また、foo.class == Hashよりは、Hashのサブクラスのインスタンスでも良いfoo.is_a?(Hash)の方が良いと思います。

#コメントに対して追記。「そういうメソッド」の例。

def has_value?(item, key, value)
    item.is_a?(Hash) && item.dig(*key) == value
end

# キーはシンボルとする
res = {error:
{title: "Unauthorized"}
}
if has_value?(res,[:error,:title],'Unauthorized')
    ～～
end

Comments

[退会済みユーザー]

ありがとうございます。
別の方にコメントで返答させていただいた状況になっております。
お助けいただけないでしょうか。。

[otn]

キーが文字列なのか、シンボルなのか混乱していますね。
「シンボルとは何か」について理解されていますか？
コードは本分に追記します。

[退会済みユーザー]

ありがとうございます。
ご指摘の通り、実はシンボルや文字列の違いを理解しておらず、下記のようなコードにしていました。

def has_hash?(item, *key, value)
item = item.deep_symbolize_keys
if key.is_a?(Array)
keys = ':'+key.join(',:')
eval("#{item}.dig(#{keys}) == '#{value}'")
elsif item.is_a?(Hash)
item[key] == value
end
end

アドバイスをいただいて今はこうしました。

def has_hash?(item, key, value)
item = item.deep_symbolize_keys
key = key.map &:to_sym
item.is_a?(Hash) && item.dig(*key) == value
end

これならシンボルでも文字列でもOKかなと。
rails依存にはなりますが。
名前も、has_value?は既に存在していたのでhas_hash?に変更してみました。

いけてますかね！？

[退会済みユーザー]

訂正　itemがハッシュじゃない場合deep_symbolize_keysでエラーになるので下記のようにしました。

def has_hash?(item, key, value)
key = key.map &:to_sym
if item.is_a?(Hash)
item = item.deep_symbolize_keys
item.dig(*key) == value
end
end