c - feof 过早变为真

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++) { .读取的数据可能包含空字符，这会影响此处字符串 的良好使用。

---

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