diff artifacts/src/main/java/org/dive4elements/river/artifacts/states/LocationSelect.java @ 9584:1a7cfeb1ff89

Punkt 10.8 WQ-Vorschau
author gernotbelger
date Wed, 09 Jan 2019 17:13:25 +0100
parents e4606eae8ea5
children
line wrap: on
line diff
--- a/artifacts/src/main/java/org/dive4elements/river/artifacts/states/LocationSelect.java	Wed Jan 09 14:48:53 2019 +0100
+++ b/artifacts/src/main/java/org/dive4elements/river/artifacts/states/LocationSelect.java	Wed Jan 09 17:13:25 2019 +0100
@@ -8,21 +8,15 @@
 
 package org.dive4elements.river.artifacts.states;
 
-import gnu.trove.TDoubleArrayList;
-
 import org.apache.log4j.Logger;
-
+import org.dive4elements.artifactdatabase.data.StateData;
+import org.dive4elements.artifacts.Artifact;
+import org.dive4elements.artifacts.CallContext;
+import org.dive4elements.artifacts.common.utils.XMLUtils;
+import org.dive4elements.river.artifacts.D4EArtifact;
 import org.w3c.dom.Element;
 
-import org.dive4elements.artifacts.Artifact;
-import org.dive4elements.artifacts.CallContext;
-
-import org.dive4elements.artifacts.common.utils.XMLUtils;
-
-import org.dive4elements.artifactdatabase.data.StateData;
-
-import org.dive4elements.river.artifacts.D4EArtifact;
-
+import gnu.trove.TDoubleArrayList;
 
 /**
  * This state is used to realize the input of multiple locations as string.
@@ -34,29 +28,22 @@
  */
 public class LocationSelect extends LocationDistanceSelect {
 
-    /** The log used in this class.*/
+    private static final long serialVersionUID = 1L;
+    /** The log used in this class. */
     private static Logger log = Logger.getLogger(LocationSelect.class);
 
-
     public LocationSelect() {
     }
 
-
     /** UI Provider (which input method should the client provide to user. */
     @Override
     protected String getUIProvider() {
         return "location_panel";
     }
 
-
     @Override
-    protected Element[] createItems(
-        XMLUtils.ElementCreator cr,
-        Artifact    artifact,
-        String      name,
-        CallContext context)
-    {
-        double[] minmax = getMinMax(artifact);
+    protected Element[] createItems(final XMLUtils.ElementCreator cr, final Artifact artifact, final String name, final CallContext context) {
+        final double[] minmax = getMinMax(artifact);
 
         double minVal = Double.MIN_VALUE;
         double maxVal = Double.MAX_VALUE;
@@ -64,19 +51,14 @@
         if (minmax != null) {
             minVal = minmax[0];
             maxVal = minmax[1];
-        }
-        else {
+        } else {
             log.warn("Could not read min/max distance values!");
         }
 
         if (name.equals(LOCATIONS)) {
-            Element min = createItem(
-                cr,
-                new String[] {"min", new Double(minVal).toString()});
+            final Element min = createItem(cr, new String[] { "min", new Double(minVal).toString() });
 
-            Element max = createItem(
-                cr,
-                new String[] {"max", new Double(maxVal).toString()});
+            final Element max = createItem(cr, new String[] { "max", new Double(maxVal).toString() });
 
             return new Element[] { min, max };
         }
@@ -84,53 +66,48 @@
         return null;
     }
 
-
     /** Validates data from artifact. */
     @Override
-    public boolean validate(Artifact artifact)
-    throws IllegalArgumentException
-    {
+    public boolean validate(final Artifact artifact) throws IllegalArgumentException {
         log.debug("LocationSelect.validate");
 
-        D4EArtifact flys = (D4EArtifact) artifact;
-        StateData    data = getData(flys, LOCATIONS);
+        final D4EArtifact flys = (D4EArtifact) artifact;
+        final StateData data = getData(flys, LOCATIONS);
 
-        String locationStr = data != null
-            ? (String) data.getValue()
-            : null;
+        final String locationStr = data != null ? (String) data.getValue() : null;
 
         if (locationStr == null || locationStr.length() == 0) {
             log.error("No locations given.");
             throw new IllegalArgumentException("error_empty_state");
         }
 
-        double[] minmax = getMinMax(artifact);
-        double[] mm     = extractLocations(locationStr);
+        final double[] minmax = getMinMax(artifact);
+        final double[] mm = extractLocations(locationStr);
 
         log.debug("Inserted min location: " + mm[0]);
-        log.debug("Inserted max location: " + mm[mm.length-1]);
+        log.debug("Inserted max location: " + mm[mm.length - 1]);
 
-        return validateBounds(minmax[0], minmax[1], mm[0], mm[mm.length-1], 0d);
+        return validateBounds(minmax[0], minmax[1], mm[0], mm[mm.length - 1], 0d);
     }
 
-
     /**
      * This method takes a string that consist of whitespace separated double
      * values and returns the double values as array.
      *
-     * @param locationStr The locations inserted in this state.
+     * @param locationStr
+     *            The locations inserted in this state.
      *
      * @return the locations as array.
      */
-    protected double[] extractLocations(String locationStr) {
-        String[] tmp               = locationStr.split(" ");
-        TDoubleArrayList locations = new TDoubleArrayList();
+    protected double[] extractLocations(final String locationStr) {
+        final String[] tmp = locationStr.split(" ");
+        final TDoubleArrayList locations = new TDoubleArrayList();
 
-        for (String l: tmp) {
+        for (final String l : tmp) {
             try {
                 locations.add(Double.parseDouble(l));
             }
-            catch (NumberFormatException nfe) {
+            catch (final NumberFormatException nfe) {
                 log.warn(nfe, nfe);
             }
         }

http://dive4elements.wald.intevation.org