オフショア、 ニアショアチームとの 分散型コードレビューの 進め方
オフショアチーム、
主に実装コードに
個人的観測範囲で、 良く 観測される 事象
個人的観測範囲ですが、
- レビューが
ビックバンレビューに なりやすい
納品と
- レビューが
集中型レビューで レビュアーの 作業負荷が 高い
コードレビューで
- ラボ拠点が
違うと、 ノウハウの 共有が されない
組織、
分散型コードレビュー とは?
役立つ
レビューは
広く 分散させましょう。
一般的なよび方ではない
何故、 分散型コードレビューを したいのか?
コードレビューで
- 1個人に、
負荷が 集中することが あった。 - ノウハウ、
ナレッジが 他の メンバーに 貯まらない。
もう1つは
ただ、
デメリットと 対策
分散型コードレビュー には、
分散型コードレビュー 担当者の
デメリット | 対策 |
---|---|
実装の標準化の対応が難しい(指摘事項にバラツキが出る) | コーディング規約の整備、コーディングルール数を多くして、その中から指摘する(定期的にルールを増やす) |
レビュアーのスキルに差が出る | 並行して勉強会を実施する |
書いたり、見たりのコンテクストスイッチングによる集中が途切れる | 実装作業を担当していないメンバーが持ち回って実施する。※ちょっとこれが効果があるかは自信はない。 |
前提と して 実施すべきもの
ニアショア、
- レビュー目的を
チーム内で 定義する。
コードレビューを
「コードレビューが
終了した ときに、 どういう 状態に なっていれば 成功なのか」を 意識した ことは あるでしょうか。 答えが 「NO」ならば、 コードレビューの 実施方法その ものを 見直した 方が 良さそうです。
分散型コードレビューだと、
- ニアショア、
オフショアの チームと 品質に ついて 合意形成する
オフショアラボと
- 日本の
エンジニアの こだわりの 「品質」を 定義する - オフショア側の
エンジニアと 品質の 定義を 共有する - 定義を
仕組み 化する
日本の<wbr>エンジニアの<wbr>こだわりの<wbr>「品質」を<wbr>定義する<wbr>
は
コードレビューの フェーズに ついて
おそらく
開発側
- 静的解析ツール、
チェックリストに よる 開発者自身の セルフチェック - コードレビュー
- 静的解析ツール、
受入側
- コードレビュー、
仕様整合性確認の ための コードレビュー
- コードレビュー、
レビューの 流れの イメージ図
レビューの
図にしてみました。
レビュー観点、 実施方法
ニアショア、
Class構造
Class構造に
開発規模に
- 実施方法
INPUTは
Javaであれば、
コード俯瞰
書き上がった
実施方法
CPDやSonarQube等の
コードクローンを チェク、 酷い ものを リジェクトする。 PMD、
SpotBugs、 SonarQube等の 静的解析チェクに より、 バグの 偏りを 確認、 バグ数が 多い ソースを 重点的に 確認する。
SQL
SQLを
- 実施方法
チェックリスト、コーディング規約で 一般的な 考慮事項を 確認する。 サービスの 仕様理解者が、 SQLを 確認して 指摘する。 (こっちも 頻出事項は 規約なり、 チェックリストに して 文書化する )
トランザクション
トランザクションの
- 実施方法
シーケンス図を
差分コード
構成管理上の
境界値などの
- 実施方法
構成管理の
アドホック
観点設定を
いわゆる
非機能要件
サービス側で
一般的に
- アクセシビリティ
- パフォーマンス
- セキュリティ・監査
- ユーザービリティ
レビューを 下読みと 本レビューに 分ける
レビューの
回数を
参考
以下、
- コードレビューを
成功させる ために CTOが 考えるべき7つの ことー監修:高遠和也(株式会社LIG CTO) | FLEXY(フレキシー) - コードレビュー - Wikipedia
- 役立つ
コードレビュー 8つの ヒント | POSTD - チームで
コードを 書き始めた 後、 「どうやら レビューって やつを した 方が 良いらしい」くらいの 若手に 向けた 資料です。 - 【和訳】
コードレビューの ベストプラクティス - Qiita - コードレビューで
意識している こと(する 時/される 時) - Qiita - コードレビューを
効率化する ちょっとした 工夫 - Qiita - オフショアラボと
日本の 品質の 考え方の 違いと 仕組みづくり(コードレビュー編) - Sider Blog - オフショア委託先の
レビューの やり方を チェックしていますか? | Splus Software Co., Ltd. - コードレビューの
道具、 使っていますか? – IBM Developer - メンバーに
恨まれそうな 3つの コードレビュー施策を 徹底したら、 逆に メンバーが 爆速で 成長した 話 - Qiita
以上です。
コメント