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

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

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

[13 仕様書無しさん 2019/03/25(月) 22:59:39.19]

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

とちゃんと聞かないと

[14 仕様書無しさん 2019/03/26(火) 00:18:16.55]

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

[15 仕様書無しさん 2019/03/27(水) 17:59:16.62]

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

[16 仕様書無しさん 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;
}
}

[17 仕様書無しさん 2019/03/29(金) 22:28:11.07]

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

[18 仕様書無しさん 2019/03/29(金) 22:44:07.76]

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

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

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

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

[19 仕様書無しさん 2019/03/29(金) 22:49:01.49]

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

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

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

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

[20 仕様書無しさん 2019/03/30(土) 01:16:18.96]

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

[21 仕様書無しさん 2019/09/15(日) 02:33:13.09]

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

[22 仕様書無しさん 2020/01/29(水) 20:31:30.43]

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