Обработчик пакетов на основе TCP NetworkStream

@PVD · Регистрация: 01.06.2014

Студворк — интернет-сервис помощи студентам

Собственно, хочу сделать так, чтобы данные читались из NetworkStream и записывались MemoryStream.
Проблема в том, что я не могу узнать сколько доступно байт для чтения из NetworkStream и поэтому приходится выделить буфер типа byte[] (так как количество считанных может быть больше чем доступная память для MemoryStream).
Собственно, какое решение посоветуете?

Не морочить мозги и выделять буфер, так как для современных ОЗУ выделение буфера в несколько килобайт ничтожно.
Сделать буфер ничтожно маленьким и использовать его многократно.
Сделать общий буфер и писать в него.
Использовать некоторую функцию для записи из NetworkStream в MemoryStream напрямую. (которая увеличит размер буфера MemoryStream, при необходимости).

Что я по поводу этого думаю:

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

Добавлено через 3 минуты
Или, просто считываем из NetworkStream записывая в буфер MemoryStream, если буфер MemoryStream закончился, разширяем его? Хотя, тогда уже проще будет использовать дополнительный буфер и использовать MemoryStream.Write() ?

Добавлено через 1 час 26 минут
Собственно написал вот так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
using System;
using System.Net;
using System.Net.Sockets;
using System.IO;
using System.Collections.Generic;
 
namespace PacketManager
{
    /// <summary>
    /// Базовый интерфейс пакетов, которыми оперирует PacketManager</see>.
    /// </summary>
    public interface IPacket
    {
        /// <summary>
        /// При определении, возвращает размер в байтах, который необходим, чтобы сохранить пакет.
        /// </summary>
        ushort Size { get; }
        /// <summary>
        /// При определении, задает имеют ли фиксированный размер все пакеты данного типа
        /// </summary>
        bool FixedSize { get; }
        /// <summary>
        /// При переопределении, записывает данные пакета в бинарный поток.
        /// </summary>
        /// <param name="stream">Бинарный поток.</param>
        void Write(BinaryWriter stream);
        /// <summary>
        /// При переопределении, считывает данные пакета из бинарного потока.
        /// </summary>
        /// <param name="stream">Бинарный поток.</param>
        /// <param name="readCount">Количество данных, которое должно быть считано.</param>
        void Read(BinaryReader stream, int readCount);
    }
 
    /// <summary>
    /// Типизирует ошибки возникающие, при упаковке и распаковке пакетов.
    /// </summary>
    public class IPacketException : Exception
        {
            public IPacketException() : base() { }
            public IPacketException(string message) : base(message) { }
            public IPacketException(string message, Exception innerException) : base(message, innerException) { }
        }
 
    /// <summary>
    /// Класс для обмена сообщениями по сети.
    /// </summary>
    class PacketManager
    {
        /// <summary>
        /// Общий тип исключений генерируемых в PacketManager.
        /// </summary>
        public class PacketManagerException : Exception
        {
            public PacketManagerException() : base() { }
            public PacketManagerException(string message) : base(message) { }
            public PacketManagerException(string message, Exception innerException) : base(message, innerException) { }
        }
        /// <summary>
        /// Поток TCP соединения.
        /// </summary>
        NetworkStream Stream;
        /// <summary>
        /// Буфер записи в NetworkStream.
        /// </summary>
        MemoryStream StreamWriterBuffer;
        /// <summary>
        /// Буфер чтения из NetworkStream.
        /// </summary>
        MemoryStream StreamReaderBuffer;
        /// <summary>
        /// Бинарная запись в NetworkStream.
        /// </summary>
        BinaryWriter StreamWriter;
        /// <summary>
        /// Бинарное чтение из NetworkStream.
        /// </summary>
        BinaryReader StreamReader;
        /// <summary>
        /// Общий список всех пакетов для использования.
        /// </summary>
        public static readonly List<Type> PacketTypes;
        /// <summary>
        /// Образует список всех пакетов для использования.
        /// </summary>
        static PacketManager()
        {
            PacketTypes = new List<Type>();
        }
 
