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

1 ：仕様書無しさん：2019/03/24(日) 00:31:33.20 .net

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

2 ：仕様書無しさん：2019/03/24(日) 00:56:57.76 .net

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

3 ：仕様書無しさん：2019/03/24(日) 01:16:17.61 .net

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

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

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

4 ：仕様書無しさん：2019/03/24(日) 01:36:08.11 .net

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

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

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

5 ：仕様書無しさん：2019/03/24(日) 01:42:02.03 .net

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

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

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

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

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

6 ：仕様書無しさん：2019/03/24(日) 02:10:57.99 .net

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

7 ：仕様書無しさん：2019/03/24(日) 17:24:14.05 .net

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

8 ：仕様書無しさん：2019/03/24(日) 20:51:45.73 .net

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

9 ：仕様書無しさん：2019/03/24(日) 22:20:10.48 .net

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

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

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

10 ：仕様書無しさん：2019/03/25(月) 07:23:09.31 .net

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

11 ：仕様書無しさん：2019/03/25(月) 08:00:48.13 .net

>>10
真顔で

お金かかるならやらなくていいよ

って言われた

12 ：仕様書無しさん：2019/03/25(月) 12:29:58.94 .net

資格が必要のない業界だから、昨日までコンビニバイトをしていた人間でも
見よう見まねでいきなりコードを書くことができる
無資格の人間が手術をするようなもの
そんな品質保証がゼロの世界だからな

せめて有資格制度にでもなればいいのだが
なぜかこの業界は無資格の人間が手術をしまくっている悲惨な状況

13 ：仕様書無しさん：2019/03/25(月) 22:59:39.19 .net

>>11
じゃあ、品質は保証しませんけどそれでいいですか?

とちゃんと聞かないと

14 ：仕様書無しさん：2019/03/26(火) 00:18:16.55 .net

>>13
そうやって先手を打っておくことでソースコードの体裁に関してツッコませるのを防ぐことができる

15 ：仕様書無しさん：2019/03/27(水) 17:59:16.62 .net

>>12
応用情報以上を持ってない人間は商用プログラムの作成を禁止とするだけで質は向上する

16 ：仕様書無しさん：2019/03/29(金) 22:23:10.08 .net

for (j = 0; j < h; j++) {
float w1in = j * (N / m);
float w2in = w1in + N_2;
w1 = cos(w1in / (FFTN / (2 * PI)));
w2 = sin(w2in / (FFTN / (2 * PI)));
for (k = j; k < N; k += m) {
kp = k + h;
s1 = fReal[kp] * w1 - fImag[kp] * w2;
s2 = fReal[kp] * w2 + fImag[kp] * w1;
t1 = fReal[k] + s1;
fReal[kp] = fReal[k] - s1;
fReal[k] = t1;
t2 = fImag[k] + s2;
fImag[kp] = fImag[k] - s2;
fImag[k] = t2;
}
}

17 ：仕様書無しさん：2019/03/29(金) 22:28:11.07 .net

すでにアウトプットされたプログラムの品質保証を
人の資質に求めるＩＴ従業者って

18 ：仕様書無しさん：2019/03/29(金) 22:44:07.76 .net

コードレビューのチェックリストなんて作れないよｗ

コードレビューをやる前提として
lintツールや静的解析ツールの導入が必須

lintツールや静的解析ツールで自動的に指摘される部分は
コードレビューの対象ですら無い。やり直し。

lintツールや静的解析ツールで見つけられないようなものを
コードレビューで見つけるわけで、そういうものは
チェックリストは作れない

19 ：仕様書無しさん：2019/03/29(金) 22:49:01.49 .net

コードレビューの品質と、製品の品質を勘違いしているバカが多い。
どんなにコードが汚くても、バグがなければ製品の品質は高くなる。

だがメンテナンス性は低い。製品の品質を高くすること
つまりバグを減らすことがコードレビューの目的じゃない。
製品開発のコストを下げるのがコードレビューの目的

だからコードレビューの結果を客に報告するなんて意味がないんだ
客は製品をいくらで売ってくれるかは興味があるが
製品を作るのにいくら掛かるかは興味ない

製品開発のコストを安くしたとき、それに合わせて製品の価格まで
安くするのはビジネス的には愚か者。単に儲け減らしているだけ

20 ：仕様書無しさん：2019/03/30(土) 01:16:18.96 .net

最近の規約と文法とフレームワークでガチガチに守られたプログラムは昔ほどレビューいらない

21 ：仕様書無しさん：2019/09/15(日) 02:33:13.09 .net

普段経験と感覚ででレビューしてるから、
誰か俺のためにチェックリスト作ってくれ

22 ：仕様書無しさん：2020/01/29(水) 20:31:30 .net

レビューしてもらいに行ったら
ここってみんなと合わせたほうがいいかもねーって
なんとなくペンディングだか修正だか結局結論がわかんない状態で
ばんばん先に行く
おかげでどこも着手できない