20年Rubyを触ってきて初めて踏んだattr_readerのケツカンマ問題、あるいはdefの返り値がシンボルであることの問題 - joker1007’s diary

今日Rubyを書いていて、なんじゃこれと思った動作があった。 試しに以下のコードを実行してみて欲しい。 class Foo attr_reader :hoge, :fuga, def initialize(a, b) p a p b end end Foo.new 実行してみると分かるが、これには例外が出ない。 initializeで定義した必須引数はどうなったのか？ よくよく見るとattr_readerの引数の末尾に,がある。 つまり、このコードは分かりやすく書くと以下の様になる。 class Foo attr_reader(:hoge, :fuga, def initialize(a, b) p a p b end) end Foo.new このコードを更に分かり易く書くとこうなる。 class Foo def initialize(a, b) p a p b end attr_rea

[sora_h]

一度踏んでめちゃ時間失ったことある。複数行に渡る場合は括弧付けてる…。

[kobito19]

"複数行に渡る場合は括弧付けてる" ruby知らんがこれは複数行に渡すつもりがないわけだからその方針でも防げんのでは

[sakuro]

tailing commaじゃなくてtrailing comma (こういうときの形容詞は leading ⇔ trailing)

[mas-higa]

def がシンボル返すのは private def foo ... end とか書けるようにするためだからなぁ...

[tmtms]

rubocop が怒ってくれそうな気がする

[arikui1911]

まぁ ruby -w で実行すれば二重定義の warning が出るんすけどね。

[tkmkg8m]

怖いなーこれ

[razokulover]

サイゼの間違い探しっぽい

[yamadadadada2]

rubocopがアップを始めていそう

[devorgachem]

Rubocop で検知させたい

[sgo2]

この手の罠を踏んだ結果上手く動いてるパターンが稀に有るから困る。大体のコードは書く回数より読む回数の方が多いのだから、書く手間より読み解く手間が少ない方が良い。

[dekasasaki]

セミコロン「俺がいればこんなことには…」

[Kazumi007]

本題とはズレるけど、ケツカンマのもっとかっこいい呼び名はないのかなあ。tailing commaだから、末尾カンマ？

[threetea0407]

ブコメの「この手の負債が積み重なる」ってどういう状況なんだろ

[sevenspice]

ruby,railsはコスパ良くて最初は楽しい。でもこの手の負債もかなり積み重なるから運用が長期的になるほど辛くなる。まあ、お陰で食えてるんだけどね…

[koyancya]

分かってしまえば分かるけど、ハマるとドツボそう