【JavaScript】関数名をclose()にするのは気をつけろ

単なる自分のやらかしの共有です。この記事は事実をもとにしたフィクションです。 消し忘れたclose()関数 あるときモーダルコンポーネントを閉じる、close()関数というのを作りました。後日、リファクタリングをするときこの呼び出されているclose()関数がとあるコンポーネントから取り除かれることになりました。しかし誤って1つ削除し忘れてしまい、close()関数が取り残されてしまいます。 このとき、import文なども取り除かれたのですが、何故か未定義エラーなどは出ずに見過ごされてしまいました。（察しの良い人はピンとくるかも） 何故か消えるウィンドウ そんな中、消し忘れたコンポーネントを含む画面で特定の手順を踏むと開いたブラウザウィンドウが閉じてしまう現象が確認されるようになりました。 どうやら発生源は先程のリファクタリング作業らしい… window.closeというJavaScrip

[shingo-sasaki-0529]

そんなこと起こらんやろって思っておもむろに適当なコンポーネントに close() 呼び出しいれたらエディタは何の警告も出さない動かしたら普通にタブ閉じてウケた。

[igrep]

shadowingの警告がないと気づきにくいよね

[zyzy]

JSって名前空間名からのパスをつけて呼ぶ仕組みないん？　他の言語ならシンプル命名する場合、普通はそれで回避するんだが

[tomopo]

そらそうやろとしか

[strawberryhunter]

そうか。同一か親のスコープにはthisとか必要ないからグローバルスコープと区別が付かないもんね。

[yarumato]

“消し忘れたclose関数呼び出しは、JavaScriptの組込み関数window.closeとして解釈され、ブラウザウィンドウを閉じる関数へと転生。予防法：closeやopenのような1単語の命名をやめよう。closeModalという関数名なら役割が明確”

[rryu]

即時関数を作ればその中では割とやり方だというのは何か良くなさそうという事例がこれか。