c - 行数之谜？

Question

我的目标是实现一个计算文件行数的函数。空文件被认为没有行。如果给定文件的最后一行不为空，尽管没有以换行符结尾，也应该算作一行。

我想出了以下代码:

   int linecount(const char *filename)
{
    FILE *f = fopen(filename, "r");
    if(!f)
        return -1;
    int lines = 0;
    int c = 0;
    int n = 0;
    while((c = fgetc(f)) != EOF){
        if(c == '\n')
            lines++;
        n++;
    }
    if(n==0)
        return 0; //return 0 if the file is empty
    if(c!='\n' && !isspace(c))
        lines++; //count the last line if it's not empty
    fclose(f);
    return lines;
}

然而，即使在玩了一个多小时之后，我也无法弄清楚为什么它的返回值线在某些情况下太大了......

Answer 1

你已经很接近了，这里是你如何做到的:

int linecount(const char *filename) {
  FILE *f = fopen(filename, "r");
  if (!f)
    return -1;
  int lines = 0;
  int c = 0;
  int n = 0;
  int read_line = 0;
  while ((c = fgetc(f)) != EOF) {
    read_line = 1;
    if (c == '\n') {
      lines++;
      read_line = 0;
    }
    n++;
  }
  if (n == 0)
    return 0;  //return 0 if the file is empty
  if(read_line)
    lines++;
  fclose(f);
  return lines;
}

我们想知道我们是否开始阅读一行，以及是否在该行的末尾遇到换行符。因此，我们使用另一个变量，称为 read_line 我们将其用作标志。

如果我们刚开始读取一行，我们将它设置为 1 (true)，如果我们刚遇到一个换行符(行尾)，我们将它设置为 0 (false)。

现在，如果我们有类似的东西:

1[newline]
2[newline]
3

我们会没事的，因为我们需要在读取文件后检查 read_line。如果是这样，我们必须将行计数器加一。

这也行:

1[newline]
2[newline]
3[newline]

因为我们在读取文件后看到三个换行符并且 read_line 为 0。

同样适用于这种情况:

1[newline]
2[newline]
3[newline]
[nothing here]

因为我们的标志在读取文件后将等于 0，因为第 3 个换行符应该将其设置为 0 并且我们实际上从未进入循环中的第 4 行，因为没有任何内容可读。

对于您之前的实现，如评论中所述，这一行:

if(c!='\n' && !isspace(c))

将在 c 等于 EOF 的情况下执行。

---

或者您可以只使用 fgets()你完成了。检查示例:

#include <stdio.h>
#include <string.h>

#define bufSize 1024

int main(int argc, char *argv[])
{
  FILE *fp;
  char buf[bufSize];

  if ((fp = fopen("test.txt", "rb")) == NULL)
  { /* Open source file. */
    perror("fopen source-file");
    return 1;
  }
  int lines = 0;
  while (fgets(buf, sizeof(buf), fp) != NULL)
  { /* While we don't reach the end of source. */
    /* Read characters from source file to fill buffer. */
    /* fgets will stop when it finds a newline. */
    lines++;
  }
  printf("lines = %d\n", lines);
  fclose(fp);
  return 0;
}