Как обеспечить создание только одного XMPPConnection? Использовать синглтоны?

Question

Я занимаюсь созданием веб-службы, которая будет получать запросы на отправку сообщений через XMPP. Однако все сообщения будут отправляться с одной учетной записи (сервер регистрируется и отправляет уведомления пользователям).

В этом заключается проблема - как ее реализовать? Я пытался реализовать класс XMPPConnection как синглтон, но я застрял в передаче аргументов конструкторам, содержащих имя хоста, порт, JID и т.д.

Как я уже читал here, одноэлементно с параметрами не одноэлементно ... Таким образом, я думал о решении его следующим образом (это какой-то завод?):

public class XMPPConnectionSingleton 
{ 
    private volatile static XMPPConnectionSingleton anInstance; 
    private volatile static XMPPConnection connection; 

    public static XMPPConnectionSingleton getInstance() { 
     if(anInstance == null) { 
      synchronized (XMPPConnectionSingleton.class) { 
       if(anInstance == null) 
        anInstance = new XMPPConnectionSingleton(); 
      } 
     } 
     return anInstance; 
    } 

    public void init(String server, int port, String jid, String password, String resource) 
    { 
     ConnectionConfiguration conf = new ConnectionConfiguration(server, port); 
     connection = new XMPPConnection(conf); 
     // logging in, etc. 
    } 
} 

это хороший способ пойти? Или, может быть, лучше сделать класс упаковки для XMPPConnection, приняв конструктор без параметров?

Answer 1

Вы можете использовать одноэлементный, прочитайте это question и соответствующие страницы here и here. Самый чистый - инициализировать в статическом блоке, но это, вероятно, не всегда возможно.

Ужасный, простой подход - синхронизация в одном экземпляре, и все вызовы WS ждут. Это будет отлично работать для низкой емкости. Использование ReentrantLock может несколько усилить эту логику - например, отправив ответ WS-ответа, а не весь HTTP-вызов.

Если вам нужно много одновременных сеансов, попробуйте проверить, может ли сервер входить в систему с несколькими клиентами, а затем инициализировать столько клиентов, сколько захотите, а затем использовать ReentrantLock, чтобы определить, какой из них в настоящее время свободен.

В зависимости от логики в ваших вызовах WS (требуется ли подтверждение того, что сообщение отправлено в XMPP?), Вы можете подумать о очередности вызовов и обслуживании их из другого потока, возвращая WS-вызов до отправки фактического сообщения (Время удержания http обычно 30 секунд, но может быть изменено).

Возможно даже, чтобы каждый сеанс WS подключался и имел свой сеанс в течение всего времени вызова.

Edit: Я предлагаю вам идти с

// Correct lazy initialization in Java 
@ThreadSafe 
class Foo { 
    private static class HelperHolder { 
     public static Helper helper = new Helper(); 
    } 

    public static Helper getHelper() { 
     return HelperHolder.helper; 
    } 
} 

где помощник

1. вспомогательный класс, который вытащить данные входа где-то, обертывания класс клиента XMPP или
2. сам клиентский класс XMPP с жестко запрограммированной информацией для входа в систему.

Затем просто используйте синхронизированный оператор на возвращаемом объекте.

Answer 2

Это звучит для меня как неправильный подход. Почему бы просто не запросить WS-запросы в очереди и попросить один поток прочитать запросы и написать сообщения через xmpp. Будет только одно соединение, но нет необходимости создавать какой-либо пользовательский код для создания одноэлементного соединения вообще.