Java多线程优化

Question

在我的程序中，我尝试了解如何使用ExecutorService优化程序。 由于某些原因，它在两个Urls上卡住了一点。 http://sjsu.edu/和https://paypal.com 。 当位于这两个位置上时，它不会继续执行其他URL。

即使两个域的响应速度不够快，其他三个可用线程是否也不应继续运行？

如何以最佳方式解决此问题？

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

        String[] hostList = {"http://crunchify.com", "http://yahoo.com",
            "http://www.ebay.com", "http://google.com",
            "http://www.example.co", "https://paypal.com",
            "http://bing.com/", "http://techcrunch.com/",
            "http://mashable.com/", "http://thenextweb.com/",
            "http://wordpress.com/", "http://cphbusiness.dk/",
            "http://example.com/", "http://sjsu.edu/",
            "http://ebay.co.uk/", "http://google.co.uk/",
            "http://www.wikipedia.org/",
            "http://dr.dk", "http://pol.dk", "https://www.google.dk",
            "http://phoronix.com", "http://www.webupd8.org/",
            "https://studypoint-plaul.rhcloud.com/", "http://stackoverflow.com",
            "http://docs.oracle.com", "https://fronter.com",
            "http://imgur.com/", "http://www.imagemagick.org"
        };

        List<CallableImpl> callList = new ArrayList();
        ExecutorService es = Executors.newFixedThreadPool(4);

        for (String url : hostList) {
            CallableImpl callable = new CallableImpl(url);
            callList.add(callable);
        }
        for (CallableImpl callableImpl : callList) {
            System.out.println("Trying to connect to: " + callableImpl.getUrl());
            Future<String> lol = es.submit(callableImpl);
            System.out.println("status: " + lol.get());
        }

        es.shutdown();

    }
}

我的Callable实现

public class CallableImpl implements Callable<String> {

    private final String url;

    public CallableImpl(String url) {
        this.url = url;
    }

    public String getUrl() {
        return url;
    }

    @Override
    public String call() {
        String result = "Error";

        try {
            URL siteURL = new URL(url);
            HttpURLConnection connection = (HttpURLConnection) siteURL
                    .openConnection();
            connection.setRequestMethod("GET");

            connection.connect();

            int code = connection.getResponseCode();
            if (code == 200) {
                result = "Green";
            }

            if (code == 301) {
                result = "Redirect";
            }

        } catch (IOException e) {
            result = "->Red<-";
        }
        return result;
    }
}

Answer 1

在您的代码中，将Callable Future.get()提交给ExecutorService ，然后立即调用Future.get() ，它将阻塞直到结果准备好（或在运行时引发异常）。

您最好将ExecutorService与CompletionSerivce包装在一起，后者可以在准备好结果后立即提供结果。 并将for循环分为两个循环：一个循环提交所有Callable ，第二个循环检查结果。

ExecutorService es = Executors.newFixedThreadPool(4);
ExecutorCompletionService<String> completionService = new ExecutorCompletionService<>(es);

for (CallableImpl callableImpl : callList) {
        System.out.println("Trying to connect to: " + callableImpl.getUrl());
        completionService.submit(callableImpl);
}
for (int i = 0; i < callList.size(); ++i) {
    completionService.take().get();   //fetch next finished Future and check its result
}

Answer 2

问题

创建后，您get()在Future直接调用get() ，从而阻塞主线程。 因此，您根本没有任何并行调用，这使ExecutorService本质上无用。 您的代码等效于自己简单地调用callableImpl.call() 。

解

如果要继续执行并让每个CallableImpl并行运行，请不要调用get() 。 相反，您可以在es.awaitTermination()之后调用es.shutdown() 。

Answer 3

我建议使用Java 8中添加的CompletableFuture并为其添加回调。

CompletableFuture.supplyAsync(myCallable::call, es)
        .thenAccept(result -> {
            something(result);
        });

我建议您将Callable变成供应商，以简化此过程。

Answer 4

您写道：“它不会继续执行其他URL”-我相信它会执行，但是您的日志消息具有误导性，因为它与实际执行没有紧密联系。 要解决此问题，请执行以下操作：

1. 将System.out.println("Trying to connect to: ")和System.out.println("status: ")移到CallableImpl.call()方法中。
2. 根本不要调用lol.get() 。

这样，您将看到处理每个URL的开始和结束的实际顺序。