使用C中的套接字在多个客户端上传输文件时格式化错误

时间:2012-04-03 14:32:59

标签: c sockets file-transfer

我正在尝试实施一个简单的代码测试程序:ACM ICPC或Google Code Jam在线评审员的基本版本。

评估部分有效(到目前为止),但我对文件传输部分感到愚蠢。

基本上我的设置是:

我有一个服务器(永远在线)和属于团队的多个客户端系统,他们将提交他们的代码。他们可以在团队中相互传输文件,但服务器将保留传输日志。服务器有时可以向所有或某些客户发送通知。最终代码将提交给特定的空客户端计算机进行评估。

现在,我的服务器端代码出现问题。我正在使用

select(fdmax+1, &read_fds, NULL, NULL, NULL)

识别准备好向我写入数据的客户端 - 我知道这种轮询可以通过fork()以某种方式完成,但我对此并不确定也不满意。

为了接收数据到服务器,我使用以下功能:

void send_recv(int i, fd_set *master, int sockfd, int fdmax)
{
    int nbytes_recvd, j;
    char recv_buf[BUFSIZE], buf[BUFSIZE];

    if ((nbytes_recvd = recv(i, recv_buf, BUFSIZE, 0)) <= 0) {
        if (nbytes_recvd == 0) {
            printf("socket %d hung up\n", i);
        }else {
            perror("recv");
        }
        close(i);
        FD_CLR(i, master);
    }else {
        FILE *logger;
        logger = fopen(fname2, "w");
        fprintf(logger,"Client %i> %s", i,recv_buf);
        for(j = 0; j <= fdmax; j++){
            send_to_all(j, i, sockfd, nbytes_recvd, recv_buf, master );
        }
    }
}

这里,我没有记录通过流分别传输的所有文件数据,而是一个难以解析的乱七八糟的混乱。

此功能实际上是传输文件(所有ANSI * .c):

void send_to_all(int j, int i, int sockfd, int nbytes_recvd, char *recv_buf, fd_set *master)
{
    if (FD_ISSET(j, master)){
        if (j != sockfd && j != i) {    //prevents data rewrite to incoming host
            int client;
            sscanf(recv_buf, "%d >", &client);
            if(client == 0){                //universal send
                if (send(j, recv_buf, nbytes_recvd, 0) == -1) {
                    perror("send");
                }
            }                               //send to specified host
            else if (send(client, recv_buf, nbytes_recvd, 0) == -1) {
                perror("send");
            }
        }
    }
}

此处传入的文件会丢失所有格式,并且每次都会打印两次。

我的服务器代码的精简版本位于:http://codepad.org/TTejTfbL

我的问题是,我做了一些根本错误的事情吗?我的逻辑(以及扩展名,我的代码)是否已经无可挽回地破坏了?

如果有人能指出我正确的方向,我会感到欣慰。 或者甚至告诉我把它扔掉,从头开始。 (^; __; ^)

1 个答案:

答案 0 :(得分:1)

您不能假设您可以在一次通话中send()任意数量的数据,并使其单调成功或失败。库实现的缓冲区有限制。

你很有可能通过例如20 KB到send(),并在仅发送4,711个字节后成功返回。这将要求您在缓冲区中向前推进适当的数量,减少“要去的字节数”,然后重试。

在您发送所有数据或从套接字收到I / O错误之前,需要循环播放。