Thread-safe singleton

Path	csiph.com!fu-berlin.de!uni-berlin.de!individual.net!not-for-mail
From	"Christian H. Kuhn" <qno-news@qno.de>
Newsgroups	de.comp.lang.java
Subject	Thread-safe singleton
Date	Wed, 6 Sep 2017 18:29:33 +0200
Lines	63
Message-ID	<f1am3bF8aciU1@mid.individual.net> (permalink)
Mime-Version	1.0
Content-Type	text/plain; charset=utf-8
Content-Transfer-Encoding	8bit
X-Trace	individual.net vo3X06KEL91KPkjz1tSlJwWVJ6Vz5B9yozdYfrGPwoHgHUZ+Y=
Cancel-Lock	sha1:bSkztPAuN2CRXgcyA4OjH5GGogk=
X-Mozilla-News-Host	snews://news.individual.net:563
User-Agent	Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0
Xref	csiph.com de.comp.lang.java:13127

Show key headers only | View raw

Hallo Gemeinde,

Ich dachte, ich hätte das mit der Thread-Sicherheit inzwischen
verstanden. PMD widerspricht mir.

final class QFdsbDatabase {

    private static volatile Connection connection;

    private static final String DATABASE = "database";

    /**
     * Private constructor to avoid instantiation.
     */
    private QFdsbDatabase() {
        // empty
    }

    /**
     * Creates a Connection to a FIDE/DSB database and stores it in a
static field for use. Access data is given by an
     * ini4j ini file.
     *
     * @return Connection to a FIDE/DSB database.
     * @throws QFdsbException
     *             Exception
     */
    static Connection getConnection() throws QFdsbException {

        if (null == connection) {
            synchronized (connection) {
                if (null == connection) {

		[... Daten aus ini4j-File, dataSource erstellen ... ]

                    try (Connection myConnection =
dataSource.getConnection()) {
                        myConnection.setAutoCommit(true);
                        connection = myConnection;
                    } catch (SQLException e) {
                        throw new QFdsbException("Problems while opening
or closing JDBC Connection to player database",
                                e);
                    }
                }
            }
        }
        return connection;
    }

}

Lazy initialization der statischen Variablen, weil ich die geworfenen
Exceptions sehen will, sonst hätte ich einen static block genommen. Mit
einem intitialization-on-demand holder vertage ich das Problem in eine
Unterklasse. Double-checked locking sollte funktionieren, PMD behauptet
das Gegenteil.

Andere Frage: Wird die Connection nicht am Ende schon durch den
impliziten finally-Block geschlossen, bevor sie ausgeliefert wird?

TIA
QNo

Back to de.comp.lang.java | Previous | Next — Next in thread | Find similar

Thread

Thread-safe singleton "Christian H. Kuhn" <qno-news@qno.de> - 2017-09-06 18:29 +0200
  Re: Thread-safe singleton Patrick Roemer <sangamon@netcologne.de> - 2017-09-06 21:06 +0200
  Re: Thread-safe singleton v_borchert@despammed.com (Volker Borchert) - 2017-09-06 20:02 +0000
    Re: Thread-safe singleton Patrick Roemer <sangamon@netcologne.de> - 2017-09-06 23:27 +0200
      Re: Thread-safe singleton v_borchert@despammed.com (Volker Borchert) - 2017-09-07 16:45 +0000
      Re: Thread-safe singleton "Christian H. Kuhn" <qno-news@qno.de> - 2017-09-11 01:08 +0200
        Re: Thread-safe singleton Patrick Roemer <sangamon@netcologne.de> - 2017-09-11 18:14 +0200
  Re: Thread-safe singleton Marcel Mueller <news.5.maazl@spamgourmet.org> - 2017-09-06 23:26 +0200
    Re: Thread-safe singleton Claus Reibenstein <4spamersonly@kabelmail.de> - 2017-09-10 13:52 +0200

csiph-web