Linting also about constructors with too many parameters

ftomassetti · Aug 2, 2014 · c206ff4 · c206ff4
1 parent 0b1c320
commit c206ff4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/app/linter.clj b/src/app/linter.clj
@@ -10,7 +10,10 @@
 (defrecord Check [operation params message])
 
 (def checks
-  [(Check. classesWithManyConstructorsOp {:threshold 4} "This class has too many constructors (#1#). Considers the Builder pattern")])
+  [(Check. classesWithManyConstructorsOp {:threshold 4}
+     "This class has too many constructors (#1#). Consider using static factory methods or the Builder pattern")
+   (Check. constructorsWithManyParametersOp {:threshold 6}
+     "This constructor has too many parameters (#1#). Consider using the Builder pattern")])
 
 (defn replaceParamsInMessage [message result]
   (clojure.string/replace message "#1#" (toString (nth result 1))))