Ruby 数据包缓冲和分割

Question

所以，我想这有点宽泛，但我会尽力缩小范围。我有一个服务器（带有 EventMachine），有时数据包会被分割，但有时它们会被缓冲。因此，我尝试创建一个函数来缓冲/取消缓冲它们。我确实做了一些东西，但它没有“按预期”工作。老实说，我什至怀疑我什至可以称其为“几乎不起作用”。

首先，我想指出数据包的结构：

• 数据包的前四个字节是它的 ID，或数据包的名称 (name)。
• 接下来的四个字节是数据包“msg”部分的长度 (len)。
• msg 部分之前的最后四个字节是引用字段，它有多种用途 (ref)。

注意：lenf是len的原始格式，所以是一个字符串，我认为它并不那么重要。

---

缓冲区代码

def split(data)
    if ($packet_buffer != "" && !$packet_buffer.nil?)
        data = $packet_buffer + data
        $packet_buffer = ""
    end
    last = 0
    packets = []
    loop do
        if data[last..-1].length < 8
            $packet_buffer = data[last..-1]
            break
        end
        name = data[last...last+=4]
        lenf = data[last...last+4]

        len = 0
        data[last...last+=4].each_byte {|b| len+=b}

        if !data[last+4..-1].nil? && data[last+4..-1].length < len
            $packet_buffer = data
            break
        end

        ref = data[last...last+=4]
        msg = data[last...last+=len]

        packets << (name << lenf << ref << msg)

        break if data[last..-1].nil?
    end
    packets
end

---

TLDR

如何在 Ruby 中分割缓冲和缓冲区分割数据包/数据（由 EventMachine 传递）？

更新：

我不确定到底出了什么问题，但该方法似乎没有正确分割或缓冲数据包。当它接收少量数据时（我认为这些数据既没有缓冲也没有分割），它工作得很好。

有时它甚至可以成功分割数据包，如果它们被缓冲，但缓冲似乎根本不起作用

---

我相当确定我在这里弄乱了一些“逻辑”部分，但是我只是不明白它是什么是。任何帮助将不胜感激。

谢谢

Answer 1

好吧，这是一个让我跳出来的错误：

len = 0
data[last...last+=4].each_byte {|b| len+=b}

您没有指定存储长度的格式，但如果它是一个小端整数，那么您应该执行类似 len = (len>>8) + 的操作(b<<24) 而不是像您现在所做的那样将所有字节添加在一起。如果 len 始终小于 256，您当前的算法将正常工作。

这里可能隐藏着其他逻辑错误。我不喜欢您使用令人困惑的表达式，例如 data[last..-1].nil?;我会将它们重写为涉及 data.length 和 last 的简单不等式。

如果您想真正清理代码，那么我建议采用一种不同的方法：一次将字节输入到名为 process_byte 的新函数中。该函数将负责跟踪它需要的任何状态信息（例如，它期望接下来接收消息的哪一部分），将字节组装成完整的消息，并将完整的消息传递给更高级别的代码。 process_byte 函数不知道字节是如何打包的，因此您将立即消除程序可能存在的某一类错误。

您可以使用 Ruby 纤维以一种很好的方式实现 process_byte 函数，这样您就可以编写看起来同步的代码（例如 len += get_next_byte() ），但实际上是同步的异步。

Answer 2

好吧，我想出了一种正确的方法，感谢大卫·格雷森的帮助，因为他的回答消除了我的很多困惑/疑虑：

def split(data)
    packets = []
    loop do
        if !$packet_buffer.nil?
            data = $packet_buffer << data
            $packet_buffer = nil
        end

        if data.length < 8
            $packet_buffer = data
            break
        end


        len = calc_Uint32(data[4...8])

        if data.length-12 < len
            $packet_buffer = data
            break
        end

        packets << data[0...12+len]
        data[0...12+len] = ''

        break if data.length == 0
    end
    packets
end #split

我真诚地怀疑有人会发现它有用，因为它不是那么普遍，但我希望最终有人能找到它的用途。