回答編集履歴
2
誤字修正
answer
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
仕様書や、レビューチェックリストは存在しないのでしょうか？
|
|
2
2
|
|
|
3
3
|
あらかじめ定められているルールから逸脱しているのならともかくとして、
|
|
4
|
-
そうではなくて、細かな部分を
|
|
4
|
+
そうではなくて、細かな部分をインプリメンターに任せている環境であれば、よほどおかしい場合を除いて、
|
|
5
5
|
意見は言いますけど最終的にインプリメンターの意見を尊重します。
|
|
6
6
|
|
|
7
7
|
ルールの無いプログラムのスタイルやインタフェースの方針などの細かなところで自分の理想100％を目指してしまうと、何のためのレビューなのか目的を見失うきがします。
|
1
追記
answer
CHANGED
|
@@ -1,4 +1,10 @@
|
|
|
1
1
|
仕様書や、レビューチェックリストは存在しないのでしょうか？
|
|
2
2
|
|
|
3
3
|
あらかじめ定められているルールから逸脱しているのならともかくとして、
|
|
4
|
+
そうではなくて、細かな部分を印譜裏面他に任せている環境であれば、よほどおかしい場合を除いて、
|
|
4
|
-
|
|
5
|
+
意見は言いますけど最終的にインプリメンターの意見を尊重します。
|
|
6
|
+
|
|
7
|
+
ルールの無いプログラムのスタイルやインタフェースの方針などの細かなところで自分の理想100％を目指してしまうと、何のためのレビューなのか目的を見失うきがします。
|
|
8
|
+
自分のレビューは80点主義でやっています。
|
|
9
|
+
|
|
10
|
+
個人的には、ルール逸脱や誤っていたり、明らかに大きく非効率な場合を除いては、細かなコードに修正は求めません。もっと上流の部分のクラス構成や、エラー処理に過不足が無いか（本来はもっと前段で行われるべき事と思いますが、プロトタイピングっぽい手法の時には）の方を重点的に見ます。
|