Question

@Path("/getVersion")
    @POST
    @Produces(MediaType.APPLICATION_JSON)
    public List getVersion(String getVersionJson) throws InterruptedException {
        List outputList = new ArrayList();
        try {
            JSONArray jsonArr = new JSONArray(getVersionJson);

            for (int j = 0; j < jsonArr.length(); j++) {
                JSONObject jsonObj = jsonArr.getJSONObject(j);
                String ip = jsonObj.getString("ipaddress");
                String username = jsonObj.getString("username");
                String password = jsonObj.getString("password");

                new Thread() {
                    public void run() {
                        //outputList.add(ip);
                        final String connectionStatus = getSSHConnection(ip, username, password);
                        // outputList.add(connectionStatus);
                        if (connectionStatus.equals("Connected")) {
                            // outputList.add(connectionStatus);
                            //Version Check
                            expect.send("bwshowver" + "\n");
                            if (expect.expect("$") > -1) {
                                String contt = "";
                                contt = (expect.before);
                                if (contt != null && contt != "") {

                                    contt = contt.replaceAll("\n+", "\n");
                                    contt = contt.replaceAll(" +", " ");

                                    String splitter[] = contt.split("\n");

                                    for (int i = 0; i < splitter.length; i++) {
                                        if (splitter[i].contains("Patches")) {
                                            final String patches = splitter[i];
                                        }
                                        //version
                                        if (splitter[i].contains("version")) {
                                            // final String version = splitter[i];
                                            outputList.add(splitter[i]);

                                        }

                                    }

                                } else {
                                    final String v1 = "Error in version check";
                                    System.out.println("Error in version check");
                                }
                            }
                        } else {
                            outputList.add(connectionStatus);
                        }
                    }
                }.start();
            }
        } catch (Exception e) {
            final String v = "Error";
            //     logger.error("Exception in getVersion Function-ServService Class: " + e.getMessage());
        } finally {
            stopSSH();
        }
        Thread.sleep(20000);
        //outputList.add("ffk");
        // outputList.add("f");
        return outputList;
    }

我使用Threads创建了一个方法。在这个方法中，我发送一个由json对象组成的jsonarray。以前我创建了这个方法没有线程，它正常工作。现在将它改成线程后，我在不同的时间得到不同的结果。你能告诉我哪里出错了吗？

Answer 1

您可以在上面的代码中改进3件主要内容：

从多个线程修改线程安全集合时使用它们。如果该集合没有明确提及它是线程安全的，那么您需要自己获取锁定或切换到线程安全的集合
使用Thread.sleep（）进行同步通常是一个坏主意。您应该使用更精确的同步方法来避免不稳定的行为。我在这里选择使用CountDownLatch，您可以使用其他同步工具。
线程内部抛出的异常不会传播到主线程，但可能只是在该线程内部冒出来并且永远不会被注意到。您希望能够注意到runnable中抛出的异常并在最后处理它。我还将修改代码，向您展示如何处理异常的粗略指南。

以简单的格式回答您的问题，您可以希望对代码进行调整：

public List getVersion(String getVersionJson)
        throws InterruptedException, IOException {
    // You must use a thread-safe list.
    final List outputList = Collections.synchronizedList(new ArrayList());
    // This reference is used to keep track of exceptions thrown
    // in the runnable.
    final AtomicReference<Exception> exceptionReference =
        new AtomicReference(null);
    // This latch acts as a synchronization barrier so that all threads
    // must have put data into outputList before the code will go
    // past the latch.await() call
    final CountDownLatch latch = new CountDownLatch(jsonArr.length());
    for (int j = 0; j < jsonArr.length(); j++) {
        new Thread() {
            public void run() {
                try {   
                     // Do network I/O stuff
                     outputList.add(resultFromNetworkStuff);
                     latch.countDown();
                } catch (IOException e) {
                     // Set the exception for later retrieval
                     exceptionReference.compareAndSet(null, e);
                }
            }                        
        }.start();
    }

    // Wait for all the threads to finish before continuing.
    try {
        latch.await();
    } finally {
        // Even if we're interrupted, we want to close the connection.
        closeSSH();
    }
    if (exceptionRef.get() != null) {
        // Throw any exceptions we see in our threads.
        throw exceptionRef.get();
    }
    return outputList;
}

Answer 2

您将主线程置于休眠状态一段时间。在此期间，多个其他线程并行将数据推送到共享列表中。根据定义，这不起作用。

当你的其他线程在20秒内完成时，一切正常。但如果没有，你的方法确实返回到目前为止收集的其他线程。更糟糕的是，其他线程不断更新您已返回的列表。

你必须确保内部线程

没有以不受控制的方式访问该结果列表（相互覆盖！）
都是在从此方法返回之前完成的。

不要在代码中应用某些概念，因为您在不了解自己在做什么的情况下听说过它。你不太了解如何使用java线程以及使用它们意味着什么。因此研究这些东西而不是盲目地将它推入你的代码中。通过简单的示例了解如何使用线程;而不是像这样把它们拉进你的生产代码。

然后：你的catch块丢失了错误信息！这总是一个坏主意。将错误信息作为结果列表中的字符串推送的方法也是如此。

除此之外：阅读Robert Martin的清洁代码。您的代码将从中获益远远超过您无缘无故地添加线程。

在线程中运行时，在不同时间获得不同的结果

2 个答案: