Singleton zum Lesen von properties-Datei in eine Java-webapp; richtige Ansatz?

Meine spaghetti-monster verbraucht XML aus verschiedenen SOAP-Dienste, und die URL für jedes service ist hardcoded in der Anwendung. Ich bin in den Prozess der Zerstörung dieser hardcoding, und die Speicherung der URLs in eine properties-Datei.

In Bezug auf das Lesen der Datei mit den Eigenschaften, die ich möchte zu umfassen, die Logik in einem Singleton referenziert werden können, wie gebraucht.

Ändern:

accountLookupURL ="http://prodServer:8080/accountLookupService";

Zu diesem:

accountLookupURL =urlLister.getURL("accountLookup");

Singleton wäre, die in den urlLister.

Ich habe eher scheuen das Singleton-Muster, nur weil ich habe nicht, es zu benutzen, zuvor. Bin ich auf dem richtigen Weg, hier?

Dank!

IVR-Avenger

InformationsquelleAutor IVR Avenger | 2009-12-22
  1. 3

    Haben Sie nicht gesagt, warum Sie brauchen nur eines, was immer es ist, die werden immer die URL. Wenn das nur beinhaltet das Lesen einer properties-Datei, ich glaube nicht, dass Sie brauchen nur ein. Mir scheint, dass zwei threads Lesen die gleichen Eigenschaften, die Datei zur gleichen Zeit ist nicht ein problem überhaupt.

    Es sei denn, Sie denken, dass einige Objekt, das nur liest die properties-Datei einmal und dann werden die Inhalte für die zukünftige Verwendung. Aber dies ist eine web-Anwendung, mit der rechten? Also die Art und Weise zu begegnen, Lesen Sie in den Eigenschaften, wenn die Anwendung startet, und speichern Sie in der Anwendung Kontext. Es gibt nur einen Anwendungskontext, so ist Ihre "einzige" Objekt.

    • Ich bin mir nicht sicher, dass ich werde in der Lage sein, dies zu tun, da ein guter Teil dieser Anwendung ist abstrahiert in eine JAR-Datei, deren Quelle steht uns nicht zu. Was würde dies bedeuten?
    • IVR-Avenger, wie Sie übergeben Sie die Eigenschaften in die Jar-Datei jetzt dann?
    • Ich könnte mir vorstellen, dass die Eigenschaften gelesen werden, genau so, wie Sie beschreiben, vor. Jedoch, dass das Lesen passiert, außerhalb ein Reich, dass ich Zugang habe. Mein code erweitert-Klassen, deren Quelle ich nicht habe, und diese Klassen sind Lesen, spezifischen Eigenschaften. Im Idealfall, ich würde ändern Sie die Basisklasse, so dass diese URLs sind verfügbar, ebenso wie vorhandene Eigenschaften, aber ich habe keinen Zugriff auf die Basisklasse. Noch... 🙂
    InformationsquelleAutor Paul Clapham
  2. 2

    Als alternative, haben Sie verwenden Sie so etwas wie Apache Commons Configuration (oder vielleicht ein anderes Konfigurations-framework)?

    • (+1) es ist gut, nicht das Rad neu erfinden
    • Ja, das ist im Grunde mein Punkt.
    InformationsquelleAutor Pascal Thivent
  3. 1

    Singletons sind geeignet für dieses Szenario, ABER Sie haben zu stellen Sie sicher, Sie tun das singleton-Recht.

    So, zum Beispiel, was Bozhno schlägt ist kein singleton ist, es ist eine hässliche Mischung aus fiesen Statik nicht mockable, nicht leicht prüfbar, nicht injizierbaren, und in der Regel kommt zurück zu beißen Sie in den Arsch.

    Einem akzeptablen singleton ist nur eine Durchschnittliche Klasse mit einer bemerkenswerten Ausnahme, dass gewährleistet ist, entweder durch sich selbst oder durch äußere Fabrik/framework (e.g Spring IoC) existieren in nur einer Instanz. Wenn Sie gehen mit dem ersten Ansatz, sowas

    private MyUberSingletonClass() {
    //..do your constructor stuff, note it's private
}

private static MyUberSingletonClass instance = null;

public static synchronized MyUberSingletonClass instance() {
    if (instance == null) {
        instance = new MyUberSingletonClass();
    }
    return instance;
}

