Rubyでrescueしたらグローバルなオブジェクトを破壊してたよって話 - ANDPAD Tech Blog

こんにちは、ydahです。 2021年の12月から株式会社アンドパッドでソフトウェアエンジニアとしてANDPAD検査の開発に携わっています。 昨年、息子が生まれてから夜更かしすることがなくなり、早朝に起きては軽くジョギングをしてから、OSSプログラミングにいそしむのが朝のルーティンになった今日このごろです。 さて本稿では、Rubyの例外処理を眺めていたらrescueがグローバルなオブジェクトを破壊するケースがあったんですよという話と、その対策について話したいと思います。 発生していたケースについて 突然ではありますが以下のコードをご覧ください。 この中にグローバルなオブジェクトを壊してしまうrescueがいます。 # 1 begin raise 'foo' rescue ArgumentError end # 2 begin raise 'foo' rescue => ArgumentEr

[ydah]

書きました

[e_denker]

気をつけましょう！じゃなくて「もうrubocopに取り込み済みです」なのがカッコいい。

[sue445]

rubocopにPR送って取り込まれるところまで持っていくの偉業すぎる

[gfx]

これRuby側がコンパイル時に弾いてもいいケースではあると思うな。

[el-condor]

確かにrubyの文法的にこれ成立するのか…盲点すぎる

[rryu]

定数なのにrescueなら上書きできるのが驚き。エラーにしないのは何か理由があるのだろうか。

[khtokage]

rescue関係なくRubyの定数はwarningが出るだけで上書き可能なんですよ……　さすがに筋が悪いと思うんだけど、一度変わってからそのまま https://bit.ly/3pINWWd

[nil0303]

こいつ上書きできたんか...。 こんなん意図してやろうとする人なんていないんだから警告じゃなくて弾いて欲しいレベル。

[shiba_yu36]

自動でチェックされて助かる

[fjwr38]

知らんかった。例外のテスト書いてたら気づけないかな？

[zonu_exe]

冷静に見ればわかるんだけど、たしかに雰囲気で書いたりレビューで見逃しちゃいそう

[yutag]

[例外処理]

[mandel59]

Rubyってそうなんだ…

[katsyoshi]

これは……

[tricknotes]

素晴らしい 🎉 "ご紹介した対策で皆様のプロジェクトでも同様に検知することができますので、是非ご活用いただければ幸いです"

[shingo-sasaki-0529]

説明されるとそれはそうって感じだけど、クイズで普通に答えられなかったからレビューで見ても余裕で見落とす自信あるわ