【JavaScript】実数から整数への変換に parseInt() を使ってはいけない２つの理由🙅‍♀️ - Qiita

背景（誰のための記事？） JavaScriptプログラマのみなさまこんにちは。 最近のお仕事の傾向として、マイクロサービス化といいいますか、オブジェクト指向の延長といいますか、MVVM化といいますか、下回りは速度＆効率重視でC++で構築し、中間は保守（メンテ）＆書きやすさ＋ちょっぴり速度も重視で node.js（JavaScript）、上層のUIはなるべく広範囲で使えるように考慮して HTML＋CSS 、という３層構造 ・上層：HTML + CSS ・中層：node.js（JavaScript） ・下層：C++ が流行っていまして、今回の事件は中層の node.js スクリプトの部分で発生した問題でした。しかも、下層で生成された巨大なデータを中層で処理し、上層で表示しようとするとある条件で遅延が発生し、表示がカクつくという問題で、その「ある条件」が長期間に渡って特定できない、、、といった難

[megumin1]

浮動小数を整数にするのにparseInt()を使用する発想がなぜでてきたのかまったく理解できないです。"parse" の意味を理解していないってこと？ "parseInt( u16 / 256 )" には1秒で違和感をもてるようになるとよいですね。

[brain-box]

浮動小数以前に、整数の定義とは。https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/parseInt　ここにしっかりMath.floor替わりに使うなと書いてあるね。

[yorkfield]

入力が文字列じゃ無いのに何でparseが必要なんだって話だけど、別言語やってた人がint型へのキャストのイメージで使っちゃったのかなあ。

[delphinus35]

最初から最後まで、Math.floorの代わりにparseIntを使うようになった経緯がわからなかった。なぜそれで代用しようと思った……？

[nil0303]

「何でその発想になる？」という気持ちはわからんでもないが、これがレビューで回ってきたときに見落とさずすぐ「マズい」と気付けるだろうか？

[hylom]

parseInt()、使うことがほとんどない関数の1つな気がする（だいたいのケースはNumberコンストラクタで足りる）。基数の変換ってJavaScriptであまりやることなさそうだし。

[altar]

調査や検証の前に仕様を読め、MDNでいいから、という教訓

[eru01]

そもそもなんで実数→整数変換にparseInt()なのか理解できないんだが。その名の通り、文字列を受ける関数なんだからどう考えても異常でしょ

[e99toast]

parseIntをMath.floor代わりに使う人初めてみた。

[mnnn]

まあプログラミングに対するメンタルモデルができてないとこういう突飛なことしてしまいがちではある

[strawberryhunter]

こんなプログラム書く人がいるのか。世の中は広いな。

[yamami78651]

えー、なんかparseInt()が可哀想…使い方が間違ってるのに文句言われるとか。。

[xlc]

parseInt() 使ってるヤツを初めて見た。文字列→数値 なら + を使うし、切り捨て/切り上げなら Mathにある関数を使えばよろしい。

[newnakashima]

parseIntで小数点丸めてる人いたなぁ。

[emergent]

「parseIntに実数を渡せてしまう」ことに驚いた。加えて、ブコメひでぇなって思った。

[rryu]

JSはゼロ方向への丸めがIEで使えないMath.trunc()しかなくてそれでperseInt()が無難という判断だと思うのだが、ビット演算すると整数になるという方がまだ無難な感じがする。

[king-zessan]

私もparseInt()を使う理由が分からなかったので、ちゃんと「Math.floor()代わりに使うな」と書いているMDNは偉い、と思った。

[trace22]

普通の人はハマらん穴かな。まあ、貴重な経験かも

[ardarim]

なぜfloorでなくparseIntが浮かんだのか、そこから理解が出来ないのだが…。

[naopr]

なぜparseIntを使った…というのはありつつ面白い知見だった

[tettekete37564]

この人の言う下層の意味が分からないけどマイクロサービスとか言ってるってことはサーバーサイドの事？なんでC++なんか流行ってるんだろう。すんげー面倒くさいと思うんだが

[dominion525]

文字列を引数に取る関数に実数渡すのやめて…という気持ちしか。自動で型変換してくれるとはいえ、もう少し型について考えてあげてほしいし、なんで「問題のなさそうな」になるのよ…？

[dagama]

問題のなさそうなって何？問題しか無いだろ？JavaScript畑のやつって型をナメてるやつ多くない？

[UhoNiceGuy]

これはハマる//正直、配列のindexは明示的にintにキャストしたいのはある。a[2.0]とかなんか気持ち悪い。あと、暗黙のキャスト嫌い

[stabucky]

これは知らなかった。「parseInt」は「解析して整数化する」という意味でしょう。間違うよ。

[greenbow]

さんざん突っ込まれてるけど parseInt 使うのは珍しいのでは。12.3 → "12.3" で . までが有効な整数文字列となるから 12 になるのか。なるほど

[sukoyakacha]

このレベルの記事は大量にあるのにたまに目に付いたものだけ通りすがりに殴っていくスタンスやべーっす

[gengesa]

こういう間違いを防いでくれるからTypeScriptの方が初心者に優しい

[el-condor]

寧ろこの方法を使おうと思った理由が知りたい。

[knjname]

これは使い方をそもそも間違ってるけど atoi な用途でも parseInt そもそも使ってないなあ (+str が多い)

[jgoamakf]

parseIntの引数に数値を指定すると、1回文字列に変換してる？1/9 追記）parseInt関数が自ら変換すると書いてあった。 https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/parseInt#%E5%BC%95%E6%95%B0