present? と blank? が嫌い - eagletmt's blog

params の中身のように入っているオブジェクトのクラスが事前に分からないものに対して空っぽい文字列の場合と存在しない場合を区別したくないときに限って blank? を使うのは分かるけど、 nil チェックをするために blank? を使ったり、配列が空かどうかをチェックしたいだけなのに blank? を使ったりすると、 blank? の挙動を正確に理解して nil と空配列を区別したくないから使っているのか、それとも nil がくるかどうか分からないので適当に防御的に blank? を使っているのか、 あるいは blank? しか知らないのかが読みとれずにめんどくさいと思うことがよくある。 かわりに empty? を使っていれば empty? を持っているオブジェクトは blank? を持っているオブジェクトより少ないので読み手に伝わる情報量が大きくなるし、 かわりに nil? を使っ

[gfx]

わかる。しかし !empty? の代わりにpresent? は使ってしまうな。

[umai_bow]

Railsあまり書かないけどこの手のやつの気持ちはわかる

[june29]

any と all はブロックなしでは使わないなあ。総じて「ちゃんと理解して正しく使え」という主張だと捉えました。当該の2つについては、なんとなくで使われることが多い、と。

[bopperjp]

やべ、any? の挙動シラナカッタ

[hoppie]

[].all?とかも間違いやすい

[tikkss]

分かる・・けど any? は知らずに使ってた。気を付けよう。

[yatmsu]

記事の内容とは関係無いけど、present?といえば某EC系サイト作ってた時に商品にプレゼントが混じってて、判定用のメソッドにpresent?を定義できなくてあああってなった。しょうがなくis_present?にした。

[willnet]

わかる

[wordi]

わかる、[false].any?がfalseなのもクソ

[castaneai]

rubyはこのような曖昧さがあるから、好きになれない

[mactkg]

ややこしい名前がたくさんあるのはどうなのか

[baronhorse]

わかる

[qsona]

100%同意した

[kamipo]

k0kubunさんが素のempty?は速いって言ってたのを信じてRails coreではempty?を使うようにしている

[razokulover]

たしかにわかる…

[tofu-kun]

わかる