c# - 关于优化 C# NET 代码块的反馈

Question

我刚刚花了好几个小时阅读 TCP 服务器和我想要实现的协议(protocol)，最终一切正常。我注意到代码看起来完全是胡说八道(这是正确的用法吗？我不是英国人)并希望得到一些关于优化它的反馈，主要是为了重用和可读性。

数据包格式总是 int, int, int, string, string。

try
{
    BinaryReader reader = new BinaryReader(clientStream);
    int packetsize = reader.ReadInt32();
    int requestid = reader.ReadInt32();
    int serverdata = reader.ReadInt32();
    Console.WriteLine("Packet Size: {0} RequestID: {1} ServerData: {2}", packetsize, requestid, serverdata);

    List<byte> str = new List<byte>();
    byte nextByte = reader.ReadByte();

    while (nextByte != 0)
    {
        str.Add(nextByte);
        nextByte = reader.ReadByte();
    }

    // Password Sent to be Authenticated
    string string1 = Encoding.UTF8.GetString(str.ToArray());

    str.Clear();
    nextByte = reader.ReadByte();

    while (nextByte != 0)
    {
        str.Add(nextByte);
        nextByte = reader.ReadByte();
    }

    // NULL string
    string string2 = Encoding.UTF8.GetString(str.ToArray());

    Console.WriteLine("String1: {0} String2: {1}", string1, string2);

    // Reply to Authentication Request
    MemoryStream stream = new MemoryStream();
    BinaryWriter writer = new BinaryWriter(stream);

    writer.Write((int)(1)); // Packet Size
    writer.Write((int)(requestid)); // Mirror RequestID if Authenticated, -1 if Failed
    byte[] buffer = stream.ToArray();

    clientStream.Write(buffer, 0, buffer.Length);
    clientStream.Flush();
}

我还将处理其他格式相同 (int/int/int/str/str) 但值不同的数据包类型。我或许可以创建一个数据包类，但这有点超出了我如何将其应用于此场景的知识范围。如果有任何不同，这就是我正在实现的协议(protocol)。

http://developer.valvesoftware.com/wiki/Source_RCON_Protocol

Answer 1

想法:

• 除了一些整数，你并没有真正使用阅读器；否则，您所需要的只是 ReadByte，您可以从 Stream 中执行此操作，并避免一些间接/混淆
• 手动读取整数以避免字节顺序问题
• 逐字节读取可能很昂贵；如果可能，尝试通过遍历 Read 而不是 ReadByte 来填充缓冲区(或者更确切地说:读取正确数量的数据)
• 如果多个消息通过同一个管道，读到 EOF 可能会失败(要么损坏数据，要么永远阻塞)；您通常需要终止符序列或长度前缀。我更喜欢后者，因为它让您可以使用 Read 而不是 ReadByte
• 我假设在您的示例中是 packetSize；使用它是关键:分隔消息，验证您有完整消息，并拒绝超大数据<
• 考虑async(BeginRead)是否合适——有时合适，有时不合适；请注意，这会使处理变得更加棘手，因为您不能将“使用”与异步一起使用
• 使用 MemoryStream 时，使用 .GetBuffer() 结合 .Length 的开销比使用 .ToArray() 少