[ETロボコン2019]ETロボコンで学ぶレビュー技法 / etrobochu4-20191007

Slide 1

Slide 1 text

もう、審査で減点されない︕ ETロボコンで学ぶレビュー技法 2015年10⽉7⽇ ETロボコン中四国地区実⾏委員会実⾏委員河内⼀弘

Slide 2

Slide 2 text

2 はじめにこの資料は、2015年10⽉3⽇(⼟)に⾏われた、「ETロボコン2015 中四国地区秋の独⾃勉強会」で⾏った演習内容を元に再構成したものです。会場︓福⼭⼤学宮地茂記念館（広島県福⼭市）

Slide 3

Slide 3 text

ETロボコンといえば、モデル審査ですが・・・ 3 モデルのレビュー、できてますか︖

Slide 4

Slide 4 text

4 今年のモデル審査、頑張っているのに、少しのミスで評価を下げたチームがチラホラ。・・・モッタイナイ。

Slide 5

Slide 5 text

5 レビューといえば、「いくら⾒ても問題を⾒つけられない…」「うちは初⼼者ばかりだし…」「やってもあまり効果がないんじゃない︖」と感じている⼈も多いのではないでしょうか。

Slide 6

Slide 6 text

6 でも、こうした問題は、レビューのやり⽅を⼯夫すれば解決できます︕ ETロボコンに限らず、仕事や学業でも役⽴つ（かもしれない）レビューを体験してみましょう︕

Slide 7

Slide 7 text

7 レビューの種類マネジメントレビュー管理者が、開発状況の把握や承認のために⾏うオーディット法令や規則に沿っているかチェックするために⾏うピアレビュー成果物に問題がないかチェックするために⾏う

Slide 8

Slide 8 text

8 今回取り扱うレビューマネジメントレビュー管理者が、開発状況の把握や承認のために⾏うオーディット法令や規則に沿っているかチェックするために⾏うピアレビュー成果物に問題がないかチェックするために⾏う

Slide 9

Slide 9 text

9 今回取り扱うレビューピアレビュー成果物に問題がないかチェックするために⾏うウォークスルー作成者主導のカジュアルなチェックテクニカルレビュー技術リーダー主導の成果物チェック・経験ベース・シナリオベース（今回取り上げる内容）インスペクションルールに沿った厳格なチェック

Slide 10

Slide 10 text

10 レビュープロセスレビミテング再作業フロアプ計画キクオフミテング個々の準備 ( 前準備 ) ココはやってるココが重要︕ JSTQB Foundation Levelシラバスで定義されているプロセス

Slide 11

Slide 11 text

11 レビューは段取り⼋分︕ 1. 計画 ü ⼈選する ü 役割を決める ü レビューのやり⽅を決める ü どこをレビューするか決める ü レビューをはじめる条件を決める。 2. キックオフミーティング ü レビュー対象物を配布する。 ü 参加者にレビューの⽬的とやり⽅を説明する。 3. 個々の準備（前準備） ü 各⾃でドキュメントをチェックする。 ü ⾒つけた⽋陥、質問、コメントをメモする。

Slide 12

Slide 12 text

12 4. レビューミーティング ü 議論を⾏い、結果を記録する。 ü ⽋陥について述べ、扱いを決める。 5. 再作業 ü ⾒つけた⽋陥を修正する。 ü どのように⽋陥を直したか記録する。 6. フォローアップ ü ⽋陥を処理したかチェックする。

Slide 13

Slide 13 text

レビューのポイント「観点」を決める 13

Slide 14

Slide 14 text

14 レビュー観点とはこのレビューで、「どのような問題を⾒つけるべきなのか」を考えた結果導き出された、注⽬すべき点

Slide 15

Slide 15 text

15 どこから観点を⾒つけるのか v 規約類から ü 参加規約 ü 競技規約 ü 審査規約 v 技術情報から ü 開発プラットフォーム ü プログラミング⾔語 ü UML⽂法 v 既存の成果物や経験から ü 前回のモデルやソースコード ü 過去の経験や失敗事例、先輩からの引き継ぎ事項 ü 他チームのモデル ü これまでの調査、分析結果

Slide 16

Slide 16 text

16 観点を⾒つける審査基準に着⽬します。 [A1] [A2] [A] ⾒つけた観点には[A1][A2]といったIDを付けておきます。

Slide 17

Slide 17 text

観点を⾒つける 17 [B1] [B2] [B3] [B]

Slide 18

Slide 18 text

