[Lada-commits] [PATCH 02 of 10] Remove null-checks for already validated mandatory fields

Wald Commits scm-commit at wald.intevation.org
Thu Sep 29 16:54:43 CEST 2016


# HG changeset patch
# User Tom Gottfried <tom at intevation.de>
# Date 1474649237 -7200
# Node ID 0d0362fe761969251e578d42ef1393df4dc616f1
# Parent  6d26e94b643e6d5e7079e3e9d68cf266c989f3bb
Remove null-checks for already validated mandatory fields.

diff -r 6d26e94b643e -r 0d0362fe7619 src/main/java/de/intevation/lada/factory/ProbeFactory.java
--- a/src/main/java/de/intevation/lada/factory/ProbeFactory.java	Fri Sep 23 18:35:52 2016 +0200
+++ b/src/main/java/de/intevation/lada/factory/ProbeFactory.java	Fri Sep 23 18:47:17 2016 +0200
@@ -163,10 +163,8 @@
     ) {
         int offset = messprogramm.getIntervallOffset() == null ?
             0 : messprogramm.getIntervallOffset();
-        int teilVon = messprogramm.getTeilintervallVon() == null ?
-            0 : messprogramm.getTeilintervallVon() - 1;
-        int teilBis = messprogramm.getTeilintervallBis() == null ?
-            0 : messprogramm.getTeilintervallBis();
+        int teilVon = messprogramm.getTeilintervallVon() - 1;
+        int teilBis = messprogramm.getTeilintervallBis();
         int manualDuration = teilBis - teilVon;
 
         List<LProbe> proben = new ArrayList<LProbe>();
@@ -208,10 +206,8 @@
     ) {
         int offset = messprogramm.getIntervallOffset() == null ?
             0 : messprogramm.getIntervallOffset();
-        int teilVon = messprogramm.getTeilintervallVon() == null ?
-            0 : messprogramm.getTeilintervallVon() - 1;
-        int teilBis = messprogramm.getTeilintervallBis() == null ?
-            0 : messprogramm.getTeilintervallBis();
+        int teilVon = messprogramm.getTeilintervallVon() - 1;
+        int teilBis = messprogramm.getTeilintervallBis();
         int manualDuration = teilBis - teilVon;
 
         List<LProbe> proben = new ArrayList<LProbe>();
@@ -253,10 +249,8 @@
     ) {
         int offset = messprogramm.getIntervallOffset() == null ?
             0 : messprogramm.getIntervallOffset();
-        int teilVon = messprogramm.getTeilintervallVon() == null ?
-            0 : messprogramm.getTeilintervallVon() - 1;
-        int teilBis = messprogramm.getTeilintervallBis() == null ?
-            0 : messprogramm.getTeilintervallBis();
+        int teilVon = messprogramm.getTeilintervallVon() - 1;
+        int teilBis = messprogramm.getTeilintervallBis();
         int manualDuration = teilBis - teilVon;
 
         List<LProbe> proben = new ArrayList<LProbe>();
@@ -569,10 +563,8 @@
         for (int i = 0; i < years; i++) {
             Calendar cStart = Calendar.getInstance();
             cStart.set(startYear + i, start.get(Calendar.MONTH), start.get(Calendar.DAY_OF_MONTH));
-            if (messprogramm.getGueltigVon() == null ||
-                messprogramm.getGueltigVon() <= 0 ||
-                (realStart > messprogramm.getGueltigVon() &&
-                 i == 0)
+            if (messprogramm.getGueltigVon() <= 0
+                || (realStart > messprogramm.getGueltigVon() && i == 0)
             ) {
                 intervals[0][0] = start.getTime();
             }
@@ -584,10 +576,8 @@
 
             Calendar cEnd = Calendar.getInstance();
             cEnd.set(startYear + i, end.get(Calendar.MONTH), end.get(Calendar.DAY_OF_MONTH));
-            if (messprogramm.getGueltigBis() == null ||
-                messprogramm.getGueltigBis() <= 0 ||
-                (realEnd < messprogramm.getGueltigBis() &&
-                 i == years - 1)
+            if (messprogramm.getGueltigBis() <= 0
+                || (realEnd < messprogramm.getGueltigBis() && i == years - 1)
             ) {
                 intervals[i][1] = cEnd.getTime();
             }


More information about the Lada-commits mailing list