diff options
author | 2018-10-30 13:22:36 -0400 | |
---|---|---|
committer | 2018-11-02 14:12:49 -0400 | |
commit | 23b36e3c69f25d8322afb79d5e6bace519db7cd3 (patch) | |
tree | 5223fe7c12d0b81877759e0c19653c6249c8cf29 /src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java | |
parent | a088960969f4d54e39cd83c8e24565301b6c7d67 (diff) |
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>
Diffstat (limited to 'src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java')
-rw-r--r-- | src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java b/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java index 2abfb31..e04e98f 100644 --- a/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java +++ b/src/main/java/org/onap/pomba/contextbuilder/sdnc/model/VnfAssignments.java @@ -27,13 +27,15 @@ public class VnfAssignments { @SerializedName("vnf-networks") @Expose - private List<VnfNetwork> vnfNetworks = new ArrayList<VnfNetwork>(); + private List<VnfNetwork> vnfNetworks = new ArrayList<>(); @SerializedName("availability-zones") @Expose - private List<AvailabilityZone> availabilityZones = new ArrayList<AvailabilityZone>(); + private List<AvailabilityZone> availabilityZones = new ArrayList<>(); @SerializedName("vnf-vms") @Expose - private List<VnfVm> vnfVms = new ArrayList<VnfVm>(); + private List<VnfVm> vnfVms = new ArrayList<>(); + + private final static String NULL_STR = "<null>"; /** * No args constructor for use in serialization @@ -85,15 +87,15 @@ public class VnfAssignments { sb.append(VnfAssignments.class.getName()).append('@').append(Integer.toHexString(System.identityHashCode(this))).append('['); sb.append("vnfNetworks"); sb.append('='); - sb.append(((this.vnfNetworks == null)?"<null>":this.vnfNetworks)); + sb.append(((this.vnfNetworks == null)?NULL_STR:this.vnfNetworks)); sb.append(','); sb.append("availabilityZones"); sb.append('='); - sb.append(((this.availabilityZones == null)?"<null>":this.availabilityZones)); + sb.append(((this.availabilityZones == null)?NULL_STR:this.availabilityZones)); sb.append(','); sb.append("vnfVms"); sb.append('='); - sb.append(((this.vnfVms == null)?"<null>":this.vnfVms)); + sb.append(((this.vnfVms == null)?NULL_STR:this.vnfVms)); sb.append(','); if (sb.charAt((sb.length()- 1)) == ',') { sb.setCharAt((sb.length()- 1), ']'); @@ -117,7 +119,7 @@ public class VnfAssignments { if (other == this) { return true; } - if ((other instanceof VnfAssignments) == false) { + if (!(other instanceof VnfAssignments)) { return false; } VnfAssignments rhs = ((VnfAssignments) other); |