観点の詳細化 18 [B2-1] 振る舞い図の要素として、クラス図で定義したオブジェクトが使われているか︖ (システム全体の振る舞いじゃないよ︖） [B2-2] 振る舞い図は審査課題で選択したことの説明になっているか︖ （課題と関係ないこと書いてないよね︖）＜詳細化の例＞ [B2-3] 振る舞い図は正確に動作するか︖ (ちゃんと動くよね︖） [B2]

Slide 19

Slide 19 text

19 チェック⽅法を決める ID 観点どこをどのように B2-1 振る舞い図の要素として、構造で定義したものが使われているか︖ シーケンス図、コミュニケーション図が描かれている箇所ライフラインの名称がクラス名と⼀致していることを確認する。 B2-2 振る舞い図は審査課題で選択したことの説明になっているか︖ 審査課題の記述箇所と振る舞いについて説明しているページ振る舞いの内容を調べ、審査課題と⼀致しているか確認する。 B2-3 振る舞い図は正確に動作するか︖ シーケンス図、コミュニケーション図が描かれている場所メッセージを辿りながら、ライントレース⾛⾏が継続できるか確認する。 ↑メンバー間で認識が異ならない程度まで具体的に書く。

Slide 20

Slide 20 text

20 結果を記録する No. 問題点観点修正⽅法 1 クラス図では「ライントレーサ」クラスがあるが、シーケンス図にライントレーサが出てこない。 B2-1 ライントレーサを追加する。 2 3

Slide 21

Slide 21 text

〜演習〜⾃分たちのモデルをレビューしてみましょう。 21

Slide 22

Slide 22 text

22 今回取り上げる⽅法 v 名古屋⼤学の森崎修司先⽣が提唱している技法「シナリオレビュー」を参考にしました。 v 特徴 • 「重要な問題」を早期かつ確実に検出できる。 • レビューアの勘や経験、そのときの気分などに左右されにくい。 • 前準備をきちんと⾏う必要がある（必要な情報や知識も含め）

Slide 23

Slide 23 text

対象物と配布資料 23 配布物・レビュー観点シート・レビュー記録シート・ETロボコン2015審査規約・A3⽩紙レビュー対象・地区⼤会提出モデル＜レビュー観点シート＞＜レビュー記録＞

Slide 24

Slide 24 text

ブレーンストーミング 24 ⾒つけたい問題を書き出していきます。審査規約とか [気になった点] ・⽂字がつぶれて読みづらいです。残念︕ ・構造と振る舞いの⼀貫性がない。⾻太さが⾜りません。・責務が集中しすぎています。激おこぷんぷん丸です。審査コメントとか過去の失敗とか

Slide 25

Slide 25 text

25 ＜観点のトリアージ＞⾒つけたい問題を絞り込み、モデル観点シートに書き出します。１２３ ※今回は時間の都合で3つまでとします。絞り⽅の例・後で⼤きな問題になりそうなもの審査で⼤減点されそうなもの、他の箇所や後の作業に影響しそうなもの。など・確認箇所が多そうなもの

Slide 26

Slide 26 text

観点を決めます 26 問題を⾒つけるためには、具体的にどんなことに注⽬しないといけないのかを考え、書きます（今回は3つまでとします）クラス図とシーケンス図の⼀貫性が取れていないミスをなくしたい。シーケンス図に登場するオブジェクトがクラス図にも記載されているか︖ シーケンス図のメッセージの名前がクラス図の操作名と⼀致しているか︖ シーケンス図のオブジェクト間のメッセージとクラス図の関連線が⼀致しているか︖

Slide 27

Slide 27 text

探し⽅を決めます 27 どこをどういう⼿順で探していくのかを記載します。シーケンス図に登場するオブジェクトがクラス図にも記載されているか︖ クラス図とシーケンス図の記載箇所・2ページの2-1 ・3ページの3-2 クラス図のクラス名を⾚ペンで囲み、シーケンス図に同じオブジェクト名が出ていたら、そこも⾚ペンで囲む。⽚⽅にしかなければ、レ印でチェックする。記載の細かさは、「レビュー参加者が同じ認識を持てる程度」がベスト。（話し合いながら、細かすぎず、⼤雑把すぎないところを探しましょう）

Slide 28

Slide 28 text

実際にモデルを調べます。 28 観点シートに沿ったやり⽅ですすめましょう。シーケンス図に登場するオブジェクトがクラス図にも記載されているか︖ クラス図とシーケンス図の記載箇所・2ページの2-1 ・3ページの3-2 クラス図のクラス名を⾚ペンで囲み、シーケンス図に同じオブジェクト名が出ていたら、そこも⾚ペンで囲む。⽚⽅にしかなければ、✖印でチェックする。 1つめのシナリオが終わってから、次のシナリオへ進みましょう。本来は観点毎に分担して調べるのですが、今回は皆で⾒ていきましょう。

Slide 29

Slide 29 text

29 結果を記録しましょうクラス図では「ライントレーサ」クラスがあるが、シーケンス図にライントレーサが出てこない。 B2-1 観点IDを記録しておけば、どういう種類の指摘が多いのか、分析するのに役⽴ちます。

Slide 30

Slide 30 text

発表タイム 30 観点をどのように決めたかどんな問題を⾒つけられたか

Slide 31

Slide 31 text

31 成功させるポイント v ⽬的をはっきりさせる ü 「このレビューで、どんな問題を⾒つけたいのか︖」をはっきりさせ、メンバー間で認識を合わせておくこと。 ü 段取り⼋分。準備が重要。 ü この技法を使うポイントをうまく絞る（トリアージ）⾒つけたい問題の重み付けが重要 v メンバーに合った確認⽅法を決める ü 「メンバーが共通の認識を持てる確認⽅法」を記載。細かすぎず⼤雑把すぎずのポイントを探す。

Slide 32

Slide 32 text

32 気をつけるポイント v 勘と経験も、うまく組み合わせる ü シナリオ化することで、勘と経験を活かしにくくなるデメリットもある。うまく使い分ける。経験ベースのレビューも組み合わせ、そこで⾒つかった問題を観点リストに取り込み、徐々に形式知化していくのがよい。 v なんでもかんでも⽬視に頼らない ü 静的解析ツールに頼った⽅が確実に⾒つけられるものもある。 ü 中にはテストで⾒つけた⽅が低コストなものもある。

Slide 33

Slide 33 text

33 メリット v ノウハウを引き継げる ü レビュー観点や、その調べ⽅を後輩へ引き継げる。モデルやソースを渡すだけだと、悪い点まで引き継がれる。ノウハウを引き継げば、悪しき⾵習を断ち切れ、同じ失敗を繰り返さなくなる。 v やったことが形に残る ü どこをどのように調べたのか記録が残り、⾒直ししやすい。 ü ⽂章化することでメンバーの意識を合わせやすくなる。

Slide 34

Slide 34 text

34 メリット v 時間を⾒積もれるようになる。 ü ⼿順が明確なので、繰り返すことでレビュー時間を⾒積もることができるようになる。 ü やみくもに時間を消費することがなくなる。 v 不⾜スキルや調査漏れがわかる ü スキルや調査が⾜りてないと、観点を挙げることができない。 ü ⾃分たちの⼒や取り組みのどこが⾜りないのかがわかる。

Slide 35

Slide 35 text

35 参加者の感想こういうやり⽅は仕事でも経験がなかったが、確実に狙った問題を⾒つけられ、効果がありそう。最初に、⾒つけたい問題や観点を決めるのが難しい。前準備にも確認作業にも時間がかかる。計画や優先順位付けが重要そう。

Slide 36

Slide 36 text

36 参加者の感想観点を書き出すことによって、いろいろな効能がありそうですね。レビュアーによってレビューの品質がまちまちになるの防げる、レビューを機械的に、素早く、繰り返し⾏えることによる品質の向上など。複数⼈で観点を出し合うことでいろいろ発⾒もありそうです。引き継ぎにも効果絶⼤ですね。今までの「なんか変なとこないか⾒といて」から⾶躍的にレベルアップできそうです。明⽇から仕事でも使っていきます。

Slide 37

Slide 37 text

参考⽂献・引⽤ 37 ソフトウェアテスト教科書 JSTQB Foundation 第3版 http://amzn.to/1FA7Atx 間違いだらけの設計レビュー [改訂版] 森崎修司著 http://amzn.to/1LiS1Zo ETロボコン2015 審査規約 http://www.etrobo.jp/2015/gaiyou/shinsakiyaku.php ETロボコン2015 実施説明会資料 http://www.etrobo.jp/2015/taikai/image/PDF/01etrc2015_zent aisetumei.pdf

Slide 38

Slide 38 text

ETロボコンの情報はこちらから︕ 38 http://etrobo.jp/ みんなでETロボコンに参加して、設計スキル、品質スキルを⾼めましょう︕

Slide 39

Slide 39 text

END 39