Ist es sicher, dieselbe Liste gleichzeitig zu durchlaufen?

Question

Ich habe eine Liste

testList= new ArrayList<String>(); // gets initialized once at the very beginning init(). This list never changes. It's static and final. 

ich eine statische Methode, die, wenn ein Eingabewert überprüft in dieser Liste ist:

public static boolean isInList (String var1) 
      throws InterruptedException { 

     boolean in = false; 

     for (String s : testList) { 

      if (s.equals(var1)) 
      { 
       in = true; 
       break; 
      } 
     } 

     return in; 
    } 

Ich habe eine Menge von Threads, die diese Methode gleichzeitig verwenden und überprüfen, ob Ein bestimmter Wert ist in dieser Liste. Es scheint gut zu funktionieren. Ich bin mir jedoch nicht sicher, ob das sicher ist. Mache ich das richtig? Ist es threadsicher?

Answer 1

Wie lange verwenden können, wie Sie, dass niemand auf der Liste garantieren kann, ist das Schreiben, dann ist es sicher.

Beachten Sie, dass auch wenn die Liste ist static und final, der Code selbst garantiert nicht, dass die Liste nicht verändert wird. Ich empfehle stattdessen, stattdessen Collections.unmodifiableList() zu verwenden, da garantiert wird, dass der Liste kein Element hinzugefügt oder aus der Liste entfernt wird.

---

By the way, können Sie Ihren Code auf diese umschreiben:

public static boolean isInList(String var1) { 
    for (String s : testList) { 
     if (Objects.equals(s, var1)) { 
      return true; 
     } 
    } 
    return false; 
} 

oder nur

testList.contains(var1); 

Answer 2

Es ist Thread-sicher, solange kein Thread die Liste ändert, während andere Threads es lesen.

Wenn Sie Iteratoren über die Liste verwenden, werden sie "fail-fast" (wie in throw ConcurrentModificationException), wenn die Liste unter ihnen geändert wird. Andere Methoden des Zugriffs (d. H. get(n)) werden keine Ausnahme auslösen, können jedoch unerwartete Ergebnisse liefern.

Dies ist alles sehr detailliert im Javadoc für List und ArrayList, die Sie sorgfältig studieren sollten.

Answer 3

ArrayList ist kein Thread-sicheres Objekt. Es mag jetzt für Sie funktionieren, aber im Allgemeinen sollten Sie bei der Arbeit mit Threads sicherstellen, dass Sie thread-sichere Objekte verwenden, die erwartungsgemäß mit Ihren Threads arbeiten.

Sie Collections.synchronizedList()

testList = Collections.synchronizedList(new ArrayList<String>());