Merge pull request #212 from MaxKellermann/Dispatcher_containsKey

bailuk · web-flow · commit ec641ba4f36e · 2026-03-18T06:39:55.000+01:00
lib/Dispatcher: eliminate the HashMap.containsKey() call
diff --git a/aat-lib/src/main/java/ch/bailu/aat_lib/dispatcher/Dispatcher.kt b/aat-lib/src/main/java/ch/bailu/aat_lib/dispatcher/Dispatcher.kt
@@ -29,10 +29,12 @@ class Dispatcher : DispatcherInterface {
     }
 
     private fun getTargetList(iid: Int): TargetList {
-        if (!targets.containsKey(iid)) {
-            targets[iid] = TargetList()
+        var list = targets[iid]
+        if (list == null) {
+            list = TargetList()
+            targets[iid] = list
         }
-        return targets[iid]!!
+        return list
     }
 
     override fun addSource(source: SourceInterface) {

Original file line number	Diff line number	Diff line change
`@@ -29,10 +29,12 @@ class Dispatcher : DispatcherInterface {`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`private fun getTargetList(iid: Int): TargetList {`
`32`		`- if (!targets.containsKey(iid)) {`
`33`		`- targets[iid] = TargetList()`
	`32`	`+ var list = targets[iid]`
	`33`	`+ if (list == null) {`
	`34`	`+ list = TargetList()`
	`35`	`+ targets[iid] = list`
`34`	`36`	`}`
`35`		`- return targets[iid]!!`
	`37`	`+ return list`
`36`	`38`	`}`
`37`	`39`
`38`	`40`	`override fun addSource(source: SourceInterface) {`