【ドキュメント不良】部署検索のシーケンス図における内容の記載漏れ #10
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?
レビュー完了後の部署検索のシーケンス図において以下の不備があることが判明したため修正する。
【事象】
システムの解析のため、AOP(@Around)によるトレーシングでメソッド呼び出しをログ出力し、その記録をもとにシークエンス図を作成・提出したところ、「洗い出しの粒度が不足している」との指摘を受けた。具体的には、Controller→Service→Repository間で受け渡されるForm/Model/Entityの変換処理が図に反映されていなかった。
【原因】
Entity→Model変換(createLoginUser()等)のような、これらのオブジェクトが持つメソッド呼び出しはAOPのpointcutの対象範囲であったため。
【対策】
AOPトレーシングの結果を鵜呑みにせず、対象メソッド(Controller/Service)のソースコードを直接確認し、戻り値生成箇所における変換処理の有無を洗い出す。