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

**仕様書無しさん** · 2019/03/24(日) 00:31:33.20

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

**仕様書無しさん** · 2019/03/24(日) 00:56:57.76

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

**仕様書無しさん** · 2019/03/24(日) 01:16:17.61

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

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

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

**仕様書無しさん** · 2019/03/24(日) 01:36:08.11

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

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

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

**仕様書無しさん** · 2019/03/24(日) 01:42:02.03

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

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

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

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

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

**仕様書無しさん** · 2019/03/24(日) 02:10:57.99

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

**仕様書無しさん** · 2019/03/24(日) 17:24:14.05

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

**仕様書無しさん** · 2019/03/24(日) 20:51:45.73

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

**仕様書無しさん** · 2019/03/24(日) 22:20:10.48

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

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

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

**仕様書無しさん** · 2019/03/25(月) 07:23:09.31

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

**仕様書無しさん** · 2019/03/25(月) 08:00:48.13

>>10
真顔で

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

って言われた

**仕様書無しさん** · 2019/03/25(月) 12:29:58.94

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

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

**仕様書無しさん** · 2019/03/25(月) 22:59:39.19

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

とちゃんと聞かないと

**仕様書無しさん** · 2019/03/26(火) 00:18:16.55

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

**仕様書無しさん** · 2019/03/27(水) 17:59:16.62

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

**仕様書無しさん** · 2019/03/29(金) 22:23:10.08

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;
}
}

**仕様書無しさん** · 2019/03/29(金) 22:28:11.07

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

**仕様書無しさん** · 2019/03/29(金) 22:44:07.76

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

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

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

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

**仕様書無しさん** · 2019/03/29(金) 22:49:01.49

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

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

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

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

**仕様書無しさん** · 2019/03/30(土) 01:16:18.96

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

**仕様書無しさん** · 2019/09/15(日) 02:33:13.09

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

**仕様書無しさん** · 2020/01/29(水) 20:31:30.43

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