コードレビューって士道不覚悟と一緒よな

[1 仕様書無しさん 2019/03/24(日) 00:31:33.20]

要するにレビュアーの気分とか都合

[2 仕様書無しさん 2019/03/24(日) 00:56:57.76]

チェックリストのないレビューはゴミ

[3 仕様書無しさん 2019/03/24(日) 01:16:17.61]

そもそもコードレビューをなぜやるのかわかっていない現場だとそうなる

コードレビューで一番大事なのはチェックリスト
これをお客にソースコードと一緒に出すことで
このチェックリストに書いてある品質を保証する
という証左にする

これがコードレビューを行うことで
ソースコードの品質を上げる仕組み
もちろんコードレビューのチェックリストの内容にはお客との同意が必要

[4 仕様書無しさん 2019/03/24(日) 01:36:08.11]

この仕組みがわかっていれば

・レビューのチェックリストがない
→レビューをやる前と後で何が違うんですか？
→お客にどうやって説明する気ですか？
→何をもって品質と定義するんですか？
→この間、逆のこと言ってませんでした？（笑）

等々そもそもチェックリストのないコードレビューなど社会人がやることなど絶対にないことにすぐ気がつく

[5 仕様書無しさん 2019/03/24(日) 01:42:02.03]

ではチェックリストとは何か？

これは言い換えると
コードレビューの品質と言うことになる
気分で指摘項目の異なるレビューなんて誰も金なんか出さない

このソースコードはこのチェックリストの観点での品質を保証しています

これがチェックリストの本来の使い方

なのでこれが存在しないコードレビューはそもそも何をやっているのか理解不能
ということになる

[6 仕様書無しさん 2019/03/24(日) 02:10:57.99]

プログラム板にレビューのスレがｗ

[7 仕様書無しさん 2019/03/24(日) 17:24:14.05]

競技プログラマの書いたコードをレビューしたときの気持ちを５７５で表せ

[8 仕様書無しさん 2019/03/24(日) 20:51:45.73]

競プロのコードはあれは酷いな
読めやしない
何をやっているのか意味不明すぎる

[9 仕様書無しさん 2019/03/24(日) 22:20:10.48]

未経験でITプログラマになった新米PGの情報交換の場はここでもできるよ

https://medaka.5ch.net/test/read.cgi/job/1553427446/l50

ここで情報共有してみな　
未経験で入社したやつらの生の声聞けるよ

[10 仕様書無しさん 2019/03/25(月) 07:23:09.31]

業務システムで外部仕様にUML書くとよく解らないと言ってくるお客さんにチェックリストの合意とか出来るのかな