From 23b36e3c69f25d8322afb79d5e6bace519db7cd3 Mon Sep 17 00:00:00 2001
From: "Tait,Trevor(rt0435)" <rtait@amdocs.com>
Date: Tue, 30 Oct 2018 13:22:36 -0400
Subject: Address Bugs and Vulnerabilities in Sonar Report

Addressing Bugs and Vulnerabilities in Sonar Report.

https://sonar.onap.org/dashboard?id=org.onap.logging-analytics.pomba%3Apomba-sdnc-context-builder

Issue-ID: LOG-791
Change-Id: I11b984cac77c8a5674103672fa0e1332ed0aa4f7
Signed-off-by: Tait,Trevor(rt0435) <rtait@amdocs.com>
---
 .../pomba/contextbuilder/sdnc/model/VmNetwork.java   | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

(limited to 'src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VmNetwork.java')

diff --git a/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VmNetwork.java b/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VmNetwork.java
index cb12c26..27ddbdf 100644
--- a/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VmNetwork.java
+++ b/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VmNetwork.java
@@ -30,7 +30,7 @@ public class VmNetwork {
     private String networkRole;
     @SerializedName("network-ips-v6")
     @Expose
-    private List<NetworkIpV6> networkIpsV6 = new ArrayList<NetworkIpV6>();
+    private List<NetworkIpV6> networkIpsV6 = new ArrayList<>();
     @SerializedName("floating-ip")
     @Expose
     private String floatingIp;
@@ -39,11 +39,13 @@ public class VmNetwork {
     private String useDhcp;
     @SerializedName("network-ips")
     @Expose
-    private List<NetworkIp> networkIps = new ArrayList<NetworkIp>();
+    private List<NetworkIp> networkIps = new ArrayList<>();
     @SerializedName("floating-ip-v6")
     @Expose
     private String floatingIpV6;
 
+    private final static String NULL_STR = "<null>";
+
     /**
      * No args constructor for use in serialization
      *
@@ -124,27 +126,27 @@ public class VmNetwork {
         sb.append(VmNetwork.class.getName()).append('@').append(Integer.toHexString(System.identityHashCode(this))).append('[');
         sb.append("networkRole");
         sb.append('=');
-        sb.append(((this.networkRole == null)?"<null>":this.networkRole));
+        sb.append(((this.networkRole == null)?NULL_STR:this.networkRole));
         sb.append(',');
         sb.append("networkIpsV6");
         sb.append('=');
-        sb.append(((this.networkIpsV6 == null)?"<null>":this.networkIpsV6));
+        sb.append(((this.networkIpsV6 == null)?NULL_STR:this.networkIpsV6));
         sb.append(',');
         sb.append("floatingIp");
         sb.append('=');
-        sb.append(((this.floatingIp == null)?"<null>":this.floatingIp));
+        sb.append(((this.floatingIp == null)?NULL_STR:this.floatingIp));
         sb.append(',');
         sb.append("useDhcp");
         sb.append('=');
-        sb.append(((this.useDhcp == null)?"<null>":this.useDhcp));
+        sb.append(((this.useDhcp == null)?NULL_STR:this.useDhcp));
         sb.append(',');
         sb.append("networkIps");
         sb.append('=');
-        sb.append(((this.networkIps == null)?"<null>":this.networkIps));
+        sb.append(((this.networkIps == null)?NULL_STR:this.networkIps));
         sb.append(',');
         sb.append("floatingIpV6");
         sb.append('=');
-        sb.append(((this.floatingIpV6 == null)?"<null>":this.floatingIpV6));
+        sb.append(((this.floatingIpV6 == null)?NULL_STR:this.floatingIpV6));
         sb.append(',');
         if (sb.charAt((sb.length()- 1)) == ',') {
             sb.setCharAt((sb.length()- 1), ']');
@@ -171,7 +173,7 @@ public class VmNetwork {
         if (other == this) {
             return true;
         }
-        if ((other instanceof VmNetwork) == false) {
+        if (!(other instanceof VmNetwork)) {
             return false;
         }
         VmNetwork rhs = ((VmNetwork) other);
-- 
cgit 1.2.3-korg