        public void SendPacket(IPacket packet)
        {
            var packetID = PacketTypes.IndexOf(packet.GetType());
            if (packetID > -1)
            {
                StreamWriter.Write((byte)packetID);
                if (!packet.FixedSize)
                    StreamWriter.Write(packet.Size);
                packet.Write(StreamWriter);
                try
                {
                    Stream.Write(StreamWriterBuffer.GetBuffer(), 0, (int)StreamWriterBuffer.Position);
                }
                catch (IOException ex)
                {
                    throw new PacketManagerException("IO exception while writing to NetworkStream occured", ex);
                }
                StreamWriterBuffer.Seek(0, SeekOrigin.Begin);
            }
            else
                throw new PacketManagerException(String.Format("Tried to send packet \"{0}\" which not listed in PacketTypes", packet));
        }
 
        readonly byte[] _tempBuffer = new byte[4096];
        readonly IPacket[] _packetBuffer = new IPacket[64];
 
        public IPacket[] RecievePackets(out int recievedCount)
        {
            recievedCount = 0;
            if (Stream.DataAvailable) // Если есть данные для чтения из NetworkStream.
            {
                try
                {
                    int readCount = Stream.Read(_tempBuffer, 0, 4096); // Читаем данные из NetworkStream.
                    if (readCount > 0) // Данные из NetworkStream прочитаны.
                    {
                        StreamReaderBuffer.Write(_tempBuffer, 0, readCount); // Запоминаем прочитанные данные
                        StreamReaderBuffer.Seek(-readCount, SeekOrigin.Current);
                        for (; (StreamReaderBuffer.Position - StreamReaderBuffer.Length > 0) && (recievedCount < 64); recievedCount++) // Пока есть данные и количество полученных пакетов < 64
                        {
                            byte packetID = StreamReader.ReadByte(); // Узнаем ID пакета.
                            if (packetID < PacketTypes.Count) // Если известный ID пакета.
                            {
                                _packetBuffer[recievedCount] = Activator.CreateInstance(PacketTypes[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                if (_packetBuffer[recievedCount].FixedSize) // Если пакет данного типа имеет фиксированную длину.
                                {
                                    if(_packetBuffer[recievedCount].Size > 0) // Если фиксированный размер пакета больше нуля.
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= _packetBuffer[recievedCount].Size) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount].Read(StreamReader, _packetBuffer[recievedCount].Size); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch(IPacketException ex) // Если при распаковке пакетов возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки пакета, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Пакет прочитан (Размер фиксированный и он равен нулю)
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                        {
                                            recievedCount++;
                                            StreamReaderBuffer.SetLength(0);
                                            break;
                                        }
                                    }
                                } 
                                else // Пакет имеет переменную длину.
                                {
                                    if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= 2) // Если можем считать размер пакета.
                                    {
                                        ushort packetSize = StreamReader.ReadUInt16(); // Размер пакета.
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= packetSize) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount] = Activator.CreateInstance(PacketTypes[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                                _packetBuffer[recievedCount].Read(StreamReader, packetSize); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch (IPacketException ex) // Если при распаковке пакета возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-3, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Не можем считать размер пакета, завершаем обработку данных.
                                    {
                                        StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                        break;
                                    }
                                }
                            }
                            else // Неизвестный ID пакета.
                                throw new PacketManagerException(String.Format("PacketManager got packet with unknown ID:{0}", packetID));
                        }
                        if (recievedCount > 0)
                            return _packetBuffer;
                        else
                            return null;
                    }
                    else // Не смогли прочитать данные из NetworkStream.
                        throw new PacketManagerException("Recieved 0 bytes of data, looks like Socket is closed.");
                }
                catch (IOException ex) // Во время чтения данных из NetworkStream произошла ошибка.
                {
                    throw new PacketManagerException("IO exception while writing to NetworkStream occured", ex);
                }
 
            } // Данные для чтения из NetworkStream отсутствуют.
            else
                return null;
        }
        
