伪装过早成为现实

Question

我试图通过一次读取一个字符直到缓冲区（一个字符数组）已满，然后发送它，然后重复直到文件末尾，一次通过套接字发送文件。但是由于某种原因，feof在文件结束之前变为真（我相信可能是缓冲区数组已满的时候）。

int c;
int end_loop = 1;
char to_send[BUFFER_LENGTH];
while (end_loop == 1) { //For the entire file
    printf("creating next data\n");
    bzero(to_send, BUFFER_LENGTH);
    for (int i = 0; i < BUFFER_LENGTH - 1; i++) { // read data from file until buffer full or EOF
        c = fgetc(f);
        if (c == EOF) { // If end of file break loop
            printf("EOF\n");
            end_loop = 0;
            break;
        }
        to_send[i] = c;
    }
    for (int i = 0; i < strlen(to_send); i++) {
        printf("%c", to_send[i]);
    }
    n = write(sockfd, to_send, strlen(to_send));
    if (n < 0) {
        perror("ERROR writing to socket");
        exit(1);
    }
}
n = write(sockfd, "EOF", 3);
if (n < 0) {
    perror("ERROR writing to socket\n");
    exit(1);
}

这是一个简单文件的输出

feof可能不是问题所在，因为尽管满足了“ EOF”，但代码似乎仍在循环。

编辑：添加了来自注释的建议。仍然会发生错误。

Answer 1

代码需要跟踪读取的字符数，然后printf(), write()跟踪那么多字符。

while (end_loop == 1) {
    ...
    // for (int i = 0; i < BUFFER_LENGTH - 1; i++) {
    int i;
    for (i = 0; i < BUFFER_LENGTH - 1; i++) { 
      ...
    }
    // for (int i = 0; i < BUFFER_LENGTH - 1; i++) {
    //    printf("%c", to_send[j]);
    for (int j = 0; j < i; j++) {
      ...
    // n = write(sockfd, to_send, BUFFER_LENGTH);
    n = write(sockfd, to_send, i);
    ...
}

其他问题包括

测试EOF，不仅测试feof(f)

    c = fgetc(f);
    // if (feof(f)) { // If end of file break loop 
    if (c == EOF) { // If end of file break loop **or input error**

填充char数组时不需要-1。 -1可能对字符串有用，但是在这里使用字符串也不是最好的主意，因为fgetc()可能返回空字符。

//                                   vvv 
// for (int i = 0; i < BUFFER_LENGTH - 1; i++)
for (int i = 0; i < BUFFER_LENGTH; i++)
// or better 
size_t i; 
for (i = 0; i < sizeof to_send; i++)

---

OP修改后的代码中答案Hmmmmmmmm。

请勿使用for (int i = 0; i < strlen(to_send); i++) {。读取的数据可能包含空字符，这会否定此处对 strings 的充分利用。

---

如果OP不想在最后一行之前看到"EOF\n"，请在"EOF\n"循环之后打印while。