Change "try" to try-with-resources 11/14611/2
authorsurya-huawei <a.u.surya@huawei.com>
Fri, 22 Sep 2017 13:44:49 +0000 (19:14 +0530)
committerSURYA A U <a.u.surya@huawei.com>
Mon, 25 Sep 2017 08:27:59 +0000 (08:27 +0000)
One major issue in sdnc/northbound module
*This is done for a guaranteed closing of resource
and avoiding the finally block

Issue-Id: CCSDK-87
Change-Id: I3ab9119f0b10f47d16864aa3619cc54e069bb9de
Signed-off-by: surya-huawei <a.u.surya@huawei.com>
vnfapi/provider/src/main/java/org/onap/sdnc/vnfapi/VnfSdnUtil.java

index c4d8c52..522ee7f 100644 (file)
@@ -53,23 +53,15 @@ public class VnfSdnUtil extends MdsalHelper {
         File propFile = new File(ODLHOME.getAbsolutePath() + "/configuration/vnfapi.properties");
         String propFileName = propFile.getAbsolutePath();
         properties = new Properties();
-        InputStream input = null;
         if (propFile.isFile() && propFile.canRead()) {
-            try    {
-                input = new FileInputStream(propFile);
+            try (InputStream input = new FileInputStream(propFile)) {
                 properties.load(input);
                 LOG.info("Loaded properties from " + propFileName );
                 setYangMappingProperties(properties);
+            } catch (IOException e) {
+                LOG.error("Failed to close properties file " + propFileName +"\n",e);
             } catch (Exception e) {
                 LOG.error("Failed to load properties " + propFileName +"\n",e);
-            } finally {
-                if (input != null) {
-                    try {
-                        input.close();
-                    } catch (IOException e) {
-                        LOG.error("Failed to close properties file " + propFileName +"\n",e);
-                    }
-                }
             }
         }
     }