O problema do código é que se houver mais de uma chamada concorrente no primeiro acesso ao método getInstance()
ele pode criar duas instâncias de Teste.
A solução básica para isso é sincronizar o método:
public static synchronized Teste getInstance()
{
if ( INSTANCE == null )
{
INSTANCE = new Teste();
}
return INSTANCE;
}
O problema desta abordagem é que todas as chamadas estarão sujeitas a bloqueios, deixando a execução geral do programa mais lenta. Imagine um método assim num servidor de aplicação com vários usuários acessando o sistema? É terrível.
Uma solução melhor seria um bloco synchronized
dentro do IF
:
public static Teste getInstance()
{
if ( INSTANCE == null ) {
synchronized (Teste.class) {
INSTANCE = new Teste();
}
}
return INSTANCE;
}
Isso resolve o problema da sincronização em todos os acessos, mas é uma solução "ingênua", pois na verdade voltamos ao problema inicial. Como o IF
não está sincronizado, duas threads
diferentes podem entrar no IF
ao mesmo tempo e, mesmo com a sincronização, elas retornarão instâncias diferentes quando INSTANCE
for null
.
Então, a solução mais "pura" para o singleton pattern seria acrescentar uma verificação dupla, assim:
public static Teste getInstance()
{
if ( INSTANCE == null ) {
synchronized (Teste.class) {
if ( INSTANCE == null ) {
INSTANCE = new Teste();
}
}
}
return INSTANCE;
}
Com essa última abordagem, garantimos que não haverá perda de desempenho por causa de sincronização desnecessária do método inteiro.
Além disso, garantimos que, mesmo que duas chamadas concorrentes ao método entrem dentro do primeiro IF
, daí temos uma nova verificação sincronizada que garante uma única instância de Teste.
Então, no pior caso, se houver duas ou mais chamadas concorrentes no primeiro acesso a getInstance
(quando INSTANCE
ainda é null
), apenas estas primeiras chamadas serão sincronizadas, sendo que após a primeira atribuição de INSTANCE
, nenhuma chamada posterior será sincronizada.
Uma leitura completa sobre isso você encontra no
Head First Design Patterns.
Recomendo que pesquise sobre Singleton
Nota: o construtor private
impede que qualquer outra classe crie inadvertidamente ou arbitrariamente uma instância indesejada de Teste.
Como o construtor está privado ele nunca será chamado.
esta conclusáo nao esta correta, se o construtor é privado, ele só é acessivel dentro da propria classe.Teste
em um ambiente concorrente. Minha pergunta no SOen (em inglês) discute algumas das soluções clássicas para concurrent lazy-loading: stackoverflow.com/q/6189099/664577