public String getUberUsefulStuff(){
    return "42";
}

    Das ist akzeptabel, wenn Sie nicht wirklich das Gefühl, die Notwendigkeit für eine Fabrik, anders, und nicht mit jedem IoC-container in Ihrer app (gute Idee, denken über die Verwendung einer wenn). Beachten Sie den Unterschied von Bozhno Beispiel: das ist eine gute Vanille-Klasse, wo nur statisch eine Instanz var und eine Methode, um es zurückzugeben. Beachten Sie auch die synchronized-Schlüsselwort erforderlich für die lazy-Initialisierung.

    update: Pascal empfiehlt diese sehr coolen post über einen besseren Weg, um lazy-init singletons in die Kommentare unten: http://crazybob.org/2007/01/lazy-loading-singletons.html

    • Möchten Sie vielleicht, dies zu Lesen: crazybob.org/2007/01/lazy-loading-singletons.html
    • Sehr, sehr schön! Dank Pascal. Viel ordentlicher als die synchronisierte idiom. Ich Frage mich nur, ob dieses Verhalten ist einheitlich für alle JVMs seit 1.3 -- ich habe seit Blackberry devt in letzter Zeit, und wie Sie wissen CLDC ist eine Hündin.
    InformationsquelleAutor Paul Milovanov
  4. 1

    Basierend auf Ihre Vorschläge, und die Tatsache, dass ich nicht glaube, ich habe so viel Zugriff auf diese Anwendung, wie ich gehofft hatte (eine Menge, es ist abstrahiert in kompilierter code), hier ist die Lösung, die ich habe, gekocht. Dies ist, natürlich, ein stub und ausgestaltet werden muss mit besseres exception-handling und ähnliches.

    public class WebServiceURLs {

  private static class WebServiceURLsHolder  
  {  
    public static WebServiceURLs webServiceURLs = new WebServiceURLs(); 
  }

  private Properties webServiceURLs;


  public WebServiceURLs()
  {
    try
    {
      Properties newURLProperties = new Properties();      
      InputStreamReader inputStream = new InputStreamReader(
          FileLoader.class.getClassLoader().getResourceAsStream("../../config/URLs.properties") );
      newURLProperties.load(inputStream);
      webServiceURLs =newURLProperties;
    }
    catch (Exception e)
    {
      webServiceURLs =null;
    }        
  }

  public String getURLFromKey(String urlKey)
  {    
    if (webServiceURLs==null)
      return null;
    else
      return webServiceURLs.getProperty(urlKey);
  }

  public static WebServiceURLs getInstance()
  {
    return WebServiceURLsHolder.webServiceURLs;
  }

}

    Ist dies eine gute Leistung als meine "erste" Singleton?

    Dank,

    IVR-Avenger

    • Das ist nicht gerade ein singleton. Sollten Sie ein Objekt der WebServiceUrls eine Instanz-variable von sich selbst und machen Sie den Konstruktor privat. Die getInstance () - Methode überprüfen soll, ob das Objekt erstellt wurde und erzeugen, wenn Sie nicht vor der Rückkehr es.
    • Okay. Aber es imitiert die Struktur detailliert in Pascal link, vor. Ist das nicht der Ansatz, den ich hätte nehmen sollen?
    InformationsquelleAutor IVR Avenger
  5. 0

    Dennoch nochmals darauf hinweisen, das offensichtliche, Singleton verwendet werden, wenn alle client-code sollte sprechen auf eine einzige Instanz der Klasse. Also, verwenden Sie eine Singleton-IFF Sie sicher, dass Sie würde nicht wollen, laden mehrere properties-Dateien auf einmal. Ich persönlich möchte in der Lage sein, die Funktionalität (laden mehrere properties-Dateien).

    • Mit einer singleton-schließt nicht aus, dass das laden mehrere properties-Dateien auf einmal... ich habe einige code, der verwendet ein singleton zu zentralisieren, den Zugriff auf mehrere properties-Dateien (eigentlich ResourceBundles in meinem Fall). Vielleicht meintest du mehrere properties-Dateien enthalten die Definitionen für den gleichen Namen für die Eigenschaft?
    • Richtig, ich würde nicht erwarten, dass ich hätte zu befürchten, name-clashes, wenn ich geladen habe mehrere properties-Dateien, da das system nicht (unbedingt) annehmen, Sie haben nichts miteinander zu tun.
    • Auch, wenn auch nicht dem Namen Auseinandersetzungen, möchten Sie vielleicht, um eine logische Trennung der Sätze von Eigenschaften (dh der Zugriff auf Sätze von verschiedenen Instanzen). Sie könnte namespace, aber ich wäre nicht verrückt nach, das zu tun.
    InformationsquelleAutor danben
  6. 0

    Singletons sind veränderlich Statik und daher böse. (Vorausgesetzt, eine halbwegs brauchbare definition von "singleton".

    Code, der verwendet die statische (transitive Beziehung), hat Annahmen über so ziemlich alles andere (in diesem Fall ein web-server und dem internet). Veränderlich Statik schlechtes design und schlechtes design lässt viele Aspekte go rotten (Abhängigkeit, Verständlichkeit, testing, Sicherheit, etc).

    Als ein Beispiel, das einzige, was anhalten späten Versionen von JUnit 3 verwendet wird, in einem Sandkasten laden einer Konfigurations-Datei in einem static initialiser. Bei Parametrierung von Oben, hätte es kein Problem.

    • -1 zirkuläre Erklärung, ohne auf die Frage oder die Bereitstellung von Informationen,
    InformationsquelleAutor Tom Hawtin - tackline
Schreibe einen Kommentar