Fixed Sonar issues in NodeClass.java 31/64431/1
authorAmaresh Kumar <kamaresh@in.ibm.com>
Tue, 4 Sep 2018 13:22:48 +0000 (18:52 +0530)
committerAmaresh Kumar <kamaresh@in.ibm.com>
Tue, 4 Sep 2018 13:23:00 +0000 (18:53 +0530)
https://sonar.onap.org/project/issues?id=org.onap.dmaap.datarouter%3Aparent&open=AWWU4OsgxOv3LmQdDYyE&resolved=false&severities=BLOCKER&types=BUG

Issue-ID: DMAAP-698
Change-Id: I1609a5c300e55f1eb8bd8b4966d456fb3905cc00
Signed-off-by: Amaresh Kumar <kamaresh@in.ibm.com>
datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/NodeClass.java

index 1b36851..9d43bd0 100755 (executable)
@@ -105,20 +105,20 @@ public abstract class NodeClass extends Syncable {
     public static void reload() {\r
         Map<String, Integer> m = new HashMap<String, Integer>();\r
         PreparedStatement ps = null;\r
-        ResultSet rs=null;\r
+\r
         try {\r
             DB db = new DB();\r
             @SuppressWarnings("resource")\r
             Connection conn = db.getConnection();\r
             String sql = "select NODEID, NAME from NODES";\r
             ps = conn.prepareStatement(sql);\r
-            rs = ps.executeQuery();\r
-            while (rs.next()) {\r
-                int id = rs.getInt("NODEID");\r
-                String name = rs.getString("NAME");\r
-                m.put(name, id);\r
+            try(ResultSet rs = ps.executeQuery()) {\r
+                while (rs.next()) {\r
+                    int id = rs.getInt("NODEID");\r
+                    String name = rs.getString("NAME");\r
+                    m.put(name, id);\r
+                }\r
             }\r
-            rs.close();\r
             ps.close();\r
             db.release(conn);\r
         } catch (SQLException e) {\r
@@ -128,10 +128,6 @@ public abstract class NodeClass extends Syncable {
                 if(ps!=null){\r
                     ps.close();\r
                 }\r
-                if(rs!=null){\r
-                    rs.close();\r
-                }\r
-\r
             } catch (SQLException e) {\r
                 intLogger.error("PROV0005 doInsert: " + e.getMessage(),e);\r
             }\r