Mercurial > dive4elements > river
diff flys-artifacts/src/main/java/de/intevation/flys/exports/fixings/FixDeltaWtGenerator.java @ 3404:d7b065b88f10
Simplified code.
flys-artifacts/trunk@5056 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author | Sascha L. Teichmann <sascha.teichmann@intevation.de> |
---|---|
date | Thu, 19 Jul 2012 09:11:56 +0000 |
parents | f062b5a90e26 |
children | 3732e1bab0aa |
line wrap: on
line diff
--- a/flys-artifacts/src/main/java/de/intevation/flys/exports/fixings/FixDeltaWtGenerator.java Wed Jul 18 16:37:41 2012 +0000 +++ b/flys-artifacts/src/main/java/de/intevation/flys/exports/fixings/FixDeltaWtGenerator.java Thu Jul 19 09:11:56 2012 +0000 @@ -446,14 +446,19 @@ public void init(Document request, OutputStream out, CallContext context) { super.init(request, out, context); - logger.debug("currentKm = " + new Double(getCurrentKmFromRequest())); - context.putContextValue("currentKm", new Double(getCurrentKmFromRequest())); + Double currentKm = getCurrentKmFromRequest(); + + if (logger.isDebugEnabled()) { + logger.debug("currentKm = " + currentKm); + } + + context.putContextValue("currentKm", currentKm); StyledValueMarker marker = new StyledValueMarker(0, request); valueMarker.add(marker); } - public double getCurrentKmFromRequest() { + public Double getCurrentKmFromRequest() { Element km = (Element)XMLUtils.xpath( request, XPATH_CHART_CURRENTKM, @@ -461,21 +466,19 @@ ArtifactNamespaceContext.INSTANCE); if (km == null) { - return -1d; + return Double.valueOf(-1d); } String uri = ArtifactNamespaceContext.NAMESPACE_URI; - + // XXX: When using XPath already why is the 'km' + // attribute not fetched directly? String currentKm = km.getAttributeNS(uri, "km"); try { - double d = Double.valueOf(currentKm).doubleValue(); - return d; + return Double.valueOf(currentKm); } - catch(NumberFormatException nfe) { - return -1d; + catch (NumberFormatException nfe) { + return Double.valueOf(-1d); } } - - } // vim:set ts=4 sw=4 si et sta sts=4 fenc=utf-8 :