Минусы singleton`ов Android?

Question

[Анатолий Сергеевич]

Добрый день.

Вот singleton который я использую

public class singleton
{
    private static singleton instance;

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

Какие у нее есть минусы, можете дать совет как правильно реализовать singleton`ы

Answer 1

[IceJOKER]

Как минимум сделать конструктор приватным (:

Answer 2

[Денис Загаевский]

Как уже говорили - проблемы при тестировании, сильная связность, сложно поддерживать такой код. Возникает желание хранить там какие-то данные, это неправильно, потому что приложение могут убить, активити восстановится, а данные - нет.
Дальше идут утечки памяти (в случае, если в синглтоне каким-либо образом остаются ссылки на вьюхи или контекст).

В вашей конкретной реализации - постоянные накладные расходы при каждом доступе к инстансу (синхронизация). Лучше сделать double-check или instance holder.
Тут вот подробно всё расписано.

Answer 3

[mitaichik]

Забыли еще добавить и про возможные утечки памяти, если вы там контексты передаете.

Вообще, и лучше, и удобнее использовать DI, для андройда популярен Dagger 2 от гугла https://google.github.io/dagger/