Use Hash#fetch instead of Hash#[]

[sheerun]

So the error is visible immediately and not "undefined method ... for nil:NilClass"

[jandudulski]

Do you propose using fetch in absolutely every case?

[teamon]

Same as with any other method call, hash#[] is no different, this is ruby, you can get nil enywhere, using fetch gives nothing really.

{:a => nil}.fetch(:a).foo # => still NME

[szajbus]

@teamon Consider another use case:

{:a => nil}.fetch(:b).foo # => IndexError: key not found
{:a => nil}[:b].foo # => NoMethodError

I think it makes sense in some situations as it can produce better backtrace. I would put it in guidelines with annotation "Consider using fetch instead of [](key)", not as a default.

[sheerun]

@jandudulski Of course not...

[teamon]

"Consider" sounds good :)

[Ostrzy]

:+1: for "Consider..."