cqfn#390 CR

HDouss · Feb 24, 2020 · 1befa48 · 1befa48
1 parent e1926f4
commit 1befa48
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/main/java/org/jpeek/XslReport.java b/src/main/java/org/jpeek/XslReport.java
@@ -144,6 +144,10 @@ final class XslReport implements Report {
      * @param mean Mean
      * @param sigma Sigma
      * @checkstyle ParameterNumberCheck (10 lines)
+     * @todo #390:30min this constructor now has too many arguments. We should find a way to refactor the constructor
+     *  or the class to have fewer parameters. We could start by analyzing the usage of this.params (args in this
+     *  constructor) and get rid of it if it is not used. Another idea could be to have a data class contaning
+     *  reporting params: name, args, mean, sigma.
      */
     XslReport(final XML xml, final String name,
         final Map<String, Object> args, final Calculus calc,