这是在 MINA 中编写 ProtocolDecoder 的正确方法吗？

Question

public class CustomProtocolDecoder extends CumulativeProtocolDecoder{
    byte currentCmd = -1;
    int currentSize = -1;
    boolean isFirst = false;
    @Override
    protected boolean doDecode(IoSession is, ByteBuffer bb, ProtocolDecoderOutput pdo) throws Exception {
            if(currentCmd == -1)
            {
                currentCmd = bb.get();
                currentSize = Packet.getSize(currentCmd);
                isFirst = true;
            }
            while(bb.remaining() > 0)
            {
                if(!isFirst)
                {
                    currentCmd = bb.get();
                    currentSize = Packet.getSize(currentCmd);
                }
                else
                    isFirst = false;
                //System.err.println(currentCmd + " " + bb.remaining() + " " + currentSize);
                if(bb.remaining() >= currentSize - 1)
                {
                    Packet p = PacketDecoder.decodePacket(bb, currentCmd);
                    pdo.write(p);
                }
                else
                {
                    bb.flip();
                    return false;
                }
            }
            if(bb.remaining() == 0)
                return true;
            else
                return false;
    }
}

有人看到这段代码有什么问题吗？当一次收到大量数据包时，即使只有一个客户端连接，其中一个数据包也可能在最后被切断（例如，12 字节而不是 15 字节），这显然是不好的。

Answer 1

在示例的帮助下弄清楚了——我对真假感到困惑，并且没有意识到我应该跟踪输入缓冲区的位置。最重要的是，我不知道我不需要 while 循环。谢谢！

    protected boolean doDecode(IoSession is, ByteBuffer bb, ProtocolDecoderOutput pdo) throws Exception {
    int start = bb.position();
    currentCmd = bb.get();
    currentSize = Packet.getSize(currentCmd);
    //System.err.println(currentCmd + " " + bb.remaining() + " " + currentSize);
    if(bb.remaining() >= currentSize - 1)
    {
        Packet p = PacketDecoder.decodePacket(bb, currentCmd);
        pdo.write(p);
        if(bb.remaining() == 0)
            return false;
        else
            return true;
    }
    else
    {
        bb.position(start);
        return false;
    }
}

Answer 2

我发现有点难以理解您要在这里解码的协议。那里肯定看起来有点困惑；）

您是否正在编写一些需要在同一连接上发出许多请求的内容？如果是这样，那就太好了，这就是 Mina 所擅长的......

通常，我希望 MINA 解码器检查它是否收到完整的消息，然后，如果没有，则将 IoBuffer 的指针返回到它所在的位置在方法的开始处。

通常，完整的消息将由分隔符或消息开头的长度字段确定。

api 文档中提供的示例非常好。
它正在寻找回车+换行符的分隔符：

http://mina.apache.org/report/trunk/apidocs/org/apache/mina/filter/codec/CumulativeProtocolDecoder.html

hth