タグ

ブックマーク / moznion.hatenablog.com (2)

  • コードレビューの高まった言葉 - 職質アンチパターン

    ブログ間違った,普段こういう事はこっちに書いてます. http://moznion.hatenadiary.com 最近自分がコードレビューで使いがち,あるいは表立って使ってないんだけど内心評す時に使う言葉が色々とあり,まとめてみることとした.参考にしない方が良いと思う. 左は言葉,右は説明. 屈強 - コードが力強い時に使う.例えば長い一枚スクリプトとか,コメントが一切ないバッチ処理とか.やや批判的な意味合いで使うことが多い. マッチョ - 屈強と同じ文脈で使いがち 屈強だけどしなやか - 屈強だけどしなやかな時に使う.好意的な屈強さと言える. モノリス - 長大なトランザクションスクリプト見た時とかに使う.やや批判的. 言い訳ないですか - 後で直していくぞ! というメンタルの時に書かれたコードのコメントが案外少ない時に使う言葉.言い訳は無いよりあった方が良い.実際には「もうちょっと言

    コードレビューの高まった言葉 - 職質アンチパターン
  • レビュアーとレビュイーの関係に関して - 職質アンチパターン

    感じていた違和感の正体がわかった. レビューにおいて,レビュアーとレビュイーの関係には上も下もない. レビューという場では,両者の立場は対等でなければならない.さもなければ,「このレビューおかしい気がするけれど,あの人は立場が上だから指摘しにくい」だとか,「相手は格下だから適当にレビューしても良い」だとか,そういう良くない雰囲気が形成されてしまう. 確かに,レビュアーというポジションはチームの技術力が高い人やそれに伴ってパワーのある人が担うケースが多いと思う.レビュイーはそれに萎縮してしまいがちというか,僕もその1人なんだけれど,そういうのは実際のところ保身でしかなくて,下手に意見言うとレビュアーとの関係悪くなりそう,みたいな卑屈な理屈に基づく駄目な萎縮な感じがする.こういう良くないところはちゃんと直して,健全化していかなければまずい. あとレビューされた内容は素直に受け入れるべきだと思う

    レビュアーとレビュイーの関係に関して - 職質アンチパターン
  • 1