avoid using forEach

sokra · sokra · commit 142b90e2d062 · 2017-11-24T17:22:45.000+01:00
diff --git a/lib/DependenciesBlock.js b/lib/DependenciesBlock.js
@@ -6,10 +6,6 @@
 
 const DependenciesBlockVariable = require("./DependenciesBlockVariable");
 
-const disconnect = i => i.disconnect();
-
-const unseal = i => i.unseal();
-
 class DependenciesBlock {
 	constructor() {
 		this.dependencies = [];
@@ -42,39 +38,54 @@ class DependenciesBlock {
 	}
 
 	updateHash(hash) {
-		const updateHash = i => i.updateHash(hash);
-
-		this.dependencies.forEach(updateHash);
-		this.blocks.forEach(updateHash);
-		this.variables.forEach(updateHash);
+		for(const dep of this.dependencies)
+			dep.updateHash(hash);
+		for(const block of this.blocks)
+			block.updateHash(hash);
+		for(const variable of this.variables)
+			variable.updateHash(hash);
 	}
 
 	disconnect() {
-		this.dependencies.forEach(disconnect);
-		this.blocks.forEach(disconnect);
-		this.variables.forEach(disconnect);
+		for(const dep of this.dependencies)
+			dep.disconnect();
+		for(const block of this.blocks)
+			block.disconnect();
+		for(const variable of this.variables)
+			variable.disconnect();
 	}
 
 	unseal() {
-		this.blocks.forEach(unseal);
+		for(const block of this.blocks)
+			block.unseal();
 	}
 
 	hasDependencies(filter) {
 		if(filter) {
-			if(this.dependencies.some(filter)) {
-				return true;
+			for(const dep of this.dependencies) {
+				if(filter(dep))
+					return true;
 			}
 		} else {
 			if(this.dependencies.length > 0) {
 				return true;
 			}
 		}
 
-		return this.blocks.concat(this.variables).some(item => item.hasDependencies(filter));
+		for(const block of this.blocks) {
+			if(block.hasDependencies(filter))
+				return true;
+		}
+		for(const variable of this.variables) {
+			if(variable.hasDependencies(filter))
+				return true;
+		}
+		return false;
 	}
 
 	sortItems() {
-		this.blocks.forEach(block => block.sortItems());
+		for(const block of this.blocks)
+			block.sortItems();
 	}
 }
 
