CODEWITHSUNDEEP
javasocketsasynchronousniochannel
jnapor
jnapor

Reputation: 63

Why do I get a CancelledKeyException?

Why do I get an error of Cancelled Key Exception? Can anyone please help me? What part of the code of the client went wrong?

java.nio.channels.CancelledKeyException
at sun.nio.ch.SelectionKeyImpl.ensureValid(SelectionKeyImpl.java:73)
at sun.nio.ch.SelectionKeyImpl.interestOps(SelectionKeyImpl.java:82)
at viaNIO.clientasync2.write(clientasync2.java:130)
at viaNIO.clientasync2.run(clientasync2.java:53)
at java.lang.Thread.run(Thread.java:745)

java.nio.channels.CancelledKeyException
at sun.nio.ch.SelectionKeyImpl.ensureValid(SelectionKeyImpl.java:73)
at sun.nio.ch.SelectionKeyImpl.readyOps(SelectionKeyImpl.java:87)
at java.nio.channels.SelectionKey.isReadable(SelectionKey.java:289)
at viaNIO.clientasync2.run(clientasync2.java:55)
at java.lang.Thread.run(Thread.java:745)

Here is the code for the Client:

        while (!Thread.interrupted()) {

            selector.select();

            Iterator<SelectionKey> keys = selector.selectedKeys().iterator();

            while (keys.hasNext()) {
                SelectionKey key = keys.next();
                keys.remove();

                if (key.isValid()) {
                    if (key.isConnectable()) {
                        System.out.println("Connected to the server");
                        connect(key);
                    }
                    if (key.isWritable()) {
                        System.out.println("Writing to the server");
                        write(key);
                    }
                    if (key.isReadable()) {
                        System.out.println("Reading from the server");
                        read(key);
                    }
                }
            }
        }

Here is the updated Code:

   while (!Thread.interrupted()) {

            selector.select();

            Iterator<SelectionKey> keys = selector.selectedKeys().iterator();

            while (keys.hasNext()) {
                SelectionKey key = keys.next();
                keys.remove();

                if (key.isValid() && key.isConnectable()) {
                    connect(key);
                } else {
                    if (key.isValid() && key.isWritable()) {
                        System.out.println("Writing to the server");
                        write(key);
                    }
                    if (key.isValid() && key.isReadable()) {
                        System.out.println("Reading from the server");
                        read(key);
                    }
                }
            }
        }

This is also my code for the write method

 private void write(SelectionKey key) throws IOException {
    SocketChannel socket = (SocketChannel) key.channel();
    RandomAccessFile aFile = null;
    try {
        File f = new File("D:/test.rar");
        aFile = new RandomAccessFile(f, "r");
        ByteBuffer buffer = ByteBuffer.allocate(300000000);

        FileChannel inChannel = aFile.getChannel();
        while (inChannel.read(buffer) > 0) {
            buffer.flip();
            socket.write(buffer);
            buffer.clear();
        }
        aFile.close();
        socket.close();
        inChannel.close();
        Thread.sleep(1000);

        key.interestOps(SelectionKey.OP_READ);
    } catch (Exception e) {
        e.printStackTrace();
    }
}

The code for the read method

  private void read(SelectionKey key) throws IOException {
    SocketChannel channel = (SocketChannel) key.channel();
    ByteBuffer readBuffer = ByteBuffer.allocate(1000);
    readBuffer.clear();
    int length;
    try {
        length = channel.read(readBuffer);
    } catch (IOException e) {
        System.out.println("Reading problem, closing connection");
        key.cancel();
        channel.close();
        return;
    }
    if (length == -1) {
        System.out.println("Nothing was read from server");
        channel.close();
        key.cancel();
        return;
    }
    readBuffer.flip();
    byte[] buff = new byte[1024];
    readBuffer.get(buff, 0, length);
    System.out.println("Server said: " + new String(buff));
    if (readCnt != 1) {
        key.interestOps(SelectionKey.OP_WRITE);
        readCnt++;
    } else {
        key.cancel();
        close();
    }
}

The code for the connect method

 private void connect(SelectionKey key) throws IOException {
    SocketChannel channel = (SocketChannel) key.channel();
    if (channel.isConnectionPending()) {
        channel.finishConnect();
    }
    channel.configureBlocking(false);
    channel.register(selector, SelectionKey.OP_READ);
}

Upvotes: 4

Views: 13448

Answers (1)

user207421
user207421

Reputation: 311023

Because either connect() or write() closed the channel or cancelled the key before you got to the isReadable() test. You need to keep re-testing isValid(). You also need an else after the isConnectable() block, as OP_CONNECT cannot fire along with either of OP_WRITE or OP_READ:

if (key.isValid() && key.isConnectable()) {
    System.out.println("Connected to the server"); // see below
    connect(key);
} else {
    if (key.isValid() && key.isWritable()) {
        System.out.println("Writing to the server");
        write(key);
    }
    if (key.isValid() && key.isReadable()) {
        System.out.println("Reading from the server");
        read(key);
    }
}

Also the message "Connected to the server" is in the wrong place. You are about to attempt to complete the connection: if and only if finishConnect() returns true are you actually connected.

Upvotes: 2

Related Questions