From d887566d7f0ca4e46bd7c22ec735efcaf8c8e3fc Mon Sep 17 00:00:00 2001 From: Jim Bethancourt Date: Sun, 16 Mar 2025 21:08:34 -0500 Subject: [PATCH] Fixing JavaDoc & adding sample report --- dsm/src/main/java/org/hjug/dsm/DSM.java | 3 - spring-petclinic-rest-report.html | 983 ++++++++++++++++++++++++ 2 files changed, 983 insertions(+), 3 deletions(-) create mode 100644 spring-petclinic-rest-report.html diff --git a/dsm/src/main/java/org/hjug/dsm/DSM.java b/dsm/src/main/java/org/hjug/dsm/DSM.java index 93190380..618ecae0 100644 --- a/dsm/src/main/java/org/hjug/dsm/DSM.java +++ b/dsm/src/main/java/org/hjug/dsm/DSM.java @@ -245,9 +245,6 @@ void printDSM(Graph graph, List sortedActiv /** * Captures the impact of the removal of each edge above the diagonal. - * - * @param limit the number of back edges to analyze - * @return List Impact of each edge if removed. */ public List getImpactOfEdgesAboveDiagonalIfRemoved(int limit) { // get edges above diagonal for DSM graph diff --git a/spring-petclinic-rest-report.html b/spring-petclinic-rest-report.html new file mode 100644 index 00000000..9458a96f --- /dev/null +++ b/spring-petclinic-rest-report.html @@ -0,0 +1,983 @@ + + +Refactor First Report for spring-petclinic-rest 3.4.3 + + + + + + + + + + + +
+ +
+
+
+

RefactorFirst Report for spring-petclinic-rest 3.4.3

+ Back Edges +
+ Highly Coupled Classes +
+ Class Cycles +

Class Map

Excludes classes that have no incoming and outgoing edges
+ + + +
+ Red lines represent back edges to remove.
+ Zoom in / out with your mouse wheel and click/move to drag the image. +
+
+ +
+
+ Show RefactorFirst some ❤️ +
+ Star + Fork + Watch + Issue + Sponsor +

+ +
+ Current Cycle Count: 3
+ Current Average Cycle Node Count: 2.3333333333333335
+ Current Total Back Edge Count: 4
+ Current Total Min Weight Back Edge Count: 1
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
EdgeEdge WeightIn # of CyclesNew Cycle CountNew Avg Cycle Node CountAvg Node Δ ÷ Effort
Visit → Pet : 33132.00.11111111111111116
Owner → Pet : 1313132.00.025641025641025654
Role → User : 33122.5-0.055555555555555504
EntityUtils → BaseEntity : 11122.5-0.16666666666666652
+
+ Show RefactorFirst some ❤️ +
+ Star + Fork + Watch + Issue + Sponsor +

+
+
+
+
+
+
+
+
+ Show RefactorFirst some ❤️ +
+ Star + Fork + Watch + Issue + Sponsor +

Coupling Between Objects Chart Legend:

+ + + + + + +
X-Axis: Number of objects the class is coupled to
Y-Axis: Relative churn
Color: Priority of what to fix first
Circle size: Priority (Visual) of what to fix first

Highly Coupled classes by the numbers: (Refactor starting with Priority 1)

+ + + + + +
ClassPriorityChange Proneness RankCoupling CountMost Recent Commit DateCommit Count
ClinicServiceImpl.java112212/28/24, 11:16 AM23

+
+
+
+
+
+
+ +

Class Cycles by the numbers:

+ + + + + + + + + + + + + + + + + + + + + + + + + + + + +
Cycle NamePriorityClass CountRelationship Count
Pet134
Role222
BaseEntity322
+
+
+
+
+
+
+

Class Cycle : Pet

+ + + + +
+ Red lines represent back edges to remove.
+ Zoom in / out with your mouse wheel and click/move to drag the image. +
+
+ +
+
+
Bold text indicates back edge to remove to decompose cycle
+ + + + + + + + + + + + + + + + + +
ClassesRelationships
PetPet → Visit : 9
+ Pet → Owner : 3
+
OwnerOwner → Pet : 13
+
VisitVisit → Pet : 3
+
+
+
+
+
+
+

Class Cycle : Role

+ + + + +
+ Red lines represent back edges to remove.
+ Zoom in / out with your mouse wheel and click/move to drag the image. +
+
+ +
+
+
Bold text indicates back edge to remove to decompose cycle
+ + + + + + + + + + + + + + +
ClassesRelationships
RoleRole → User : 3
+
UserUser → Role : 6
+
+
+
+
+
+
+

Class Cycle : BaseEntity

+ + + + +
+ Red lines represent back edges to remove.
+ Zoom in / out with your mouse wheel and click/move to drag the image. +
+
+ +
+
+
Bold text indicates back edge to remove to decompose cycle
+ + + + + + + + + + + + + + +
ClassesRelationships
BaseEntityBaseEntity → EntityUtils : 4
+
EntityUtilsEntityUtils → BaseEntity : 1
+
+
+
+
+
+ Last Published: 3/16/25, 8:52 PM
+
+
+
+
+ +