Changeset 74 for trunk/CrossPare/src/de/ugoe/cs/cpdp/loader

Timestamp:

05/04/16 11:55:23 (9 years ago)

Author:

sherbold

Message:

• fixed some minor problems found by FindBugs?

Location:

trunk/CrossPare/src/de/ugoe/cs/cpdp/loader

Files:

• AbstractFolderLoader.java (modified) (1 diff)
• DecentDataLoader.java (modified) (3 diffs)
• DecentFolderLoader.java (modified) (1 diff)

trunk/CrossPare/src/de/ugoe/cs/cpdp/loader/AbstractFolderLoader.java

@@ -53 +53 @@
         final File dataDir = new File(path);
         final SingleVersionLoader instancesLoader = getSingleLoader();
-
+        if (dataDir.listFiles() == null) {
+            return versions;
+        }
         for (File projectDir : dataDir.listFiles()) {
             if (projectDir.isDirectory()) {
                 String projectName = projectDir.getName();
-                for (File versionFile : projectDir.listFiles()) {
-                    if (versionFile.isFile() &&
-                        instancesLoader.filenameFilter(versionFile.getName()))
-                    {
-                        String versionName = versionFile.getName();
-                        Instances data = instancesLoader.load(versionFile);
-                        versions.add(new SoftwareVersion(projectName, versionName, data));
+                if (projectDir.listFiles() != null) {
+                    for (File versionFile : projectDir.listFiles()) {
+                        if (versionFile.isFile() &&
+                            instancesLoader.filenameFilter(versionFile.getName()))
+                        {
+                            String versionName = versionFile.getName();
+                            Instances data = instancesLoader.load(versionFile);
+                            versions.add(new SoftwareVersion(projectName, versionName, data));
+                        }
                     }
                 }

trunk/CrossPare/src/de/ugoe/cs/cpdp/loader/DecentDataLoader.java

@@ -163 +163 @@
             catch (FileNotFoundException e) {
                 Console.printerrln("File with path: " + arffLocation + " was not found.");
-                e.printStackTrace();
+                throw new RuntimeException(e);
             }
             catch (IOException e) {
                 Console.printerrln("File with path: " + arffLocation + " cannot be read.");
-                e.printStackTrace();
+                throw new RuntimeException(e);
             }
 
@@ -445 +445 @@
         }
         else {
-
+            Console.printerrln("Could not determine model type, file should end with either .etl or .eol");
+            return null;
         }
 
@@ -453 +454 @@
             Console.printerrln("Parse error occured...");
             for (ParseProblem problem : module.getParseProblems()) {
-                System.err.println(problem.toString());
+                Console.printerrln(problem.toString());
             }
             // System.exit(-1);

trunk/CrossPare/src/de/ugoe/cs/cpdp/loader/DecentFolderLoader.java

@@ -64 +64 @@
          * and searches for "model.decent" files.
          */
+        if (dataDir.listFiles() == null) {
+            return versions;
+        }
         for (File projectDir : dataDir.listFiles()) {
             if (projectDir.isDirectory()) {
                 projectName = projectDir.getName();
-                for (File versionFile : projectDir.listFiles()) {
-                    loadDataFromFile(versionFile, instancesLoader, projectName, versions);
+                if (projectDir.listFiles() != null) {
+                    for (File versionFile : projectDir.listFiles()) {
+                        loadDataFromFile(versionFile, instancesLoader, projectName, versions);
+                    }
                 }
             }