程序读取代码并计算其中的行数，计数中不包括注释和空行

Question

我正在尝试创建一个程序，在给定输入文件的情况下，返回输入文件中所有代码行的计数，不包括空白行和注释行。我已经编写了以下代码，但是我需要有关如何排除包含注释和空行的行的帮助。

#include<stdio.h>
int main()
{
    int count;
    char ch;
    FILE *fptr;
    clrscr();
    fp=fopen("test.cpp","r");
    if(fp==EOF)
        {
        perror("Error:");
        }
    else
    {
        while(ch!=EOF)
        {
            ch=fgetc(fptr);
            if(ch=='\n')
                count++;
            if(ch=='\\')
                count--;
            if(ch=='\*')
                          {
                while(ch!='*\')
                                    {
                    ch=fgetc(fptr);
                                    }
            }
        }
    printf("the lines in the code are %d\n",count);
    fclose (fptr)
    }
    getchar();
    return 0;
}

如何修改上面的代码，使空行和注释行不被计算在内？

Answer 1

如果逐字符读取输入文件，则比逐行读取输入文件要多得多的工作。毕竟你在计算行数...

psudocode

    1. initialize line count to 0
    2. read a line
    3. end of file? yes: goto 7
    4. is it a good line? yes: goto 5; no: goto 2
    5. increment line count
    6. repeat from 2
    7. output line count

现在你问...什么是好的行？
对于程序的近似，我建议您考虑一行，除了由 0 个或多个空格组成的行之外的所有内容。此近似值将计算评论，但您可以从这里开发您的程序。

以下版本忽略空行上带有 // 注释的行。

版本 3 可以忽略同时包含 /* 和 */ 的行

，版本 4 将处理多行注释。

最重要的是，玩得开心！

Answer 2

C 注释为 // 和 /* */。以下几行是您的问题所在：

        if(ch=='\\')
            count--;
        if(ch=='\*')
            while(ch!='*\')
                ch=fgetc(fptr);

另一个问题是，如果没有某种状态机，您无法通过一次读取一个字符来匹配两个字符的注释分隔符。

此外，您的代码应该适应注释嵌入到实际代码行中的情况。例如。

x = 1;    // Set value of x

您最好一次读取文件一行，并检查每一行是否为空白或注释，如果不是则增加计数器。

Answer 3

您的意思是 //、/* 和 */ 而不是 \ * 和 *\

，\ 用作转义字符，这会改变其后面的字符的“含义”。

\n 给你一个换行符。使用 \\ 你会得到一个单一的 \ ，而使用 \' 你会得到一些不会关闭开头的东西 '

如果你用正确的字符替换这些注释字符，你应该得到可以编译的代码。
但它不会正确计数。

想象这样一行：

doSomething(); // foo

Answer 4

除了字符常量的问题之外，您处理fputc的方式也有错误。 fputc 返回一个 int。如果没有剩余的红色字符或出现错误，它可以返回 EOF（负整数常量），也可以返回作为 unsigned char< 读取的字符的值/code> 并转换为 int。

如果在将 fputc 的返回值与 EOF 进行比较之前将其转换为 char，则有效字符可能会等于 EOF< /code> 导致循环提前终止。

另外，while 循环并不是在第一次调用 fputc 之前启动的，因此您在第一次迭代中使用的是 ch 的未初始化值。这可能会导致任何事情发生。

形成循环的惯用方法是：

int ch;
while ((ch = fgetc()) != EOF)
{
    /* ... */
}

在循环内部，您需要小心比较返回值，因为 ch 是 unsigned char code> 转换为 int。

在大多数平台上，最简单的做法是创建一个 char 变量以进行比较，尽管您可以将字符常量通过相同的 unsigned char 到 int 转换例程。

例如

char c = ch;

if (c == '\n')

或

if (ch == (unsigned char)'\n')

其他人指出了您的字符文字的问题。

Answer 5

好吧，问题的一部分是你的 ch 变量只有一个字符长度，但是当你测试注释时，例如 \\ 和 \*，这是两个字符长，因此需要使用字符串比较。

另一个问题是，C/C++ 中的单行注释实际上以 // 开头，多行注释以 /* 开头，以 */< 结尾/代码>。

Answer 6

你可以在 Python 中编写这样的代码：

def goodline(l : str) -> int:
    if l.lstrip().startswith("/*") and l.rstrip().endswith("*/"):
        # single line
        return 0
    elif l.lstrip().startswith("/*") and not l.rstrip().endswith("*/"):
        # multi line start
        return 1
    elif not l.lstrip().startswith("/*") and l.rstrip().endswith("*/"):
        # multi line end
        return 2
    elif l.strip() == "":
        # empty line
        return 3
    elif l.lstrip().startswith("//"):
        # single line
        return 0
    else:
        # good line
        return 4

如果上面函数的 return 为 1，则继续迭代，直到 return 变为 2。