        /// <summary>
        /// Создает новый экземпляр объекта на основе существующего подключения.
        /// </summary>
        /// <param name="netStream">Поток открытого TCP подключения.</param>
        public PacketManager(NetworkStream netStream)
        {
            Stream = netStream;
            StreamWriterBuffer = new MemoryStream(4096);
            StreamReaderBuffer = new MemoryStream(4096);
            StreamReader = new BinaryReader(StreamReaderBuffer);
            StreamWriter = new BinaryWriter(StreamWriterBuffer);
        }
    }
 
    class Exec
    {
        static int Main(string[] args)
        {
            Console.ReadKey();
            return 0;
        }
    }
}

Не изобретаю ли я тут велосипед?

Добавлено через 18 минут
Я вот думаю, может, рациональнее будет добавить в интерфейс пакетов свойство PacketID, чтобы пакет сам назначал себе ID (тогда можно писать структуры пакетов не в данной сборке). И сделать нечто вроде PacketManager.RegisterPacket(IPacket)?

@PVD · 20.04.2015, 06:13 **[ТС]**

Реализовал регистрацию пакетов:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
using System;
using System.Net;
using System.Net.Sockets;
using System.IO;
using System.Collections.Generic;
 
namespace PacketManager
{
    public interface IPacket
    {
        ushort Size { get; }
        bool FixedSize { get; }
        byte ID { get; }
        void Write(BinaryWriter stream);
        void Read(BinaryReader stream, int readCount);
    }
    public class IPacketException : Exception
        {
            public IPacketException() : base() { }
            public IPacketException(string message) : base(message) { }
            public IPacketException(string message, Exception innerException) : base(message, innerException) { }
        }
    class PacketManager
    {
        public class PacketManagerException : Exception
        {
            public PacketManagerException() : base() { }
            public PacketManagerException(string message) : base(message) { }
            public PacketManagerException(string message, Exception innerException) : base(message, innerException) { }
        }
        NetworkStream Stream;
        MemoryStream StreamWriterBuffer;
        MemoryStream StreamReaderBuffer;
        BinaryWriter StreamWriter;
        BinaryReader StreamReader;
        static readonly Type[] RegisteredPackets = new Type[256];
        static int RegisteredPacketCount = 0;
        public static void RegisterPacket(Type packetType)
        {
            if (packetType.GetInterface("IPacket") != null)
            {
                if (!packetType.IsAbstract)
                {
                    IPacket packet = Activator.CreateInstance(packetType) as IPacket;
                    if (RegisteredPackets[packet.ID] == null)
                    {
                        RegisteredPackets[packet.ID] = packetType;
                        RegisteredPacketCount++;
                    }
                    else if (RegisteredPackets[packet.ID] != packetType)
                        throw new PacketManagerException(String.Format("Can not set \"{0}\" to ID \"{1}\", because \"{2}\" registered by same ID.", packetType, packet.ID, RegisteredPackets[packet.ID]));
                }
                else
                    throw new PacketManagerException(String.Format("Can not register abstract class \"{0}\"", packetType));
            }
            else
                throw new PacketManagerException(String.Format("Can not register \"{0}\", because it does not has interface IPacket.", packetType));
        }
        public static bool IsRegistered(Type packetType)
        {
            if (packetType.GetInterface("IPacket") != null)
            {
                if (!packetType.IsAbstract)
                {
                    IPacket packet = Activator.CreateInstance(packetType) as IPacket;
                    if (RegisteredPackets[packet.ID] == packetType)
                        return true;
                    else
                        return false;
                }
                else
                    return false;
            }
            else
                return false;
        }
        public static Type[] GetRegisteredPackets()
        {
            Type[] result = new Type[RegisteredPacketCount];
            for (int i = 0, j = 0; i < 256; i++)
                if (RegisteredPackets[i] != null)
                {
                    result[j] = RegisteredPackets[i];
                    j++;
                }
            return result;
        }
        public void SendPacket(IPacket packet)
        {
            if (RegisteredPackets[packet.ID] != null)
            {
                StreamWriter.Write(packet.ID);
                if (!packet.FixedSize)
                    StreamWriter.Write(packet.Size);
                packet.Write(StreamWriter);
                try
                {
                    Stream.Write(StreamWriterBuffer.GetBuffer(), 0, (int)StreamWriterBuffer.Position);
                }
                catch (IOException ex)
                {
                    throw new PacketManagerException("IO exception while writing to NetworkStream occured", ex);
                }
                StreamWriterBuffer.Seek(0, SeekOrigin.Begin);
            }
            else
                throw new PacketManagerException(String.Format("Tried to send packet \"{0}\" which not listed in PacketTypes", packet));
        }
        readonly byte[] _tempBuffer = new byte[4096];
        readonly IPacket[] _packetBuffer = new IPacket[64];
        public IPacket[] RecievePackets(out int recievedCount)
        {
            recievedCount = 0;
            if (Stream.DataAvailable) // Если есть данные для чтения из NetworkStream.
            {
                try
                {
                    int readCount = Stream.Read(_tempBuffer, 0, 4096); // Читаем данные из NetworkStream.
                    if (readCount > 0) // Данные из NetworkStream прочитаны.
                    {
                        StreamReaderBuffer.Write(_tempBuffer, 0, readCount); // Запоминаем прочитанные данные
                        StreamReaderBuffer.Seek(-readCount, SeekOrigin.Current);
                        for (; (StreamReaderBuffer.Position - StreamReaderBuffer.Length > 0) && (recievedCount < 64); recievedCount++) // Пока есть данные и количество полученных пакетов < 64
                        {
                            byte packetID = StreamReader.ReadByte(); // Узнаем ID пакета.
                            if (RegisteredPackets[packetID] != null) // Если известный ID пакета.
                            {
                                _packetBuffer[recievedCount] = Activator.CreateInstance(RegisteredPackets[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                if (_packetBuffer[recievedCount].FixedSize) // Если пакет данного типа имеет фиксированную длину.
                                {
                                    if(_packetBuffer[recievedCount].Size > 0) // Если фиксированный размер пакета больше нуля.
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= _packetBuffer[recievedCount].Size) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount].Read(StreamReader, _packetBuffer[recievedCount].Size); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch(IPacketException ex) // Если при распаковке пакетов возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки пакета, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Пакет прочитан (Размер фиксированный и он равен нулю)
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                        {
                                            recievedCount++;
                                            StreamReaderBuffer.SetLength(0);
                                            break;
                                        }
                                    }
                                } 
                                else // Пакет имеет переменную длину.
                                {
                                    if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= 2) // Если можем считать размер пакета.
                                    {
                                        ushort packetSize = StreamReader.ReadUInt16(); // Размер пакета.
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= packetSize) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount] = Activator.CreateInstance(RegisteredPackets[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                                _packetBuffer[recievedCount].Read(StreamReader, packetSize); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch (IPacketException ex) // Если при распаковке пакета возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-3, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Не можем считать размер пакета, завершаем обработку данных.
                                    {
                                        StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                        break;
                                    }
                                }
                            }
                            else // Неизвестный ID пакета.
                                throw new PacketManagerException(String.Format("PacketManager got packet with unknown ID:{0}", packetID));
                        }
                        if (recievedCount > 0)
                            return _packetBuffer;
                        else
                            return null;
                    }
                    else // Не смогли прочитать данные из NetworkStream.
                        throw new PacketManagerException("Recieved 0 bytes of data, looks like Socket is closed.");
                }
                catch (IOException ex) // Во время чтения данных из NetworkStream произошла ошибка.
                {
                    throw new PacketManagerException("IO exception while writing to NetworkStream occured", ex);
                }
 
            } // Данные для чтения из NetworkStream отсутствуют.
            else
                return null;
        }
        public PacketManager(NetworkStream netStream)
        {
            Stream = netStream;
            StreamWriterBuffer = new MemoryStream(4096);
            StreamReaderBuffer = new MemoryStream(4096);
            StreamReader = new BinaryReader(StreamReaderBuffer);
            StreamWriter = new BinaryWriter(StreamWriterBuffer);
        }
    }
 
    class SomePacket : IPacket
    {
        public ushort Size { get { return 0; } }
        public bool FixedSize { get { return true; } }
        public byte ID { get { return 0; } }
        public void Write(BinaryWriter stream)
        {
        }
        public void Read(BinaryReader stream, int readCount)
        {
        }
    }
 
    class AnotherPacket : IPacket
    {
        public ushort Size { get { return 0; } }
        public bool FixedSize { get { return true; } }
        public byte ID { get { return 1; } }
        public void Write(BinaryWriter stream)
        {
        }
        public void Read(BinaryReader stream, int readCount)
        {
        }
    }
 
    class Exec
    {
        static int Main(string[] args)
        {
            PacketManager.RegisterPacket(typeof(SomePacket));
            PacketManager.RegisterPacket(typeof(AnotherPacket));
            var packetList = PacketManager.GetRegisteredPackets();
            Console.WriteLine("Packet manager have {0} registered packets:", packetList.Length);
            for (var i = 0; i < packetList.Length; i++)
                Console.WriteLine(packetList[i]);
 
            Console.WriteLine();
            Console.ReadKey();
            return 0;
        }
    }
}

