Code Review / dmaap / datarouter.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Remove major and minor code smells in dr-prov
[dmaap/datarouter.git] / datarouter-prov / src / main / java / org / onap / dmaap / datarouter / provisioning / StatisticsServlet.java
diff --git a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/StatisticsServlet.java b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/StatisticsServlet.java
index 5fa7e16..9cbce0a 100755 (executable)
--- a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/StatisticsServlet.java
+++ b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/StatisticsServlet.java
@@ -57,8 +57,9 @@ import static org.onap.dmaap.datarouter.provisioning.utils.HttpServletUtils.send
 public class StatisticsServlet extends BaseServlet {\r
 \r
   private static final long TWENTYFOUR_HOURS = (24 * 60 * 60 * 1000L);\r
-  private static final String fmt1 = "yyyy-MM-dd'T'HH:mm:ss'Z'";\r
-  private static final String fmt2 = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'";\r
+  private static final String FMT1 = "yyyy-MM-dd'T'HH:mm:ss'Z'";\r
+  private static final String FMT2 = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'";\r
+\r
 \r
 \r
   /**\r
@@ -92,9 +93,8 @@ public class StatisticsServlet extends BaseServlet {
     resp.setContentType(LOGLIST_CONTENT_TYPE);\r
 \r
     String outputType = "json";\r
-    String feedids = null;\r
 \r
-    if (req.getParameter("feedid") == null && req.getParameter("groupid") == null) {\r
+    if (req.getParameter(FEEDID) == null && req.getParameter(GROUPID) == null) {\r
       try {\r
         resp.getOutputStream().print("Invalid request, Feedid or Group ID is required.");\r
       } catch (IOException ioe) {\r
@@ -102,69 +102,69 @@ public class StatisticsServlet extends BaseServlet {
       }\r
     }\r
 \r
-    if (req.getParameter("feedid") != null && req.getParameter("groupid") == null) {\r
-      map.put("feedids", req.getParameter("feedid").replace("|", ",").toString());\r
+    if (req.getParameter(FEEDID) != null && req.getParameter(GROUPID) == null) {\r
+      map.put(FEEDIDS, req.getParameter(FEEDID).replace("|", ","));\r
     }\r
 \r
-    if (req.getParameter("groupid") != null && req.getParameter("feedid") == null) {\r
+    if (req.getParameter(GROUPID) != null && req.getParameter(FEEDID) == null) {\r
       StringBuffer groupid1 = new StringBuffer();\r
 \r
       try {\r
-        groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter("groupid")));\r
-        map.put("feedids", groupid1.toString());\r
+        groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter(GROUPID)));\r
+        map.put(FEEDIDS, groupid1.toString());\r
       } catch (NumberFormatException | SQLException e) {\r
         eventlogger.error("PROV0172 StatisticsServlet.doGet: " + e.getMessage(), e);\r
       }\r
     }\r
-    if (req.getParameter("groupid") != null && req.getParameter("feedid") != null) {\r
+    if (req.getParameter(GROUPID) != null && req.getParameter(FEEDID) != null) {\r
       StringBuffer groupid1 = new StringBuffer();\r
 \r
       try {\r
-        groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter("groupid")));\r
+        groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter(GROUPID)));\r
         groupid1.append(",");\r
-        groupid1.append(req.getParameter("feedid").replace("|", ",").toString());\r
-        map.put("feedids", groupid1.toString());\r
+        groupid1.append(req.getParameter(FEEDID).replace("|", ","));\r
+        map.put(FEEDIDS, groupid1.toString());\r
       } catch (NumberFormatException | SQLException e) {\r
         eventlogger.error("PROV0173 StatisticsServlet.doGet: " + e.getMessage(), e);\r
       }\r
     }\r
 \r
-    if (req.getParameter("subid") != null && req.getParameter("feedid") != null) {\r
+    if (req.getParameter(SUBID) != null && req.getParameter(FEEDID) != null) {\r
       StringBuffer subidstr = new StringBuffer();\r
       subidstr.append("and e.DELIVERY_SUBID in(");\r
 \r
-      subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
+      subidstr.append(req.getParameter(SUBID).replace("|", ","));\r
       subidstr.append(")");\r
-      map.put("subid", subidstr.toString());\r
+      map.put(SUBID, subidstr.toString());\r
     }\r
-    if (req.getParameter("subid") != null && req.getParameter("groupid") != null) {\r
+    if (req.getParameter(SUBID) != null && req.getParameter(GROUPID) != null) {\r
       StringBuffer subidstr = new StringBuffer();\r
       subidstr.append("and e.DELIVERY_SUBID in(");\r
 \r
-      subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
+      subidstr.append(req.getParameter(SUBID).replace("|", ","));\r
       subidstr.append(")");\r
-      map.put("subid", subidstr.toString());\r
+      map.put(SUBID, subidstr.toString());\r
     }\r
     if (req.getParameter("type") != null) {\r
-      map.put("eventType", req.getParameter("type").replace("|", ",").toString());\r
+      map.put(EVENT_TYPE, req.getParameter("type").replace("|", ","));\r
     }\r
-    if (req.getParameter("output_type") != null) {\r
-      map.put("output_type", req.getParameter("output_type").toString());\r
+    if (req.getParameter(OUTPUT_TYPE) != null) {\r
+      map.put(OUTPUT_TYPE, req.getParameter(OUTPUT_TYPE));\r
     }\r
-    if (req.getParameter("start_time") != null) {\r
-      map.put("start_time", req.getParameter("start_time").toString());\r
+    if (req.getParameter(START_TIME) != null) {\r
+      map.put(START_TIME, req.getParameter(START_TIME));\r
     }\r
-    if (req.getParameter("end_time") != null) {\r
-      map.put("end_time", req.getParameter("end_time").toString());\r
+    if (req.getParameter(END_TIME) != null) {\r
+      map.put(END_TIME, req.getParameter(END_TIME));\r
     }\r
 \r
     if (req.getParameter("time") != null) {\r
-      map.put("start_time", req.getParameter("time").toString());\r
-      map.put("end_time", null);\r
+      map.put(START_TIME, req.getParameter("time"));\r
+      map.put(END_TIME, null);\r
     }\r
 \r
-    if (req.getParameter("output_type") != null) {\r
-      outputType = req.getParameter("output_type");\r
+    if (req.getParameter(OUTPUT_TYPE) != null) {\r
+      outputType = req.getParameter(OUTPUT_TYPE);\r
     }\r
     try {\r
       this.getRecordsForSQL(map, outputType, resp.getOutputStream(), resp);\r
@@ -225,7 +225,7 @@ public class StatisticsServlet extends BaseServlet {
    */\r
   public void rsToJson(ResultSet rs, ServletOutputStream out) throws IOException, SQLException {\r
 \r
-    String fields[] = {"FEEDNAME", "FEEDID", "FILES_PUBLISHED", "PUBLISH_LENGTH", "FILES_DELIVERED",\r
+    String[] fields = {"FEEDNAME", "FEEDID", "FILES_PUBLISHED", "PUBLISH_LENGTH", "FILES_DELIVERED",\r
         "DELIVERED_LENGTH", "SUBSCRIBER_URL", "SUBID", "PUBLISH_TIME", "DELIVERY_TIME",\r
         "AverageDelay"};\r
     StringBuffer line = new StringBuffer();\r
@@ -258,7 +258,6 @@ public class StatisticsServlet extends BaseServlet {
 \r
     DB db = null;\r
     Connection conn = null;\r
-    //PreparedStatement prepareStatement = null;\r
     ResultSet resultSet = null;\r
     String sqlGoupid = null;\r
     StringBuffer feedIds = new StringBuffer();\r
@@ -309,23 +308,23 @@ public class StatisticsServlet extends BaseServlet {
     String start_time = null;\r
     String end_time = null;\r
     String subid = " ";\r
-    if (map.get("eventType") != null) {\r
-      eventType = (String) map.get("eventType");\r
+    if (map.get(EVENT_TYPE) != null) {\r
+      eventType =  map.get(EVENT_TYPE);\r
     }\r
-    if (map.get("feedids") != null) {\r
-      feedids = (String) map.get("feedids");\r
+    if (map.get(FEEDIDS) != null) {\r
+      feedids = map.get(FEEDIDS);\r
     }\r
-    if (map.get("start_time") != null) {\r
-      start_time = (String) map.get("start_time");\r
+    if (map.get(START_TIME) != null) {\r
+      start_time = map.get(START_TIME);\r
     }\r
-    if (map.get("end_time") != null) {\r
-      end_time = (String) map.get("end_time");\r
+    if (map.get(END_TIME) != null) {\r
+      end_time =  map.get(END_TIME);\r
     }\r
     if ("all".equalsIgnoreCase(eventType)) {\r
       eventType = "PUB','DEL, EXP, PBF";\r
     }\r
-    if (map.get("subid") != null) {\r
-      subid = (String) map.get("subid");\r
+    if (map.get(SUBID) != null) {\r
+      subid = map.get(SUBID);\r
     }\r
 \r
     eventlogger.info("Generating sql query to get Statistics resultset. ");\r
@@ -412,10 +411,10 @@ public class StatisticsServlet extends BaseServlet {
   }\r
 \r
   private Map<String, String> buildMapFromRequest(HttpServletRequest req) {\r
-    Map<String, String> map = new HashMap<String, String>();\r
+    Map<String, String> map = new HashMap<>();\r
     String s = req.getParameter("type");\r
     if (s != null) {\r
-      if (s.equals("pub") || s.equals("del") || s.equals("exp")) {\r
+      if ("pub".equals(s) || "del".equals(s) || "exp".equals(s)) {\r
         map.put("type", s);\r
       } else {\r
         map.put("err", "bad type");\r
@@ -427,7 +426,7 @@ public class StatisticsServlet extends BaseServlet {
     map.put("publishSQL", "");\r
     map.put("statusSQL", "");\r
     map.put("resultSQL", "");\r
-    map.put("reasonSQL", "");\r
+    map.put(REASON_SQL, "");\r
 \r
     s = req.getParameter("publishId");\r
     if (s != null) {\r
@@ -441,11 +440,11 @@ public class StatisticsServlet extends BaseServlet {
     s = req.getParameter("statusCode");\r
     if (s != null) {\r
       String sql = null;\r
-      if (s.equals("success")) {\r
+      if ("success".equals(s)) {\r
         sql = " AND STATUS >= 200 AND STATUS < 300";\r
-      } else if (s.equals("redirect")) {\r
+      } else if ("redirect".equals(s)) {\r
         sql = " AND STATUS >= 300 AND STATUS < 400";\r
-      } else if (s.equals("failure")) {\r
+      } else if ("failure".equals(s)) {\r
         sql = " AND STATUS >= 400";\r
       } else {\r
         try {\r
@@ -467,14 +466,14 @@ public class StatisticsServlet extends BaseServlet {
     s = req.getParameter("expiryReason");\r
     if (s != null) {\r
       map.put("type", "exp");\r
-      if (s.equals("notRetryable")) {\r
-        map.put("reasonSQL", " AND REASON = 'notRetryable'");\r
-      } else if (s.equals("retriesExhausted")) {\r
-        map.put("reasonSQL", " AND REASON = 'retriesExhausted'");\r
-      } else if (s.equals("diskFull")) {\r
-        map.put("reasonSQL", " AND REASON = 'diskFull'");\r
-      } else if (s.equals("other")) {\r
-        map.put("reasonSQL", " AND REASON = 'other'");\r
+      if ("notRetryable".equals(s)) {\r
+        map.put(REASON_SQL, " AND REASON = 'notRetryable'");\r
+      } else if ("retriesExhausted".equals(s)) {\r
+        map.put(REASON_SQL, " AND REASON = 'retriesExhausted'");\r
+      } else if ("diskFull".equals(s)) {\r
+        map.put(REASON_SQL, " AND REASON = 'diskFull'");\r
+      } else if ("other".equals("other")) {\r
+        map.put(REASON_SQL, " AND REASON = 'other'");\r
       } else {\r
         map.put("err", "bad expiryReason");\r
         return map;\r
@@ -509,7 +508,7 @@ public class StatisticsServlet extends BaseServlet {
     }\r
     try {\r
       // First, look for an RFC 3339 date\r
-      String fmt = (s.indexOf('.') > 0) ? fmt2 : fmt1;\r
+      String fmt = (s.indexOf('.') > 0) ? FMT2 : FMT1;\r
       SimpleDateFormat sdf = new SimpleDateFormat(fmt);\r
       Date d = sdf.parse(s);\r
       return d.getTime();\r
@@ -517,8 +516,7 @@ public class StatisticsServlet extends BaseServlet {
     }\r
     try {\r
       // Also allow a long (in ms); useful for testing\r
-      long n = Long.parseLong(s);\r
-      return n;\r
+      return Long.parseLong(s);\r
     } catch (NumberFormatException e) {\r
     }\r
     intlogger.info("Error parsing time=" + s);\r
@@ -535,7 +533,7 @@ public class StatisticsServlet extends BaseServlet {
       DB db = new DB();\r
       try (Connection conn = db.getConnection()) {\r
         try (ResultSet rs = conn.prepareStatement(filterQuery).executeQuery()) {\r
-          if (outputType.equals("csv")) {\r
+          if ("csv".equals(outputType)) {\r
             resp.setContentType("application/octet-stream");\r
             Date date = new Date();\r
             SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-YYYY HH:mm:ss");\r
The Data Routing System project is intended to provide a common framework by which data producers can make data available to data consumers and a way for potential consumers to find feeds with the data they require. The interface to DR is exposed as a RESTful web service known as the DR Publishing and Delivery API