Один синглтон для каждого (общего типа)

Я не очень хорошо знаю Java, поэтому хотел бы спросить, реализована ли эта реализация. (Безопасность резьбы не имеет значения). Я хочу, чтобы для каждого класса T был выбран единственный уникальный синглтон.Один синглтон для каждого (общего типа)

public class MockDatabase<T> { 
    private MockDatabase() {} 

    private static Map<String, MockDatabase> singletonHolder = new HashMap<String, MockDatabase>(); 

    public static <T> MockDatabase<T> getInstance(Class<T> clazz) throws InstantiationException, IllegalAccessException { 
     MockDatabase<T> singleton = (MockDatabase<T>)singletonHolder.get(clazz.getName()); 
     if (singleton == null) { 
      singleton = new MockDatabase<T>(); 
      singletonHolder.put(clazz.getName(), singleton); 
     } 

     return singleton; 
    } 
}

источник

2017-02-06 EralpB

Почему вы используете ключ 'clazz.getName()', а не 'clazz'? –

Вы не должны использовать необработанные типы ('Map ' использует raw 'MockDatabase'). Кроме того, вы не должны использовать одну и ту же букву типа для параметра класса 'T' и общий параметр метода' ', поскольку это создает иллюзию того, что они одного типа. – RealSkeptic

Почему вы не добавляете в кеш одиночный экземпляр, который вы создаете, когда у держателя нет? – davidxxx

Простое решение, которое также поточно является использование ClassValue

public class MockDatabase<T> { 
    private static final ClassValue<MockDatabase> cache = new ClassValue<>() { 
     protected MockDatabase computerValue(Class<?> clazz) { 
      return new MockDatabase(); 
     } 
    } 
    public static <T> MockDatabase<T> getInstance(Class<T> clazz) { 
     return (MockDatabase<T>) cache.get(clazz); 
    } 
}

Cl assValue вернет тот же объект значения для любого класса и будет потокобезопасным.

Примечание: ClassValue очищается, когда класс больше не ссылается.

источник

2017-02-06 14:08:23

Это выглядит намного чище! – EralpB

@ EralpB, некоторые разработчики Java знают об этом классе, даже если он находится в java.lang –

if (singleton == null) { 
     singleton = new MockDatabase<T>(); 
}

Изменение выше кода

if (singleton == null) { 
     singleton = new MockDatabase<T>(); 
     singletonHolder.put(clazz.getName(),singleton); 
}

Другой мудрый вы получите новый объект каждый раз, когда

источник

2017-02-06 13:57:09 mahesh

Это правда, спасибо, что указали! – EralpB

Один синглтон для каждого (общего типа)

ответ

Смежные вопросы