Возможно, стоит сделать некоторый отдельный класс, который хранит и регистрирует пакеты? Это бы дало возможность, например сделать разный набор пакетов для общения между клиентом / сервером и сервером/сервером. Или к примеру до того как клиент авторизировался, один набор пакетов, после авторизации другой набор пакетов. Так, например, если клиент посылает пакет который не доступен на данном уровне доступа, сервер его просто отключит и т.д. ...

@PVD · 20.04.2015, 06:45 **[ТС]**

Отлаживать код я еще не отважился, но похоже, в

C#
1
public IPacket[] RecievePackets(out int recievedCount)

Ошибка, если пакет не будет собран, то при следующем запуске функции данные пакета будут перезаписаны и дальнейшая работа с Networkstream будет невозможна, так выбросит Exception на первой же неудачной попытке получить номер / размер пакета и т.д.
Вроде, поправил:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
public IPacket[] RecievePackets(out int recievedCount)
        {
            recievedCount = 0;
            if (Stream.DataAvailable) // Если есть данные для чтения из NetworkStream.
            {
                try
                {
                    int readCount = Stream.Read(_tempBuffer, 0, 4096); // Читаем данные из NetworkStream.
                    if (readCount > 0) // Данные из NetworkStream прочитаны.
                    {
                        long readStart = StreamReaderBuffer.Position; // Запоминаем позицию каретки
                        StreamReaderBuffer.Seek(0, SeekOrigin.End); // Ставим картеку в конец.
                        StreamReaderBuffer.Write(_tempBuffer, 0, readCount); // Запоминаем прочитанные данные.
                        StreamReaderBuffer.Seek(readStart, SeekOrigin.Begin); // Ставим каретку в позицию для чтения.
                        for (; (StreamReaderBuffer.Position - StreamReaderBuffer.Length > 0) && (recievedCount < 64); recievedCount++) // Пока есть данные и количество полученных пакетов < 64
                        {
                            byte packetID = StreamReader.ReadByte(); // Узнаем ID пакета.
                            if (RegisteredPackets[packetID] != null) // Если известный ID пакета.
                            {
                                _packetBuffer[recievedCount] = Activator.CreateInstance(RegisteredPackets[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                if (_packetBuffer[recievedCount].FixedSize) // Если пакет данного типа имеет фиксированную длину.
                                {
                                    if(_packetBuffer[recievedCount].Size > 0) // Если фиксированный размер пакета больше нуля.
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= _packetBuffer[recievedCount].Size) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount].Read(StreamReader, _packetBuffer[recievedCount].Size); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch(IPacketException ex) // Если при распаковке пакетов возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки пакета, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Пакет прочитан (Размер фиксированный и он равен нулю)
                                    {
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                        {
                                            recievedCount++;
                                            StreamReaderBuffer.SetLength(0);
                                            break;
                                        }
                                    }
                                } 
                                else // Пакет имеет переменную длину.
                                {
                                    if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= 2) // Если можем считать размер пакета.
                                    {
                                        ushort packetSize = StreamReader.ReadUInt16(); // Размер пакета.
                                        if (StreamReaderBuffer.Position - StreamReaderBuffer.Length >= packetSize) // Если данных достаточно для распаковки пакета.
                                        {
                                            try
                                            {
                                                _packetBuffer[recievedCount] = Activator.CreateInstance(RegisteredPackets[packetID]) as IPacket; // Создаем новый пакет необходимого типа.
                                                _packetBuffer[recievedCount].Read(StreamReader, packetSize); // Распаковываем пакет.
                                                if (StreamReaderBuffer.Position - StreamReaderBuffer.Length == 0) // Если данных больше нет, то завершаем обработку данных.
                                                {
                                                    recievedCount++;
                                                    StreamReaderBuffer.SetLength(0);
                                                    break;
                                                }
                                            }
                                            catch (IPacketException ex) // Если при распаковке пакета возникла ошибка.
                                            {
                                                throw new PacketManagerException(String.Format("Exception occured while unpacking packet {0} with ID {1} and size {2}", _packetBuffer[recievedCount], packetID), ex);
                                            }
                                        }
                                        else // Данных не достаточно для распаковки, завершаем обработку данных.
                                        {
                                            StreamReaderBuffer.Seek(-3, SeekOrigin.Current);
                                            break;
                                        }
                                    }
                                    else // Не можем считать размер пакета, завершаем обработку данных.
                                    {
                                        StreamReaderBuffer.Seek(-1, SeekOrigin.Current);
                                        break;
                                    }
                                }
                            }
                            else // Неизвестный ID пакета.
                                throw new PacketManagerException(String.Format("PacketManager got packet with unknown ID:{0}", packetID));
                        }
                        if (recievedCount > 0)
                            return _packetBuffer;
                        else
                            return null;
                    }
                    else // Не смогли прочитать данные из NetworkStream.
                        throw new PacketManagerException("Recieved 0 bytes of data, looks like Socket is closed.");
                }
                catch (IOException ex) // Во время чтения данных из NetworkStream произошла ошибка.
                {
                    throw new PacketManagerException("IO exception while writing to NetworkStream occured", ex);
                }
 
            } // Данные для чтения из NetworkStream отсутствуют.
            else
                return null;
        }

