Remove element in the set while iterating

user_18636680 0
Viewed 112

I have the following scenario: In the details variable, I need to remove a course from the mutable set of courses, for each student based on a condition.

If I remove as I have done below, I get concurrent modification exception, because I am modifying the variable while iterating through it.

How can I iterate the map in reverse so that it doesn't throw the deadlock, or is there a better way to handle this scenario ?

val details = MutableMap<Student, MutableSet<CourseDetail>>

details?.forEach { student, courses ->
    courses?.forEach { course ->
       if (b?.contains(course) == false) {
           details[student]?.remove(course)
    }
  }
}
kotlin
2 Answers

It sounds like you just want

for (courses in details.values) {
  courses.retainAll(b)
}

(Notably, you should also avoid null collections so you don't have to scatter ?. everywhere.)

user_8697360

You can use ConcurrentHashMap and ConcurrentHashSet it provide thread safety or

simply use iterator to remove element while iterating

details?.forEach { student, courses ->
    Iterator<CourseDetail> itr = courses.iterator();
    while (itr.hasNext())
        if (b?.contains(itr.next()) == false) itr.remove();
 }
user_181260010
Related