m-chrzan.xyz
aboutsummaryrefslogtreecommitdiff
path: root/src/main/java
diff options
context:
space:
mode:
authorMarcin Chrzanowski <marcin.j.chrzanowski@gmail.com>2019-12-30 14:50:42 +0100
committerMarcin Chrzanowski <marcin.j.chrzanowski@gmail.com>2019-12-30 14:50:42 +0100
commit9e047485e990bd5dabbee4f1e9cf58489da964df (patch)
tree1c9f6a0d9e19f083ab8160f18c2d1bc2acb546e3 /src/main/java
parentaaafdf783f765f581e723536c08d0675cc8df133 (diff)
Refactor iterable loops
Diffstat (limited to 'src/main/java')
-rw-r--r--src/main/java/pl/edu/mimuw/cloudatlas/agent/Agent.java11
-rw-r--r--src/main/java/pl/edu/mimuw/cloudatlas/agent/EventBus.java6
-rw-r--r--src/main/java/pl/edu/mimuw/cloudatlas/agent/modules/Stanik.java1
3 files changed, 3 insertions, 15 deletions
diff --git a/src/main/java/pl/edu/mimuw/cloudatlas/agent/Agent.java b/src/main/java/pl/edu/mimuw/cloudatlas/agent/Agent.java
index ce92873..392e0a1 100644
--- a/src/main/java/pl/edu/mimuw/cloudatlas/agent/Agent.java
+++ b/src/main/java/pl/edu/mimuw/cloudatlas/agent/Agent.java
@@ -2,7 +2,6 @@ package pl.edu.mimuw.cloudatlas.agent;
import java.util.ArrayList;
import java.util.HashMap;
-import java.util.Iterator;
import java.util.Map;
import pl.edu.mimuw.cloudatlas.agent.modules.Module;
@@ -27,11 +26,8 @@ public class Agent {
public static HashMap<ModuleType, Executor> initializeExecutors(
HashMap<ModuleType, Module> modules) {
HashMap<ModuleType, Executor> executors = new HashMap<ModuleType, Executor>();
- Iterator it = modules.entrySet().iterator();
- while (it.hasNext()) {
- Map.Entry<ModuleType, Module> moduleEntry =
- (Map.Entry<ModuleType, Module>) it.next();
+ for (Map.Entry<ModuleType, Module> moduleEntry : modules.entrySet()) {
Module module = moduleEntry.getValue();
Executor executor = new Executor(module);
executors.put(moduleEntry.getKey(), executor);
@@ -42,11 +38,8 @@ public class Agent {
public static ArrayList<Thread> initializeExecutorThreads(HashMap<ModuleType, Executor> executors) {
ArrayList<Thread> executorThreads = new ArrayList<Thread>();
- Iterator it = executors.entrySet().iterator();
- while (it.hasNext()) {
- Map.Entry<ModuleType, Executor> executorEntry =
- (Map.Entry<ModuleType, Executor>) it.next();
+ for (Map.Entry<ModuleType, Executor> executorEntry : executors.entrySet()) {
Thread thread = new Thread(executorEntry.getValue());
thread.setDaemon(true);
System.out.println("Initializing executor " + executorEntry.getKey());
diff --git a/src/main/java/pl/edu/mimuw/cloudatlas/agent/EventBus.java b/src/main/java/pl/edu/mimuw/cloudatlas/agent/EventBus.java
index 4f939b6..5b09253 100644
--- a/src/main/java/pl/edu/mimuw/cloudatlas/agent/EventBus.java
+++ b/src/main/java/pl/edu/mimuw/cloudatlas/agent/EventBus.java
@@ -4,7 +4,6 @@ import pl.edu.mimuw.cloudatlas.agent.messages.AgentMessage;
import pl.edu.mimuw.cloudatlas.agent.modules.ModuleType;
import java.util.HashMap;
-import java.util.Iterator;
import java.util.Map;
import java.util.concurrent.LinkedBlockingQueue;
@@ -16,10 +15,7 @@ public class EventBus implements Runnable {
private HashMap<ModuleType, Executor> executors;
void setEventBusReference() {
- Iterator it = this.executors.entrySet().iterator();
- while (it.hasNext()) {
- Map.Entry<ModuleType, Executor> executorEntry =
- (Map.Entry<ModuleType, Executor>) it.next();
+ for (Map.Entry<ModuleType, Executor> executorEntry : this.executors.entrySet()) {
executorEntry.getValue().setEventBus(this);
}
}
diff --git a/src/main/java/pl/edu/mimuw/cloudatlas/agent/modules/Stanik.java b/src/main/java/pl/edu/mimuw/cloudatlas/agent/modules/Stanik.java
index ea893d9..b030901 100644
--- a/src/main/java/pl/edu/mimuw/cloudatlas/agent/modules/Stanik.java
+++ b/src/main/java/pl/edu/mimuw/cloudatlas/agent/modules/Stanik.java
@@ -1,6 +1,5 @@
package pl.edu.mimuw.cloudatlas.agent.modules;
-import java.util.Iterator;
import java.util.HashMap;
import java.util.Map.Entry;