RestTemplate 应该是静态的全局声明吗?

我在我的代码中使用Java Callable Future。以下是我的主要代码,它使用future和可调用性 -

public class TimeoutThread {

    public static void main(String[] args) throws Exception {

        ExecutorService executor = Executors.newFixedThreadPool(5);
        Future<String> future = executor.submit(new Task());

        try {
            System.out.println("Started..");
            System.out.println(future.get(3, TimeUnit.SECONDS));
            System.out.println("Finished!");
        } catch (TimeoutException e) {
            System.out.println("Terminated!");
        }

        executor.shutdownNow();
    }
}

下面是我的类,它实现了可调用接口,我需要根据我们拥有的主机名生成URL,然后使用对服务器进行调用。如果第一个主机名中有任何异常,那么我将为另一个主机名生成URL,我将尝试进行调用。TaskRestTemplate

class Task implements Callable<String> {
    private static RestTemplate restTemplate = new RestTemplate();

    @Override
    public String call() throws Exception {

    //.. some code

    for(String hostname : hostnames)  {
            if(hostname == null) {
                continue;
            }
            try {
                String url = generateURL(hostname);         
                response = restTemplate.getForObject(url, String.class);

                // make a response and then break
                break;

            } catch (Exception ex) {
                ex.printStackTrace(); // use logger
            }
        }
    }
}

所以我的问题应该声明为静态全局变量吗?或者在这种情况下它不应该是静态的?RestTemplate


答案 1

无论哪种方式或实例都无关紧要。static

RestTemplate用于发出 HTTP 请求的方法是线程安全的,因此,每个实例都有一个实例还是所有实例都有一个共享实例是无关紧要的(垃圾回收除外)。RestTemplateTaskTask

就个人而言,我会创建类外部,并将其作为参数传递给构造函数。(尽可能使用控制反转。RestTemplateTaskTask


答案 2

从并发的角度来看,这并不重要。 是线程安全的,因此单个实例或多个实例与程序的正常运行无关。RestTemplate

但您可能需要考虑,如下所示。AsyncRestTemplate

此外,正如其他人所提到的,您应该考虑使用 IoC 方法将 REST 客户端的创建与其使用分开。Martin Fowler的这篇文章是关于这个话题的开创性讨论。