1行で発狂するコード

そうそう、別に a に代入する（a に書き戻す）必要はないよね。

さっさとreturnしちゃうのが良いんじゃないですかね。

if(a >= 10) {
    return 10 * z;
}
if(a >= 5) {
    return 5 * z;
}
if(a >= 3) {
    return 3 * z;
}
if(a >= 2) {
    return 2 * z;
}
return z;

おもしろさの中に学び（カンマ演算子）もあって良い記事でした

個人的には、ネストする三項演算子は改行とインデントが必須だと思っています。
https://qiita.com/yuba/items/9832c26dc1498e0017bb

なので今回のコードもいい感じに整形したら意外とありかも？と思って書いてみましたが、やっぱりダメですね。（再代入とカンマ演算子がｋｓすぎる）

return (
  a = a >= 10 ? 10
    : a >= 5  ? 5
    : a >= 3  ? 3
    : a >= 2  ? 2
              : 1
), z * a;

@fujitanozomu さん
おっしゃる通り、私もすぐに返すでいいかなとも思いました。
私なら最後の return は、1倍であることを明示するために、以下のように書きたいですかね。書き忘れだと思っちゃいそうだし・・

return 1 * z;

@www-tacos さん
果敢なチャレンジ、ありがとうございます
そもそもreturn の後ろにカンマ演算子を置くのが、アンチパターンですね。
個人的には、インデントはVSCodeからフォーマッタに任せたいので三項演算子は1行で書き、記事にある通り、三項演算子のまとまりをカッコで括りたい派です。
それで見づらいようならif文を使ってしまうかも。

minifyされたコードではこのカンマの使い方は基本的なテクニックですね。知っているとプロダクションビルドされたコードが読みやすくなります。

returnなのにaに代入してる時点で「後ろのほうで参照するんだな。これは心して読まねば」と察するべきということでしょうね

私としては、左が大きい不等号つらい、不要な再代入は避けたい、一行が長い、括弧があったほうがいい、という印象は持ちましたが、それほどわかりにくいとは感じませんでした。（個人の感想です）

初見で苦なく理解できました。倍率の計算とかですかね。

条件演算子の部分は左から順に読めばわかるので高難易度とは思いませんでした。
難易度が高いと感じるのは下記のようなものです。

return a = a>=3 ? 10<=a ? 10 : 5>a ? 3 : 5 : a<2 ? 1 : 2, z*a;

それと a がローカル変数であれば

return z * (a >= 10 ? 略)

で済むはずなのに、わざわざ

return a = a >= 10 ? 略, z * a

としているので、return 後も a の値が活きるのかもしれないと想像していまして、もしそうなら、そのための代入が return 内で行われている点がひどいと感じます。

蛇足ですが、PHP は三項演算子の優先順位が違う（ことがある？）ので要注意です。

参考になれば幸いです。

このコンマ演算子はC由来のもので、式しか書けない場所に複数の処理を書いたりするのに使われてました

for(i=0, j=99; i<100; i++, j--) {
}

でもCの文法をベースにしているjavaやC#が採用しなかったように、問題点も認識されていました。
javascriptがそんな,演算子を採用したのは意図的なものと言ってよさそうなので、考え方の違いを感じられる箇所ですね。

@otn さん

returnなのにaに代入してる時点で「後ろのほうで参照するんだな。これは心して読まねば」と察するべきということでしょうね
return a=<aに入れたい値>,<値を返したい式>
は単純に
a=<aに入れたい値>;return <値を返したい式>
と置き換え可能で文字数も変わりません。
そのためこういうのは意図的に紛らわしくしてる難読化や嫌がらせと考えていいと思います。

「returnの右辺に代入文」の実用的な使われ方として思い浮かぶのは「返値を変数にも保存しておきたい」を一行で済ませるといったケースですね。

function GetAccessTime(){
  //最後に取得した時刻を覚えておいて、経過時間の計算などに使用する
  return lastAccessTime = new Date();
}

たぶんこのコードを書いた人は関数型プログラミング言語を書いてきた人なんでしょうね。

let f z = let a = 略 in z * a;;

みたいな処理をそのまま直訳して

function f(z) {
    return a = 略, z * a;
}

と書いてしまったような感じがします。関数型として見ればかなり自然な書き方なので。

仮にこれをレビューする立場になったら、
可読性/メンテナンス性が低いので冗長な記述に変えてください
くらいしか浮かばなかったけど（主観だから心のなかで反論されそう）
デバッグしにくいってのもあるのか

これをASTに変換するとどういう式になるんだ。冗長なコードとあまり変わらない？

これは...何をしたいかなんとなくわかるけど、もっとこう...あるだろうってなりますね。

let f z a =
    [ 10; 5; 3; 2 ]
    |> List.tryFind (fun t -> a >= t)
    |> Option.defaultValue 1
    |> fun t -> a * t * z

頭がまともならこうじゃねぇの？
あまりにもバカらしいので動くかどうか検証してないけど。
あれかな？　掛け算には順序があります！とか言ってる脳みそウニな小学校算数信奉者なんかな？
なんでreturnするだけでカンマがどうしたいう話になるんやろ？

return z * (
		(a >= 10) 
			? 10 
			: (a >= 5) 
				? 5 
				: (a >= 3) 
					? 3 
					: (a >= 2) 
						? 2 
						: 1
	);

https://qiita.com/jaque/items/b99ed9dce78cc64fa9d2#comment-df9cc431b87878f1d3dc
これ、酷いなぁ…。
こんなのにいいねが6もつくんや…。
世も末やなぁ…。

分かったようなこといってるけど、

さっさとreturnしちゃうのが良いんじゃないですかね。
https://qiita.com/jaque/items/b99ed9dce78cc64fa9d2#comment-b04a114fe519eb0f19e0

変数定義したら負けなんか？君らの知能だと…。
頭痛くなってくるわ…。

必要もないカンマ演算子とやらを使っただけで
「おー！興味深いですねぇー！！！」
とか盛り上がるから
世間一般からは『プログラマはキモい』って思われてるし、
全く信頼されてないんよなぁ…。

「エンジニア不足です」って言われてる“エンジニア”は、
そいうキモオタの事じゃないねんけどなぁ…。

はぁ！？

何を言ってんねん…。頭痛くなってくるわ…。

上の文章と下の文章で整合性取れてないやろ。
分けるべきが結論なら何に代入しようが関係ないやん。

ほんま、力抜けるわ…。