Question

我刚开始编写一个简单的多线程应用程序用于学术目的。

我将代码分为3个类，由我的主管指定。

ResponseQueue：包含一个队列和2个插入/删除方法。

发送者：将尝试发送，即将接收消息的接收者。

包含id和响应（String）的Message类。

Junit测试用于查看代码是否有效：＆＃34; TestSendReceiveFromResponseQueue＆＃34;

代码如下：

public class ResponseQueue {

private ResponseQueue() {
}

private BlockingQueue<Message> queue = new SynchronousQueue<Message>();

public synchronized Message poll() {
    Log log = LogFactory.getLog(this.getClass());

    Message response = null;

    try {
        response = queue.take();
        if (response != null) {
            return response;
        } else {
            log.error("null");
        }

    } catch (InterruptedException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

    return null;

}

public synchronized boolean push(Message response) {
    Log log = LogFactory.getLog(this.getClass());

    System.out.println("tryin");
    try {
        queue.put(response);
        return true;
    } catch (InterruptedException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

    return false;

}

private static class LazyHolder {
    private static final ResponseQueue INSTANCE = new ResponseQueue();
}

public static ResponseQueue getInstance() {
    return LazyHolder.INSTANCE;
}

}



public class TestSendReceiveFromResponseQueue extends TestCase{
    ResponseQueue respQueue;

    @Override
    protected void setUp() throws Exception {
        super.setUp();
        respQueue= ResponseQueue.getInstance();
    }

    class Receiver implements Runnable {
        Message message;
        ArrayList<Message> messageList = new ArrayList< Message>();


        @Override
        public void run() {
            message = null;
            while(true){

                message = respQueue.poll();
                messageList.add(message);


            }
            }

    }

    class Sender implements Runnable {

        @Override
        public void run() {
            Message msg = new Message();
        msg.setResponse("test");



        respQueue.push(msg);

        }

    }

    public void test() {
        Receiver r = new Receiver();
        new Thread(r).start();

        Sender s = new Sender();
        new Thread(s).start();


        assertEquals(false, r.messageList.isEmpty());
    }
}

我不知道它为什么总是给出nullpointerException或无限循环。欢迎任何帮助。

编辑：由于有用的答案更正了代码，但它仍然总是给我错误。感谢。

Answer 1

在ResponseQueue.push中：

if(!queue.isEmpty()) {
    done = true;
    return true;
}

你的接收者线程可能会在它被放入队列后立即抓取消息，并且当发送者到达此检查时队列再次为空，导致它循环并重新发布消息。

假设您打算让发件人等待接收邮件，则应删除此检查，并且您应该更换上面的行 queue.add queue.put queue.take，这正是以正确的线程安全方式检查的。

您的接收方也应该使用queue.poll而不是take，while(true)方法将等到消息可用，因此您可以删除循环检查消息是否为空。< / p>

最后你的接收器正在运行一个无限message = null;循环，一旦它收到一条消息，它所做的第一件事就是绕回循环并再次将消息设置为null。这可能是在测试有机会检查消息字段之前发生的，因此测试最终会在无限循环中结束。

一个简单的解决方案是删除queue.take行，因为如果使用public void run() { while(true) { message = respQueue.take(); //This will wait for a message } }则不再需要检查。

System.Text.Encoding v_Unicode = System.Text.Encoding.Unicode;
System.Text.Encoding v_ANSI_Windows_1252 = System.Text.Encoding.GetEncoding(1252);

string v_Content_1252 = v_ANSI_Windows_1252.GetString(System.Text.Encoding.Convert(v_Unicode, v_ANSI_Windows_1252, v_Unicode.GetBytes(v_Content)));
byte[] ansiArray = v_ANSI_Windows_1252.GetBytes(v_Content_1252);

（注意，虽然仍然存在竞争条件，在测试可能看到前一个消息之前可能会收到新消息。此处的解决方法可能是将收到的消息存储在列表中。）

Answer 2

在poll()方法中，您将循环到response == null并且此循环位于synchronized块中。在java中使用synchronized关键字时，将在对象上获取锁定此方法的锁定。仅当线程退出方法时才会释放此锁定。这意味着，当您的Sender调用push()（也是同步方法）时，它将等待锁定进入之前可用。这将导致死锁并导致无限循环。请尝试仅对需要同步的代码使用synchronized(this){...}。

Answer 3

它总是给我假的。感谢。

首先，Receiver.messageList上根本没有同步但你有2个线程访问它（＆＃39;接收器＆＃39;线程和运行测试用例的线程）。如果没有正确的同步，您可能会或可能不会在junit-thread确实事件中看到接收器线程所做的更改。你必须同步它。

除此之外，你可能会遇到时间问题，因为你无法保证接收器的接收能力。当unittest-thread执行assertEquals(false, r.messageList.isEmpty());语句时，线程做了任何事情。最简单的解决方案是浪费一些时间＆＃39;在执行断言之前，虽然这可能不是很强大。

简单发送/接收MultiThread始终给出null异常

3 个答案: