Java 返回值的catch块方法中的递归调用

Java 返回值的catch块方法中的递归调用,java,recursion,methods,return,Java,Recursion,Methods,Return,这是我的问题: 我必须获得到代理服务(这里是activemq)的连接,所以我要这样做: public GenericMessageManager(String url, String producerQueue, String consumerQueue) { Connection connection; try { connection = getConnection(url); session = connection.cre

这是我的问题:

我必须获得到代理服务(这里是activemq)的连接,所以我要这样做:

public GenericMessageManager(String url, String producerQueue,
        String consumerQueue) {

    Connection connection;
    try {
        connection = getConnection(url);
        session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
        producer = createProducer(producerQueue);
        consumer = createConsumer(consumerQueue);
        connection.start();
        logger.info("Connection to broker started");
    } catch (JMSException e) {

    }
}

private Connection getConnection(String url) {
    ActiveMQConnectionFactory connectionFactory = new ActiveMQConnectionFactory(
            url);
    try {
        Connection connection = connectionFactory.createConnection();
        return connection;
    } catch (JMSException e) {
        if (e.getLinkedException() instanceof SocketTimeoutException) {
            logger.warn(url + " not responding, try on localhost");
            getConnection("tcp://127.0.0.1:61616");
        }
    }
    return null;
}
在getConnection()方法中,如果捕获到SocketTimeOutException,我将使用另一个url执行递归调用。这是可行的,但是第一个调用在第二个调用之前返回null,我在connection.createSession(…)上得到一个NPE


我不知道我能做些什么来解决它

我不会通过递归来解决这个问题,因为从直觉上看,这似乎不是一个需要递归的问题。我宁愿配置一个有效主机列表,并按顺序尝试它们,例如:

for (String host : hosts) {
   try {
      Connection c = getConnection(host);
      if (c != null) { 
         return c;
      }
      // log here (not sure I'd return null at all, mind)
   }
   catch (Exception e) {
      // log here...
   }
}
// fail completely
并且在失败的情况下始终抛出异常,而不是混淆异常和空值的含义/处理


上面将连接的建立(和错误处理)与重试机制隔离开来,可以说使其更简单、更容易理解。

我不会通过递归来解决这个问题,因为从直觉上看,这似乎不是一个需要递归的问题。我宁愿配置一个有效主机列表,并按顺序尝试它们,例如:

for (String host : hosts) {
   try {
      Connection c = getConnection(host);
      if (c != null) { 
         return c;
      }
      // log here (not sure I'd return null at all, mind)
   }
   catch (Exception e) {
      // log here...
   }
}
// fail completely
并且在失败的情况下始终抛出异常,而不是混淆异常和空值的含义/处理


以上将连接的建立(和错误处理)与重试机制隔离开来,可以说使其更简单、更容易理解。

也捕获NPE,我会说您忘记了
返回递归调用的结果也捕获NPE,我会说您忘了
返回递归调用的结果