それ、もっとスマートに書けるよ - JavaScript コードをもっと短く、もっとシンプルに書く Tips 4選

2017年10月6日『まぼろしのJS勉強会 #1 「ナウいJSの書き方・考え方」』にて発表した資料です。 https://maboroshi.connpass.com/event/66502/ #mbrs_js_study

[R-H]

チルダと||に関しては、短く書けるけど絶望的に分かりづらい

[new3]

ブコメで総叩きになりそうな気がするけど、プログラマ歴1、2年ぐらいに誰もが通過する点かと。ここで一度リーダブルコード読んでおきたい。既読なら"リーダブル"とか"シンプル"について誤解しているので読み直したい

[megazalrock]

基本的にビット演算子はビット演算以外に使うべきではないと思うな。特に論理演算では1つのミスの影響が大きいので、可能な限り自明なコードにするべきだと思っている。場合によっては別途変数にぶち込むまである。

[Rishatang]

えぇ…これ大真面目に良いコードだと思っているなら絶対に一緒に働きたくない

[ku__ra__ge]

可読性を犠牲にして、コードを短くすることはスマートとは言わない

[shibamu]

リクルートマーケティングパートナーズってイマイチぽいという印象を持ってしまうな

[delphinus35]

書いた人のTwitter見て「あっ……」ってなった。

[umai_bow]

フロッピーディスクをまだ使っている現場もあるんですね

[hit-the-light1326]

中二ならぬ業ニ(業界二年目)コードという言葉を思いつきました

[kencharos]

知り合いにこれと全く同じテクニックを使う人がいて頭を抱えているのだけど、出典とかあるのかな？

[quwachy]

id:kibitaki それだと途中に出てきたときに誤判定するだろうが、変な書き方するからこんな馬鹿まで出てきたぞ。（やりたいのは-1=>falseそれ以外=>trueの変換つまりx!==-1）ブクマは善悪は兎も角あってはいる。お前のはバグ

[kibitaki]

痛い記法多い。指摘にはtwitterで顔赤い。~の件は本で殴りながら!使うように矯正 ／↑ビット演算out論理否定で明示て話でお前の言う判定式立ててないけど何が見えてるの？お薬ないの？

[digo]

こういう人がgoやったら発狂しそう

[tyru]

「プログラマのはしか」に罹ってるコードがいっぱいある…

[mt_hodaka]

イミュータビリティ考えてconstにしたい気持ちはわかる。そもそもスコープが広すぎることが問題な場合が多いけど

[uzimith]

navigator.userAgentの返り値がStringなんですがそれは

[joe-re]

個人的には~以外は普通に読みやすいと思うんだけど。。ブコメで叩かれてるけど、そんなに叩くような内容かなー。自分が外れ値なのかな。。

[tilfin]

includesメソッドを使う。例の分割代入なら const [foo, bar] = の方がシンプル。

[tuki0918]

ややこしそうな所は関数にしてもらって短くわかりやすく書いてもらえばそれでいいよ

[keim_at_Si]

ワンライナーにしても中途半端だなー。

[hinaloe]

バッドノウハウが(逆転して)詰め込まれてる……

[regularexception]

新卒が、同期に自慢している　くらいの内容

[teto2645]

ないわー。スマートを履き違えてるわー。少なくともこんな人がドヤってきたら冷静にはおられんわー。

[a-kuma3]

「バッドノウハウ」っつうんだぜ、そういうの／ UA のやつは、こうかな。const is_iOS = /iPhone|iPad|iPod/.test(navigator.userAgent);

[wbbrz]

ブクマのほうが勉強になりそう

[diveintounlimit]

スマート（量的な意味で）

[htnmiki]

情報を発信するとさらにいい情報が舞い込んでくるという素晴らしいエンジニアだ

[fuji_haruka]

チルダ演算子のアレはまあイディオムみたいなものだね。僕は使わないけど、時々見かけるから読めたほうがいい、みたいな。

[nice_and_easy]

ぼくのようなJS初学者でもサラサラ読めるコードがいいなあ。言語の特性を知り尽くした人しか読めないレベルの「シンプルさ」はただのオナニーだと思うよ

[otherworld]

判定方法より3回判定を繰り返してる事が辛い。Array.prototype.someなどを使ったほうがよさそう。/lodashなど使って1行forEachを挟めばいいかと/ constと3項演算子でよさそう / reduceの中で副作用を起こさない（accを破壊してる)

[m_m3zono]

ネタかな？

[ardarim]

スマート＝俺ちょっとJSには詳しいんだ、的な意味かな？ 可読性は悪い。

[cocoasynn]

タイトル読んだ時点でバチバチに叩かれそうと思ったけど案の定だった。

[nilab]

シンプルかなぁ・・・自分だったらわかりやすい関数を自前で用意するとかかな。「2017年10月6日『まぼろしのJS勉強会 #1 「ナウいJSの書き方・考え方」』にて発表した資料です」