Make Storage to RandomAccessCollection

.swiftlint.yml

@@ -4,6 +4,7 @@ excluded:
 
 disabled_rules:
   - multiple_closures_with_trailing_closure
+  - identifier_name
 
 trailing_whitespace:
   ignores_empty_lines: true

Tests/StorageTests.swift

@@ -17,19 +17,19 @@ final class StorageTests: XCTestCase {
         let key1 = storage.add(f1)
         let key2 = storage.add(f2)
 
-        storage.forEach { f in f() }
+        for f in storage { f() }
 
         XCTAssertEqual(value, 11)
 
         storage.remove(for: key1)
 
-        storage.forEach { f in f() }
+        for f in storage { f() }
 
         XCTAssertEqual(value, 21)
 
         storage.remove(for: key2)
 
-        storage.forEach { f in f() }
+        for f in storage { f() }
 
         XCTAssertEqual(value, 21)
     }

VueFlux/Internal/Dispatcher.swift

@@ -18,7 +18,7 @@ struct Dispatcher<State: VueFlux.State> {
     ///   - action: An Action to be dispatch.
     func dispatch(action: State.Action) {
         observers.synchronized { observers in
-            observers.forEach { observer in
+            for observer in observers {
                 observer(action)
             }
         }

VueFlux/Storage.swift

@@ -28,15 +28,23 @@ public struct Storage<Element> {
             buffer.remove(at: index)
         }
     }
+}
+
+extension Storage: RandomAccessCollection {
+    public var startIndex: Int {
+        return buffer.startIndex
+    }
 
-    /// Calls the given function on each element in the collection.
-    ///
-    /// - Parameters:
-    ///   - body: A function that takes an element of the collection as a parameter.
-    public func forEach(_ body: (Element) -> Void) {
-        for (_, element) in buffer {
-            body(element)
-        }
+    public var endIndex: Int {
+        return buffer.endIndex
+    }
+
+    public func index(after i: Int) -> Int {
+        return i + 1
+    }
+
+    public subscript(position: Int) -> Element {
+        return buffer[position].element
     }
 }
 

VueFluxReactive/Internal/Subject.swift

@@ -29,7 +29,9 @@ final class Subject<Value>: Subscribable {
     ///   - value: Value to send to all observers.
     func send(value: Value) {
         observers.synchronized { observers in
-            observers.forEach { $0(value) }
+            for observer in observers {
+                observer(value)
+            }
         }
     }
 }