lowered some code smells 92/107292/3
authorIndrijeet kumar <indriku1@in.ibm.com>
Thu, 7 May 2020 03:54:11 +0000 (09:24 +0530)
committerIndrijeet Kumar <indriku1@in.ibm.com>
Thu, 7 May 2020 22:06:47 +0000 (22:06 +0000)
lowered some code smells

Issue-ID: PORTAL-813
Change-Id: Ie1a8136d352d267b8a452a3d33ddd3d544be23c0
Signed-off-by: Indrijeet Kumar <indriku1@in.ibm.com>
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/util/Utils.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/ChartDrillOptions.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/ColFilterList.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/DashboardEditorList.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/DashboardReports.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/DashboardReportsNew.java
ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/xmlobj/DataColumnList.java

index 416fbab..16d6a89 100644 (file)
@@ -246,7 +246,7 @@ public class Utils extends org.onap.portalsdk.analytics.RaptorObject {
         Vector allUsers = AppUtils.getAllUsers(query,param, isAdmin);
                Vector result = new Vector(allUsers.size());
                
-               List<Item> resultLatest = new ArrayList<Item>();
+               List<Item> resultLatest = new ArrayList<>();
 
                for (Iterator iter = allUsers.iterator(); iter.hasNext();) {
                        IdNameValue value = (IdNameValue) iter.next();
@@ -282,7 +282,7 @@ public class Utils extends org.onap.portalsdk.analytics.RaptorObject {
                Vector allRoles = AppUtils.getAllRoles(query, param, isAdmin);
                Vector result = new Vector(allRoles.size());
                
-               List<Item> resultLatest = new ArrayList<Item>();
+               List<Item> resultLatest = new ArrayList<>();
 
                for (Iterator iter = allRoles.iterator(); iter.hasNext();) {
                        IdNameValue value = (IdNameValue) iter.next();
@@ -357,11 +357,14 @@ public class Utils extends org.onap.portalsdk.analytics.RaptorObject {
         int numberOfTimesLooped = 0;
         boolean isSorted = false;
         SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMddHHmmss");
-        Date date1 = null, date2 = null;
+        Date date1 = null;
+               Date date2 = null;
         Date currDate = new Date();
         Object[] matchingfileNamesArr =  matchingFiles.toArray();
         //System.out.println("SIZE " + matchingFiles.size()); 
-        String fileName1 = "", fileName2 ="";
+        String fileName1 = "";
+               String fileName2 ="";
+               
         do {
             isSorted = true;  
         for (int j = 1; j < matchingfileNamesArr.length -  numberOfTimesLooped++; j++) {
index de8593a..dfe9113 100644 (file)
@@ -217,7 +217,7 @@ public class ChartDrillOptions {
      */
     public List<ChartDrillFormfield> getTargetFormfield() {
         if (targetFormfield == null) {
-            targetFormfield = new ArrayList<ChartDrillFormfield>();
+            targetFormfield = new ArrayList<>();
         }
         return this.targetFormfield;
     }
index 6a56d59..18cb007 100644 (file)
@@ -104,7 +104,7 @@ public class ColFilterList {
      */
     public List<ColFilterType> getColFilter() {
         if (colFilter == null) {
-            colFilter = new ArrayList<ColFilterType>();
+            colFilter = new ArrayList<>();
         }
         return this.colFilter;
     }
index b2b105f..577b896 100644 (file)
@@ -104,7 +104,7 @@ public class DashboardEditorList {
      */
     public List<DashboardEditorReport> getEditorList() {
         if (editorList == null) {
-            editorList = new ArrayList<DashboardEditorReport>();
+            editorList = new ArrayList<>();
         }
         return this.editorList;
     }
index b4ef738..8f1f1d2 100644 (file)
@@ -104,7 +104,7 @@ public class DashboardReports {
      */
     public List<Reports> getReportsList() {
         if (reportsList == null) {
-            reportsList = new ArrayList<Reports>();
+            reportsList = new ArrayList<>();
         }
         return this.reportsList;
     }
index 3eadf6f..849aa65 100644 (file)
@@ -110,7 +110,7 @@ public class DashboardReportsNew {
      */
     public List<Reports> getReportsList() {
         if (reportsList == null) {
-            reportsList = new ArrayList<Reports>();
+            reportsList = new ArrayList<>();
         }
         return this.reportsList;
     }
index dffce0e..4d9ba5d 100644 (file)
@@ -106,7 +106,7 @@ public class DataColumnList {
      */
     public List<DataColumnType> getDataColumn() {
         if (dataColumn == null) {
-            dataColumn = new ArrayList<DataColumnType>();
+            dataColumn = new ArrayList<>();
         }
         return this.dataColumn;
     }