apply codefactor requests

Author: guilyx

jupyddl/data_analyst.py

@@ -38,8 +38,8 @@ def __get_all_pddl_from_data(self):
 
     def __plot_data(self, times, total_nodes, plot_title):
         data = dict()
-        for i in range(len(total_nodes)):
-            data[total_nodes[i]] = times[i]
+        for i, val in enumerate(total_nodes):
+            data[val] = times[i]
         nodes_sorted = sorted(list(data.keys()))
         times_y = []
         for node_opened in nodes_sorted:
@@ -105,8 +105,7 @@ def __gather_data_astar(
             return [0], [0], has_multiple_files_tested
         if path:
             return [total_time], [opened_nodes], has_multiple_files_tested
-        else:
-            return [0], [0], has_multiple_files_tested
+        return [0], [0], has_multiple_files_tested
 
     def plot_astar_data(self, heuristic_key="goal_count", domain="", problem=""):
         if bool(not problem) != bool(not domain):
@@ -149,8 +148,7 @@ def __gather_data_bfs(self, domain_path="", problem_path=""):
         path, total_time, opened_nodes = apla.breadth_first_search()
         if path:
             return [total_time], [opened_nodes], has_multiple_files_tested
-        else:
-            return [0], [0], has_multiple_files_tested
+        return [0], [0], has_multiple_files_tested
 
     def plot_bfs(self, domain="", problem=""):
         title = "BFS Statistics"
@@ -193,8 +191,7 @@ def __gather_data_dfs(self, domain_path="", problem_path=""):
         path, total_time, opened_nodes = apla.depth_first_search()
         if path:
             return [total_time], [opened_nodes], has_multiple_files_tested
-        else:
-            return [0], [0], has_multiple_files_tested
+        return [0], [0], has_multiple_files_tested
 
     def plot_dfs(self, problem="", domain=""):
         title = "DFS Statistics"
@@ -237,8 +234,7 @@ def __gather_data_dijkstra(self, domain_path="", problem_path=""):
         path, total_time, opened_nodes = apla.dijktra_best_first_search()
         if path:
             return [total_time], [opened_nodes], has_multiple_files_tested
-        else:
-            return [0], [0], has_multiple_files_tested
+        return [0], [0], has_multiple_files_tested
 
     def plot_dijkstra(self, problem="", domain=""):
         title = "Dijkstra Statistics"