Сообщение не получено слушателем kryonet?

Итак, у меня возникла проблема с тестированием моей сети, ниже много кода, но это настолько мало, насколько я мог бы привести пример. Проблема, с которой я сталкиваюсь, заключается в том, что кажется, что слушатель, которого я регистрирую на сервере, и клиент никогда не вызываются, когда я отправляю сообщение.

lock.await(5000, TimeUnit.MILLISECONDS) следует дождаться получения сообщения или тайм-аута в 5 секунд. Однако слушатель должен сказать блокировке продолжить lock.countDown() после того, как она установит переменную response, которая предотвратит сбой теста.

Как вы можете догадаться, этого не происходит, сообщение отправляется, блокировка продолжается из-за тайм-аута, и тест завершается неудачно, потому что response равно нулю.

import com.esotericsoftware.kryo.Kryo;
import com.esotericsoftware.kryonet.Client;
import com.esotericsoftware.kryonet.Connection;
import com.esotericsoftware.kryonet.Listener;
import com.esotericsoftware.kryonet.Server;
import com.esotericsoftware.minlog.Log;
import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;

import java.io.IOException;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;

import static junit.framework.TestCase.fail;

public class TestMessage {
    private Server server;
    private Client client;
    private String response;
    private CountDownLatch lock = new CountDownLatch(1);

    @Before
    public void setUp() {
        Log.set(Log.LEVEL_DEBUG);

        server = new Server();
        try {
            Log.debug("Binding to port: " + 30454 + "... ");
            server.bind(30454);
            Log.debug("Bound to port" + 30454);
        } catch (IOException e) {
            Log.debug("failed");
            Log.error("Unable to bind to port: " + e.getMessage());
            server.stop();
            fail("Unable to start server");
        }
        Kryo kryo = server.getKryo();
        kryo.register(Message.class);
        Log.debug("Adding server listener");
        server.addListener(new TestListener());
        Log.debug("Starting server... ");
        server.start();
        Log.debug("Server started successfully");
    }

    @Test
    public void testPacketSending() throws IOException, InterruptedException {
        client = new Client();
        client.addListener(new TestListener());
        Kryo kryo = client.getKryo();
        kryo.register(Message.class);
        client.start();
        client.connect(5000, "127.0.0.1", 30454);

        client.sendTCP(new Message("RECEIVED"));

        lock.await(5000, TimeUnit.MILLISECONDS);

        Assert.assertNotNull(response);
    }

    private class TestListener extends Listener {
        @Override
        public void received(Connection connection, Object o) {
            Message m = (Message) o;
            response = m.message;
            Log.debug(m.message);
            lock.countDown();
        }
    }

    private class Message {
        String message;

        Message(String message) {
            this.message = message;
        }
    }
}

person Renari    schedule 21.11.2017    source источник


Ответы (1)


Проблема вот в чем:

00:00 ERROR: [kryonet] Error reading TCP from connection: Connection 1
com.esotericsoftware.kryonet.KryoNetException: Error during deserialization.
    at com.esotericsoftware.kryonet.TcpConnection.readObject(TcpConnection.java:141)
    at com.esotericsoftware.kryonet.Server.update(Server.java:205)
    at com.esotericsoftware.kryonet.Server.run(Server.java:372)
    at java.lang.Thread.run(Thread.java:745)
Caused by: com.esotericsoftware.kryo.KryoException: Class cannot be created (non-static member class): org.glytching.sandbox.kryo.TestMessage$MessageA
    at com.esotericsoftware.kryo.Kryo$DefaultInstantiatorStrategy.newInstantiatorOf(Kryo.java:1308)
    at com.esotericsoftware.kryo.Kryo.newInstantiator(Kryo.java:1127)
    at com.esotericsoftware.kryo.Kryo.newInstance(Kryo.java:1136)
    at com.esotericsoftware.kryo.serializers.FieldSerializer.create(FieldSerializer.java:562)
    at com.esotericsoftware.kryo.serializers.FieldSerializer.read(FieldSerializer.java:538)
    at com.esotericsoftware.kryo.Kryo.readClassAndObject(Kryo.java:816)
    at com.esotericsoftware.kryonet.KryoSerialization.read(KryoSerialization.java:55)
    at com.esotericsoftware.kryonet.TcpConnection.readObject(TcpConnection.java:139)
    ... 3 more

Kryonet не может десериализоваться в Message, потому что (а) это не статический внутренний класс и (б) у него нет общедоступного конструктора с нулевым аргументом. Между прочим, есть некоторая здесь информация о том, почему Kryonet не поддержка сериализации нестатических внутренних классов.

Если вы (а) реорганизуете Message в его собственный класс или сделаете его статическим и (б) предоставите ему конструктор с нулевым аргументом, тогда ваш тест будет пройден.

Наименьшее изменение, необходимое для успешного прохождения теста, — заменить это...

private class Message {
    String message;

    Message(String message) {
        this.message = message;
    }
}

... с этим:

private static class Message {
    String message;

    Message() {

    }

    Message(String message) {
        this.message = message;
    }
}

Повторите это:

ниже много кода, но это настолько мало, насколько я мог бы сделать пример

Дело о репродукции в вашем вопросе было более чем адекватным, спасибо.

person glytching    schedule 21.11.2017
comment
Ах, значит, все классы сообщений Kryonet должны быть статическими? - person Renari; 22.11.2017
comment
Только если они являются внутренними классами. - person glytching; 22.11.2017
comment
Хм, спасибо. Кажется, мне не удалось воспроизвести мою проблему, поскольку только в этом примере используются внутренние классы. Может быть, это потому, что в моем классе нет конструктора с 0 аргументами. Как вы получили этот вывод ошибки? Когда я устанавливаю Log.set(Log.LEVEL_ERROR), я не вижу этот вывод на своей консоли. - person Renari; 22.11.2017
comment
Я только что провел тест, используя предоставленный вами код. Я не вносил изменений в то, что вы предоставили. Затем, когда проблема проявилась, я добавил статическую модификацию, ноль и конструктор, и тогда тест прошел. - person glytching; 22.11.2017
comment
Я исправил настоящую проблему, потому что ни у одного из моих классов не было конструкторов с 0 аргументами. - person Renari; 22.11.2017