Change "try" to try-with-resources 19/14119/1
authorsurya-huawei <a.u.surya@huawei.com>
Thu, 21 Sep 2017 06:13:59 +0000 (11:43 +0530)
committersurya-huawei <a.u.surya@huawei.com>
Thu, 21 Sep 2017 06:17:59 +0000 (11:47 +0530)
One major issue in sdnc/northbound module
*This is done for a guaranteed closing of resource
and not worrying about finally block to close it

Issue-Id: CCSDK-87
Change-Id: Ic6ea871cb9fb61cbce24b61d7c434d3404570ae7
Signed-off-by: surya-huawei <a.u.surya@huawei.com>
generic-resource-api/provider/src/main/java/org/onap/sdnc/northbound/GenericResourceApiUtil.java

index 5b534bb..6e6da0e 100644 (file)
@@ -22,23 +22,15 @@ public class GenericResourceApiUtil extends MdsalHelper {
 
                File file = new File(PROPERTIES_FILE);
                properties = new Properties();
-               InputStream input = null;
-               if (file.isFile() && file.canRead()) {
-                       try     {
-                               input = new FileInputStream(file);
+               if(file.isFile() && file.canRead()) {
+                       try(InputStream input = new FileInputStream(file)){
                                properties.load(input);
                                LOG.info("Loaded properties from " + PROPERTIES_FILE );
                                setYangMappingProperties(properties);
+                       } catch (IOException e) {
+                               LOG.error("Failed to close properties file " + PROPERTIES_FILE +"\n",e);
                        } catch (Exception e) {
                                LOG.error("Failed to load properties " + PROPERTIES_FILE +"\n",e);
-                       } finally {
-                               if (input != null) {
-                                       try {
-                                               input.close();
-                                       } catch (IOException e) {
-                                               LOG.error("Failed to close properties file " + PROPERTIES_FILE +"\n",e);
-                                       }
-                               }
                        }
                }
        }