1行で発狂するコード - Qiita

人は1行の記述で死ぬこともある 公開からコンスタントに閲覧をいただいている以下の記事。需要がありそうなので、もう1つ記事を書いてみようと思う。 題材は前回同様のJavaScriptのクソコードではあるが、前回は冗長を極めたクソコードであるのに対し、今回はたった1行である。 1行なのに、なんと示唆深いに富む(決してほめていない)コードだろうか。 不可思議なコード では、実際に見てみよう。 これを見て、何が return で返されるかわかります？ 代入演算子 最初見たとき、代入文 a = ××× の評価値って何になるのかな？ a が return されるのかなと思いました。 割り当て操作は、割り当てられた値として評価されます。 JavaScriptの仕様を確認し、「return a = ××× は、a がローカル変数である限り、 return ××× ということか、冗長だな」と判断しました。

[nekoline]

自分は一目見てウッってなったけど、肯定派が多いんだね。確かによく見たら読めるけど、そんな事で集中力を使わせないで欲しい

[hryord]

自分が読めるかではなく一般的なプログラマーが読めるかが大事なのでは。将来そのコードを誰が触るのかわからないのだからその誰かが理解できるか想像しろ

[puruhime]

()のない三項演算子のネストは言語によっては結果が変わるので、ネストするなら()は必須だと思うよ派

[turanukimaru]

私がレビューするとしたら「なぜ一行で書く必要があってカンマ演算子も必要かをコメントに残してね☆」とコメントして差し戻すと思う。これがOKな人たちが沢山いるみたいだけど、貴方達ならどうコメントする？

[knjname]

特にカンマ演算子が良いコードとは思わないが普通に読めた

[kakusuke07]

色々言及してるけど、要はreturnでの再代入とreturn中のカンマの挙動というニッチな仕様を使っていて読みにくいという主張のよう。 つまり、return z * (a > 10 ? … のような書き方なら普通に読み下せて文句は言われないのでは

[takeda25]

一瞬でも理解に時間がかかったらAIに任せるのが吉 https://chat.openai.com/share/1aaf3863-ebcd-4fdb-bf49-544b5da3c3c1

[eggplantte]

奇妙奇天烈なショートコーディングの世界に比べたらこれぐらいは序の口だなぁ

[xlc]

JavaScriptのアロー関数は1文か否かで意味が変わるのでカンマ演算子を使うことはあるかもしれん。三項演算子はインデントを使えばネストはむしろ推奨される行為。→　https://x.com/koba0367/status/1765838133650977234

[saikyo_tongaricorn]

帰宅直前に急ぎの不具合報告上がって来て、コード眺めてたらこれが出てきても完璧に読み下せる奴だけ「俺読めるわ〜」って言え。俺は嫌だ

[vndn]

aに対する処理に名前はつけられないのかしらん。

[adgjmptw4410]

Qiitaやばいコメントするやつ湧いてて草

[kfujii]

読めたかどうかとかどうでも良くて、本番コードとして許容できるか、だろーよ。読めた報告なんなの

[blueeyedpenguin]

別の処理のついでにaを勝手に書き換えてしまうのが一番ひどい

[naka_yu_a]

コメントがあれば消極的肯定ですかね……

[lp008962]

三項演算子のネストでFizzBuzz書いてたら、シンプルな else if 記法って思った

[gengesa]

「if を羅列すりゃええやん」とレビューすれば良いだけの話でいちいち発狂するなと言いたい

[shidho]

ギリ読めた。個人的には発狂するコードといえば、(1x$number) =~ m/^(?:1?|(11+?)(?>\1+))$/; のほうがおすすめ。何をするコードかは知っているが、どうしてそう動くのか全く理解できていない。

[otation]

val SAN = 0

[dot]

読みにくい目が滑る系のコーディングは不具合の温床になりがちなので、程度差はあれどできるだけ修正した方がいい。大昔はハードウェアリソースの方が優先されたけど今はヒューマンリソースの方が重要。

[mohno]

「rm -rf /」じゃなかった。/「何が return で返されるかわかります？」←C言語だったら分かるし、JavaScriptも同じみたいだね。/オススメはされないだろうけど、カッコなしで済むような優先順位/評価順序になってるんだよ。

[softstone]

普通に読める。自分は今ならこう書く `([10,5,3,2].find(x => a >= x) || 1) * z;` けど正直ternaryのほうが俺には読みやすい。テーブルインデントならなおよし。誰にとっても読みやすいコードって幻想だと思うよ。/puruhime 滅びた筈だ

[ducky19999]

こんなんメンテできんわ

[naari_3]

もっと良いコードを書いていたいね / jsxでもこんなコード必要ないよ

[tomono-blog]

カンマ演算子の存在を知った

[shinagaki]

jsxでは必要だし、慣れ

[hubtoyo]

「(マイナーな)言語仕様を正しく知らなかったのでちゃんと読めなかった」のは書き手読み手双方に責任がありそうで、10:0にはなりそうにない。

[Shinwiki]

書いてるときは俺すげーってなってるけど次の日見たらなんでこんな風にしたのかって思うので、つまりこれはラブレター

[Helfard]

AIすげー。

[auto_chan]

がはは！そんなものExcelのIF連鎖数式地獄で慣れたわ！いや慣れてないわ！むりだわ！！！４項以上になったら恥を忍んでIF文とコメントかいてちくり～。

[caffephilia]

三項演算子のネストは適切な改行（とインデント）を入れてほしいと思っているけど、JavaScript でだけ改行時にセミコロンが補完される時の仕様を正しく把握している自信がないので私は少し見やすく書き直すかも

[Wafer]

処理の順序を示したコードを見ると、１行で書きたくなるの分かるわってなった。両者の中間ぐらいの良い書き方ないものかねぇ

[todays_mitsui]

カンマ演算子の左オペランドの範囲が不明確 (3パターンくらい思い浮かぶ) だからこのシチュエーションでカンマ演算子使うのは無しだな、と思った。読める読めないではなく、読み間違う事が困難なコードが理想。

[pwatermark]

一行にすることによるメリットが分からん メンテする気が起きないからifで書き直させるだろうなあ

[UhoNiceGuy]

読めりゃいいって問題じゃない。21世紀にwhile(*s++ = *t++);みたいなコードはやめれ。一瞥して動作が把握できないと//最初は一段の三項演算子だったが少しずつ増えていったものと予想。リファクタリングするべきでは

[kabochatori]

AIに修正させた中で面白かったアイデア。return z * Math.max(1, Math.min(2, Math.min(3, Math.min(5, Math.min(10, a))))));　　return z * ([1, 2, 3, 5, 10].find(i => a >= i) || 1);

[yamuchagold]

良いコードとは思わないが普通に読める。あり得る、というレベルかな。この世の中にはこれ以上のクソコードはいくらでも転がってる。

[JULY]

割と普通に理解できた。まぁ、ロジックとしてさほど重要でない部分をワンライナーで書く事で、他のロジックを見やすくする、という事があるので、コメント付きなら否定はしない。カッコは欲しいけど。