2013-10-17

http://anond.hatelabo.jp/20131017223222

変な仕様になった原因探ってるつもりなんじゃないの。

<理想形>

Q. おかしいと思わなかったの?

A. 思いませんでした。なぜなら最初、聞いていた仕様では~~だったので、~~という実装にしました。

とすればレビュアー

「じゃあ仕様変更が吸収できてなかったんだね。

 仕様変更があった場合には、直接の修正範囲だけじゃなくて影響範囲も考えて修正しよう。」

とか、そんな感じになった。

みたいな?

記事への反応 -
  • コードレビューでクラス構成について指摘を受けた。 「なんでこういう設計にしたの?」って聞かれて、そのようにした理由と考えを答えた。 「おかしいと思わなかったの?」って聞...

    • 変な仕様になった原因探ってるつもりなんじゃないの。 <理想形> Q. おかしいと思わなかったの? A. 思いませんでした。なぜなら最初、聞いていた仕様では~~だったので、~~と...

      • レビュアー側は頭から、お前間違ってる、なんで間違えたことしてん?反省してんの? って意味で突っ込んでんだから、いくら論理の通った理由で「おかしいと思わなかった理由」を述...

    • 反面教師だな。そのレビューワ。 「おかしいと思わなかったの?」 とか聞いてどうするつもりだったんだよ。 自分がレビューするときは 普通のレビュー : クラス構成について指...

記事への反応(ブックマークコメント)

ログイン ユーザー登録
ようこそ ゲスト さん