Побочные эффекты использования Multiton Pattern
Мне нужен ваш совет, пересмотр кода или усовершенствование моей реализации многоузловых шаблонов. Я хочу, чтобы поддержка нескольких соединений для сервера mongodb.
public class MongoDatabaseFactory {
private static volatile Map<String, MongoDatabase> connections = new ConcurrentHashMap<String, MongoDatabase>();
public static MongoDatabase getDatabase(Databases database) throws MongoException {
if (null == database) throw new MongoException("Database not found");
if (null == database.name() || database.name().isEmpty()) throw new MongoException("Database not found");
if (!connections.containsKey(database.name()) || null == connections.get(database.name())) {
synchronized (database) {
if (!connections.containsKey(database.name()) || null == connections.get(database.name())) {
connectDB(database);
}
}
}
if (!connections.get(database.name()).isAuthenticated()) {
synchronized (database) {
if (!connections.get(database.name()).isAuthenticated()) {
connectDB(database);
}
}
}
return connections.get(database.name());
}
}
Какова наилучшая практика для многотонного паттерна?
2 ответа
Как говорит Марко Топольник, ваше текущее решение не является поточно-ориентированным.
Я взял это как небольшое упражнение и написал следующий общий потокобезопасный шаблон Multition. Предназначен ли он для эффективной работы со многими потоками, и он подходит в тех случаях, когда создание объекта значения стоит дорого. Обратите внимание, что я не уверен, что в вашем конкретном случае нет более простого решения.
import java.util.concurrent.Callable;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.FutureTask;
public class ThreadSafeMultition <K, V> {
private final ConcurrentHashMap<K, FutureTask<V>> map = new ConcurrentHashMap<K, FutureTask<V>>();
private ValueFactory<K, V> factory;
public ThreadSafeMultition(ValueFactory<K, V> factory) {
this.factory = factory;
}
public V get(K key) throws InterruptedException, ExecutionException {
FutureTask<V> f = map.get(key);
if (f == null) {
f = new FutureTask<V>(new FactoryCall(key));
FutureTask<V> existing = map.putIfAbsent(key, f);
if (existing != null)
f = existing;
else // Item added successfully. Now that exclusiveness is guaranteed, start value creation.
f.run();
}
return f.get();
}
public static interface ValueFactory<K, V> {
public V create(K key) throws Exception;
}
private class FactoryCall implements Callable<V> {
private K key;
public FactoryCall(K key) {
this.key = key;
}
@Override
public V call() throws Exception {
return factory.create(key);
}
}
}
Эта строка не является поточно-ориентированной:
if (!connections.containsKey(database.name()) || null == connections.get(database.name()))
У вас будет гонка данных на хеш-карте, потому что вы не защищаете доступ к карте с помощью блокировки. Вероятно, лучшим решением было бы перенести это в synchronized
блок. Вы не должны беспокоиться о производительности здесь, по крайней мере, без твердого доказательства.