Хотя, может, вся эта затея сплошная ошибка

@Storm23 · 20.04.2015, 09:38

PVD, Сформулируйте задачу, которую вы решаете.
Формулировка "хочу сделать так, чтобы данные читались из NetworkStream и записывались MemoryStream" - не катит.
А потом еще и пакеты какие-то появляются, их зачем-то там где то регистрировать нужно, а тут еще многопоточность сбоку...Брр.. Начните с ясной формулировки задачи, может и не придется писать тонны трехколесных велосипедов.

@PVD · 20.04.2015, 10:30 **[ТС]**

Storm23, задача в том, чтобы сделать общую систему общения между двумя удаленными компьютерами.

Добавлено через 50 минут
По большей части не использую готовое, так как цель больше научиться чем сделать.

Новые блоги и статьи Все статьи Все блоги /
Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это дополнительная запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:
Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .

@Storm23 10427 / 5157 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34
	20.04.2015, 09:38
	PVD, Сформулируйте задачу, которую вы решаете. Формулировка "хочу сделать так, чтобы данные читались из NetworkStream и записывались MemoryStream" - не катит. А потом еще и пакеты какие-то появляются, их зачем-то там где то регистрировать нужно, а тут еще многопоточность сбоку...Брр.. Начните с ясной формулировки задачи, может и не придется писать тонны трехколесных велосипедов. 0

@PVD 24 / 24 / 11 Регистрация: 01.06.2014 Сообщений: 135
	20.04.2015, 10:30 [ТС]
	Storm23, задача в том, чтобы сделать общую систему общения между двумя удаленными компьютерами. Добавлено через 50 минут По большей части не использую готовое, так как цель больше научиться чем сделать. 0