diff flys-client/src/main/java/de/intevation/flys/client/server/WQInfoServiceImpl.java @ 1367:ab8eb2f544f2

Replaced stdout and stderr logging with log4j loggers in server classes. flys-client/trunk@3069 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Ingo Weinzierl <ingo.weinzierl@intevation.de>
date Tue, 25 Oct 2011 12:31:15 +0000
parents 1e73d5a4859c
children bc06a671ef60
line wrap: on
line diff
--- a/flys-client/src/main/java/de/intevation/flys/client/server/WQInfoServiceImpl.java	Tue Oct 25 11:07:14 2011 +0000
+++ b/flys-client/src/main/java/de/intevation/flys/client/server/WQInfoServiceImpl.java	Tue Oct 25 12:31:15 2011 +0000
@@ -10,6 +10,8 @@
 import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
 
+import org.apache.log4j.Logger;
+
 import com.google.gwt.user.server.rpc.RemoteServiceServlet;
 
 import de.intevation.artifacts.common.ArtifactNamespaceContext;
@@ -32,6 +34,10 @@
 extends      RemoteServiceServlet
 implements   WQInfoService
 {
+    private static final Logger logger =
+        Logger.getLogger(WQInfoServiceImpl.class);
+
+
     public static final String ERROR_NO_WQINFO_FOUND =
         "error_no_wqinfo_found";
 
@@ -46,7 +52,7 @@
         double to)
     throws ServerException
     {
-        System.out.println("WQInfoServiceImpl.getWQInfo");
+        logger.info("WQInfoServiceImpl.getWQInfo");
 
         Document doc = XMLUtils.newDocument();
 
@@ -75,7 +81,7 @@
         try {
             Document result = client.callService(url, "mainvalues", doc);
 
-            System.out.println("Extract wq info objects now.");
+            logger.debug("Extract wq info objects now.");
             WQInfoObject[] objects = extractWQInfoObjects(result);
 
             if (objects != null && objects.length > 0) {
@@ -83,7 +89,7 @@
             }
         }
         catch (ConnectionException ce) {
-            System.err.println(ce.getLocalizedMessage());
+            logger.error(ce, ce);
         }
 
         throw new ServerException(ERROR_NO_WQINFO_FOUND);
@@ -107,13 +113,13 @@
             ArtifactNamespaceContext.INSTANCE);
 
         if (list == null || list.getLength() == 0) {
-            System.err.println("No wq info found.");
+            logger.warn("No wq info found.");
 
             throw new ServerException(ERROR_NO_WQINFO_FOUND);
         }
 
         int num = list.getLength();
-        System.out.println("Response contains " + num + " objects.");
+        logger.debug("Response contains " + num + " objects.");
 
         List<WQInfoObject> objects =
             new ArrayList<WQInfoObject>(num);
@@ -126,7 +132,7 @@
             }
         }
 
-        System.out.println("Retrieved " + objects.size() + " wq values");
+        logger.debug("Retrieved " + objects.size() + " wq values");
 
         return (WQInfoObject[])
             objects.toArray(new WQInfoObject[num]);
@@ -160,11 +166,11 @@
                     new Double(value));
             }
             catch (NumberFormatException nfe) {
-                System.err.println(nfe.getLocalizedMessage());
+                logger.warn(nfe.getLocalizedMessage());
             }
         }
 
-        System.err.println("Invalid wq info object found.");
+        logger.warn("Invalid wq info object found.");
 
         return null;
     }

http://dive4elements.wald.intevation.org