rubocop / rubocop-jp

A place for RuboCop discussions in Japanese
55 stars 2 forks source link

`max{|x| x.size}` はもしかして`max_by{|x| x.size}` では? #1

Open pocke opened 7 years ago

pocke commented 7 years ago

max{|x| x.size}のような、ブロック内で<=>を使っていないmaxがあったら「もしかして max_by」と警告を出すCopがほしい

sort, min も同様

実装は比較的簡単だと思う。 注意点として、auto-correctは実装しない or デフォルトでは無効にする必要がある。何故ならば、これを自動的に直してしまうとコードが壊れる可能性があるため。