通过套接字传输文件c

Question

我想实现一个程序，将任何文件从服务器传输到客户端。我必须使用读/写功能读取和写入数据（用于学校作业）。这里是服务器和客户端的代码。

server.c

char buffer[512];   

    if((file = open(strTable[1], O_RDONLY)) == -1) { perror("Open"); } 

    while(read(file, buffer, sizeof(buffer)) != 0) 
    { 
     size = strlen(buffer)+1; 
     if(write(newsock, &size, sizeof(size)) < 0) { perror("Write"); exit(1); } 

     write_all(newsock, buffer, strlen(buffer)+1); 
    } 

    size = 4; 
    if(write(newsock, &size, sizeof(size)) < 0) { perror("Write"); exit(1); } 
    write_all(newsock, "end", 4); 

它打开strTable [1]（其中包含我想读取文件），它读取的sizeof（缓冲区）字节，然后我发送给客户多少字节我将写入套接字，然后我发送字节。这里是write_all函数。

int write_all(int sock, char* buffer, int size) 
{ 
    int nwrite, sent = 0; 

    while(sent < size) 
    { 
     if((nwrite = write(sock, buffer + sent, size - sent)) < 0) 
     { perror("Write"); exit(1); } 

     sent += nwrite; 
    } 

    return sent; 
} 

client.c

if((file = open(absolute, O_WRONLY | O_CREAT | O_TRUNC, 0644)) == -1) 
       { perror("Open"); }  

while(true) 
     { 
      received = 0; 

      /* Read the desired readable size */ 
      if(read(sock, &size, sizeof(size)) < 0) 
      { perror("Read"); pthread_exit(NULL); } 

      /* Read all data */ 
      while(received < size) 
      { 
       if((nread = read(sock, buffer + received, size - received)) < 0) 
       { perror("Read"); pthread_exit(NULL); } 

       received += nread; 
      } 

      if(strncmp(buffer, "end", 4) == 0) { break; } 

      write_all(file, buffer, strlen(buffer)+1); 
     } 

客户端打开一个文件（绝对），并在其所写的东西它读取。它首先读取它需要读取的尺寸，然后它不会停下来读取，直到达到该尺寸。如果客户端读取“结束”，则服务器已停止写入套接字，以便客户端停止读取。

问题是我无法打开转移后的文件。我读了一些我无法打开的图像。我还读了一个file.txt，我写了一些随机单词，这似乎被复制了，但它也有一些垃圾（其照片编号为2）。为什么我得到这个以及如何正确地从套接字传输文件？

enter image description here

enter image description here

Answer 1

常见的问题。 read()不终止缓冲区，因此使用strlen()作为计数无效。读/写循环应该是这样的：

int count; 
while ((count = read(inFD, buffer, sizeof buffer)) > 0) 
{ 
    if (write(outFD, buffer, count) < 0) 
    { 
     perrror("write"); // at least 
     break; 
    } 
} 

其次是错误处理如下：

if (count < 0) 
{ 
    perror("read"); // at least 
} 

你的第二个问题带有假设"end"将自身作为一个独立的消息被接收。这不能保证。您必须通过在每个文件之后关闭套接字来使用流结束，或者在每个文件之前发送长度，并且只从每个文件的流中读取许多字节。但是，由于您已经发送了长度，发送"end"无论如何都毫无意义。