[ruby-dev:40772] [Feature #1269] warning when Kernel#p is used

チケット #1269 が更新されました。 (by Yusuke Endoh)

Target version 1.9.xにセット

遠藤です。

2009年3月11日21:31 Yusuke ENDOH <[e-mail]>:
> Kernel#p は非常に便利ですが、デバッグ後に p の呼び出しをすべて消せたか
> 不安になることがあります。
> p という名前は使うときは非常に便利ですが、残念ながら検索するには非常に
> 不便です。単語区切りで検索できないエディタだと検索は限りなく困難ですし、
> 単語区切りで検索できるエディタであっても、
>
>   - p という変数や attr を使っている場合
>   - rdoc 中のサンプルコードとして p を使っている場合
>   - html 断片の <p> や sprint の %p などが文字列リテラル中にある場合
>
> など、false positive がとても多いです。
> p を再定義して出力を止めれば p の呼び出しが残っても実害はなくせますが、
> 余計なコードが残るのは気持ち悪いですし、後でほかのデバッグをするときに
> 邪魔になる恐れもあります。
>
>
> そこで、$DEBUG = true の場合に p を関数呼び出ししている箇所があったら
> 警告することを提案します。


という提案を 1 年前にしましたが、$DEBUG というインターフェイス
にケチがついていました。


2009年3月11日21:50 U.Nakamura <[e-mail]>:
> アイデア自体は悪くないと思いますが、パーサレベルでチェックす
> るのであれば、syntax check (-c)などでひっかけるのがよいのでは
> ないかと思います。


[ruby-dev:40768] を見て思いついたのですが、ruby -c -v の時だけ
警告してくれるといいんじゃないかと思いました。


蛇足ですが、他にも簡単な lint を実装するなど面白そう。
ローカル変数は全部小文字で _ 区切りとか、クラス名は Camel Case
とか。一行 80 桁とか、1 メソッドの行数は 24 行までとか。異論は
ありそう。

-- 
Yusuke ENDOH <[e-mail]>
----------------------------------------
http://redmine.ruby-lang.org/issues/show/1269

----------------------------------------
http://redmine.ruby-lang.org