Отправка изображения через TCP

Question

im пытается отправить изображение через tcp, он работает около 50% времени, остальные 50% просто дают мне черное изображение, и если я отправлю 2 в пространстве, как 3 секунды, он сработает , кто-нибудь знает почему? и как я могу исправить эту

Клиент:

    while ((i = stream.Read(datalength, 0, 4)) != 0) 
       { 

        byte[] data = new byte[BitConverter.ToInt32(datalength, 0)]; 
        stream.Read(data, 0, data.Length); 
        this.Invoke((MethodInvoker)delegate 
        { 
         try 
         { 
          Image Screenshot = byteArrayToImage(data); 
          pictureBox1.Image = Screenshot; 
         } 
         catch { } 
        }); 
       } 

Это функция, которая преобразует массив байтов изображения

 public Image byteArrayToImage(byte[] byteArrayIn) 
    { 
     MemoryStream ms = new MemoryStream(byteArrayIn); 
     Image returnImage = Image.FromStream(ms); 
     return returnImage; 
    } 

Сервер:

 Bitmap printscreen = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height); 
     Graphics graphics = Graphics.FromImage(printscreen as Image); 
     graphics.CopyFromScreen(0, 0, 0, 0, printscreen.Size); 
       byte[] data = imageToByteArray(Image); 
       stream = client.GetStream(); 
       int length = data.Length; 
       byte[] datalength = new byte[4]; 
       datalength = BitConverter.GetBytes(length); 
       stream.Write(datalength, 0, 4); 
       stream.Write(data, 0, data.Length); 

Это функция который преобразует изображение в байтовый массив

 public byte[] imageToByteArray(System.Drawing.Image imageIn) 
    { 
     MemoryStream ms = new MemoryStream(); 
     imageIn.Save(ms, System.Drawing.Imaging.ImageFormat.Gif); 
     return ms.ToArray(); 
    } 

Answer 1

Вы должны убедиться, что прочитали весь «пакет». Не уверен, что вы получаете все только потому, что вы его просите. Метод Read() вернет количество прочитанных байтов, сохранит их и зациклирует до тех пор, пока вы не получите нужное количество байтов.

Замените свой текущий Read() с этим:

int bytesReceived = 0; 
while(bytesReceived < data.Length) 
{ 
    bytesReceived += stream.Read(data, bytesReceived, data.Length - bytesReceived); 
} 

Это будет читать, пока все ваше изображение не было получено.

EDIT: Исправлена ​​ошибка с кодом, благодаря Ivan's answer.

Answer 2

Вы читаете 1000 байт, когда вам нужно читать четыре. Вы читаете и выбрасываете большую часть изображения после слова длины.

Вы также, по-видимому, игнорируете результат read() и считаете, что он заполняет буфер.