Thready Sicherheit von Karten verwendet privat Innenanwendung scoped bean & nur für Zwecke bereit, nachdem sie einmal

Question

Gebäude Ich habe eine @ApplicationScoped Bohne für alle Benutzer, die die IDS-> Namen & vice versa in Trove & java.util Karten speichert.

Ich baue die Karten nur einmal bei der Konstruktion von Bean oder (im Falle einer manuellen Aktualisierung durch den Website-Administrator).

Innerhalb der Bean-Methoden verwende ich nur die get() mit den Karten, also nicht die Karte zu ändern. Wird dies threadsicher sein, da es nur für fertige Zwecke verwendet wird? Ich teile die Karten nicht mit anderen Beans außerhalb von &, ohne die Karten zu ändern (Hinzufügen/Entfernen von Einträgen), jederzeit in meinem Code.

Ist es in diesem Fall auch notwendig, die Felder endgültig zu machen?

Code Bean wie folgt:

@ApplicationScoped 
@ManagedBean(name="directory", eager=true) 
public class directory { 

    private static TIntObjectHashMap<String> idsToNamesMap; 
    private static TreeMap<String, Integer> namesToIdsMap; 

    @PostConstruct 
    public void buildDirectory(){ 
     // building directory here .... 
    } 

    public String getName(int topicId){ 
     return idsToNamesMap.get(topicId); 
    }  

    public List<Entry<String, Integer>> searchTopicsByName(String query){ 
     return new ArrayList(namesToIdsMap.subMap(query, true, query+"z", true).entrySet()); 
    }   
} 

---

Answer 1

Sie müssen sie in diesem Fall nicht als flüchtig deklarieren oder mit irgendeiner Art von Synchronisation schützen. Solange der Konstruktierungs-Thread sie erstellt und mit dem Hauptspeicher synchronisiert.

Dafür muss der Konstruktier-Thread nur einen einzigen Schreibzugriff auf eine flüchtige Variable ausführen oder eine Synchronisationssperre eingeben/beenden. Dies wird eine Speicherbarriere passieren und alle lokalen Thread-Daten werden im Haupt-Thread sein. Dann ist es für alle anderen Threads sicher, diese Daten zu lesen.

Noch mehr - unnötiger flüchtiger oder Synchronisationsblock - kostet eine schweren Leistungseinbußen - bei jedem Zugriff auf die Variable wird es die Speicher-Schranke passieren - was eine teure Operation ist

Answer 2

Es könnte eine Sichtbarkeit Problem sein, nachdem das Objekt aufgebaut ist. Das heißt, unmittelbar nach Ihren Konstruktoraufrufen erscheinen die Maps möglicherweise für den Thread, der sie gefüllt hat, aber nicht unbedingt für andere Threads, zumindest nicht sofort. Diese Art von Problem wird ausführlich in Kapitel 3 von Java Concurrency in Practice diskutiert. Ich denke jedoch, dass, wenn Sie die Karten als volatile erklären:

private static volatile TIntObjectHashMap<String> idsToNamesMap; 
private static volatile TreeMap<String, Integer> namesToIdsMap; 

Sie OK sein sollte.

aktualisieren

ich gerade etwas realisiert, während wieder auf den Code suchen. Die Karten sind static - warum werden sie in einem Instanzkontext von einem Konstruktor gefüllt? Zunächst einmal ist es für den Leser verwirrend. Zweitens, wenn mehr als eine Instanz des Objekts erstellt wird, haben Sie zusätzliche Schreibvorgänge in den Maps, nicht nur einen, möglicherweise während andere Threads sie lesen.

Sie sollten sie entweder nicht static machen, oder sie in einem static initialization block auffüllen.