]> source.dussan.org Git - sonarqube.git/commitdiff
Move validate() function to the bottom
authorJulien Lancelot <julien.lancelot@sonarsource.com>
Fri, 21 Mar 2014 07:33:49 +0000 (08:33 +0100)
committerJulien Lancelot <julien.lancelot@sonarsource.com>
Fri, 21 Mar 2014 07:33:49 +0000 (08:33 +0100)
sonar-plugin-api/src/main/java/org/sonar/api/server/rule/DebtRemediationFunction.java

index 476fe7698494c40964e6fdb62b02a85427ff8a9f..d355dbf5fee73294a549238a0309341361da1427 100644 (file)
@@ -56,28 +56,6 @@ public class DebtRemediationFunction {
     validate();
   }
 
-  private void validate(){
-    switch (type) {
-      case LINEAR:
-        if (this.factor == null || this.offset != null) {
-          throw new ValidationException(String.format("%s is invalid, Linear remediation function should only define a factor", this));
-        }
-        break;
-      case LINEAR_OFFSET:
-        if (this.factor == null || this.offset == null) {
-          throw new ValidationException(String.format("%s is invalid,  Linear with offset remediation function should define both factor and offset", this));
-        }
-        break;
-      case CONSTANT_ISSUE:
-        if (this.factor != null || this.offset == null) {
-          throw new ValidationException(String.format("%s is invalid, Constant/issue remediation function should only define an offset", this));
-        }
-        break;
-      default:
-        throw new IllegalStateException(String.format("Remediation function of %s is unknown", this));
-    }
-  }
-
   public static DebtRemediationFunction create(Type type, @Nullable String factor, @Nullable String offset) {
     return new DebtRemediationFunction(type, factor, offset);
   }
@@ -108,6 +86,28 @@ public class DebtRemediationFunction {
     return offset;
   }
 
+  private void validate(){
+    switch (type) {
+      case LINEAR:
+        if (this.factor == null || this.offset != null) {
+          throw new ValidationException(String.format("%s is invalid, Linear remediation function should only define a factor", this));
+        }
+        break;
+      case LINEAR_OFFSET:
+        if (this.factor == null || this.offset == null) {
+          throw new ValidationException(String.format("%s is invalid,  Linear with offset remediation function should define both factor and offset", this));
+        }
+        break;
+      case CONSTANT_ISSUE:
+        if (this.factor != null || this.offset == null) {
+          throw new ValidationException(String.format("%s is invalid, Constant/issue remediation function should only define an offset", this));
+        }
+        break;
+      default:
+        throw new IllegalStateException(String.format("Remediation function of %s is unknown", this));
+    }
+  }
+
   @Override
   public boolean equals(Object o) {
     if (this == o) {