【部署機能】削除機能のシーケンス図作成 #12
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
部署検索画面で任意の部署の「削除」ボタンをクリック→ポップアップで「OK」ボタンをクリック→削除が完了し、部署検索画面に戻る
この一連の流れのシーケンス図を作成する。
@t-yuasa
部署削除機能のシーケンス図になります。
【シーケンス図_部署削除機能実行処理】部署検索画面で削除ボタンを押下してから、部署の削除が完了するまでの処理
【シーケンス図_部署一覧表示】部署が削除されてから、部署検索画面に部署一覧が表示されるまでの処理
レビューをお願いいたします。
作成するシーケンス図は、削除ボタンを押下したときのイベントのみでOKです。
「シーケンス図_部署削除機能実行処理.png」こちらが該当機能
また、可読性が非常に悪く、ライフラインが重なっている部分と、複数のクラスに重なってメソッド名が記載されている部分については修正をお願いいたします。
バグ起票後に対応してください!
@t-yuasa
可読性の向上のため、シーケンス図_部署削除機能実行処理の図を2つに分割いたしました。
・シーケンス図_部署削除処理_修正版.png
・シーケンス図_所属社員検索処理_修正版.png
レビューをお願いします。
シーケンス図は1機能の流れをまとめた資料のため、2分割はあまり推奨されない。 ※部分的に切り離すパターンはあり
コーディング規約のようにある程度の方針が定まっているので、必ず確認を取りましょう。
・2ファイル目を参照するタイミングが分かりにくい
黄色いテキストメモ(付箋)を横に置いていますが、他の付箋と混在している
・ 1枚目の図で処理を「完結」させてしまっている
「処理が別図に遷移した」のではなく、戻り値の線を書いてその後の処理を継続させているため、詳細処理な処理を別ファイルに作成したような作りになっている
・2枚目の図に「呼び出し元」が記載されている
2枚目の図は SearchEmployeeImpl の処理詳細を描くためのものなので、一番左端の起点は SearchEmployeeImpl にするべき。
DeleteDepartmentController を再登場させてしまっているため、1枚目と2枚目で同じオブジェクト間のやり取りが重複している。