良いソフトウェアとコードレビュー / Good software and code review

Scala + Caliban で作るGraphQL バックエンド / Making GraphQL Backend with Scala + Caliban

[nemoba]

レビュアーのベストプラクティスを見りゃ分かるんだけど、コードレビューはチーム形成に大切で、品質はその上で成り立つ認識が大切。

[jintrick]

黒背景に薄暗い赤って何。コードの可読性が物理的に低いwww

[Yoshiya]

コードの可読性の説明でC言語を使っていましたが、30年前ならともかく今どきセミコロン区切りで複数の変数宣言したり、ループ処理に使う変数を予め変数宣言の場所で初期化したりしますか？悪い例のコードがひどすぎる

[jojo800]

baseValue=3ってなんか言ってるようでなんも言ってない感ある

[syu-m-5151]

よくまとまっていた！ChatGPTに全てやって欲しい

[hiroomi]

SCAと見立てたら大枠だけど、お手軽。

[ghostbass]

スコープに応じた命名、ってコンセプトは良いよね。 total とか sum でよくない？とか baseValue の意味がよくわからないとかは置いておいて。

[ndanna]

後で共有

[naoya2k]

重箱の隅かもしれないけどさ、自分の部下からこの例のようなリファクタリングがPRされてきたら多分リジェクトするよ。前と後でコードの挙動が変わってるし、足し算するだけのために関数を作ったりしてるのはキモい

[turanukimaru]

誰でも良いことを言えるテーマなのに何故か具体的な話には何一つ賛同できない…。レビュアーが理解しようとしちゃダメだろ。理解が必要だなって思ったら「ここ別情報が無いと分かりにくいですね」って言わなきゃ。