#152 Improved error message

jimbethancourt · jimbethancourt · commit 9a8bb9f4d114 · 2026-01-11T14:18:18.000-06:00
Improved error message, asking users to file a bug.
diff --git a/cost-benefit-calculator/src/main/java/org/hjug/cbc/CostBenefitCalculator.java b/cost-benefit-calculator/src/main/java/org/hjug/cbc/CostBenefitCalculator.java
@@ -183,7 +183,12 @@ public <T extends Disharmony> List<ScmLogInfo> getRankedChangeProneness(List<T>
                         }
                     } catch (NullPointerException e) {
                         // Should not be reached
-                        log.error("Error looking up class SCM info.  Class: {}, Path: {}", className, path, e);
+                        log.error(
+                                "Error looking up class SCM info.  If this error is encountered, "
+                                        + "please log a bug on the RefactorFirst project and describe if the class is a nested class, lambda, etc. \nClass: {}, Path: {}",
+                                className,
+                                path,
+                                e);
                     }
 
                     Optional<ScmLogInfo> scmLogInfoOptional = Optional.ofNullable(scmLogInfo);

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,12 @@ public <T extends Disharmony> List<ScmLogInfo> getRankedChangeProneness(List<T>`
`183`	`183`	`}`
`184`	`184`	`} catch (NullPointerException e) {`
`185`	`185`	`// Should not be reached`
`186`		`- log.error("Error looking up class SCM info. Class: {}, Path: {}", className, path, e);`
	`186`	`+ log.error(`
	`187`	`+ "Error looking up class SCM info. If this error is encountered, "`
	`188`	`+ + "please log a bug on the RefactorFirst project and describe if the class is a nested class, lambda, etc. \nClass: {}, Path: {}",`
	`189`	`+ className,`
	`190`	`+ path,`
	`191`	`+ e);`
`187`	`192`	`}`
`188`	`193`
`189`	`194`	`Optional<ScmLogInfo> scmLogInfoOptional = Optional.ofNullable(scmLogInfo);`