setIntervalの第2引数に気を付けろというだけの話

この記事はtraP Advent Calendar 2023 14日目の記事です。 はじめに こんにちは。19Bの@kegraという者です。いつもゲーム開発とかWeb開発あたりをやってます。 今日はJavaScript(TypeScript)で最近やらかした割と初歩的なデカいミスを紹介したいと思います。 setInterval()による定期ポーリング 僕は今あるシステムを作っており、そのシステムではクライアントサイドが定期的にサーバーのAPIを叩いて情報を取ってくる感じになっています。いわゆるポーリングです。 クライアントサイドはWebフロントエンドで作っており、僕はこの実装でsetInterval()を使っていました。 setInterval(() => { updateData(); }, 5000); // 5秒おきにポーリング そこで本当にしょうもないのですが、第2引数を書き忘れて

[nmcli]

setInterval って名前なのにインターバルを省略できるの知らなかった。省略したことないわ。笑った。

[Fushihara]

safeSetIntervalメソッドを使い忘れる未来が確実だから、lintのカスタム定義を作るのが手っ取り早そう。typescriptのd.ts で上書き出来るか確認したけど、既存の定義を無効化は出来無さそう

[takc923]

MDNの英語版には省略された時の挙動が書いてあった

[nakag0711]

今は知らないが昔はsetIntervalは画面更新しながら全力で処理を回すときの定番方法だったのでこういう仕様になっているものと思われる

[fukken]

こういうの、設計時に何秒間隔でpollingするか決めない？決まってたらそうそう忘れたりしないと思うのだが。

[at_yasu]

「数ms間隔でサーバを高速連打する最悪のフロントエンドが爆誕」草

[ustam]

え〜デプロイ前に気づくでしょう、さすがに…。