1988 C代码有什么问题？

我正在尝试从“ The C Programming Language”（K＆R）一书中编译这段代码。它是UNIX程序的基本版本wc：

#include <stdio.h>

#define IN   1;     /* inside a word */
#define OUT  0;     /* outside a word */

/* count lines, words and characters in input */
main()
{
    int c, nl, nw, nc, state;

    state = OUT;
    nl = nw = nc = 0;
    while ((c = getchar()) != EOF) {
        ++nc;
        if (c == '\n')
            ++nl;
        if (c == ' ' || c == '\n' || c == '\t')
            state = OUT;
        else if (state == OUT) {
            state = IN;
            ++nw;
        }
    }
    printf("%d %d %d\n", nl, nw, nc);
}

我收到以下错误：

$ gcc wc.c 
wc.c: In function ‘main’:
wc.c:18: error: ‘else’ without a previous ‘if’
wc.c:18: error: expected ‘)’ before ‘;’ token

本书的第二版是1988年出版的，我对C还是很陌生。也许它与编译器版本有关，或者我只是在胡说八道。

我已经在现代C代码中看到了对该main函数的不同用法：

int main()
{
    /* code */
    return 0;
}

这是新标准还是我仍然可以使用无类型的main？

你的问题是与你的预处理器定义IN和OUT：

#define IN   1;     /* inside a word */
#define OUT  0;     /* outside a word */

请注意，您在每一个中都使用尾随分号。当预处理器扩展它们时，您的代码将大致如下所示：

    if (c == ' ' || c == '\n' || c == '\t')
        state = 0;; /* <--PROBLEM #1 */
    else if (state == 0;) { /* <--PROBLEM #2 */
        state = 1;;

第二个分号导致else之前没有if匹配号，因为您没有使用花括号。所以，从预处理器定义中删除分号IN和OUT。

在这里学到的教训是 预处理程序语句不必以分号结尾。

另外，您应该始终使用大括号！

    if (c == ' ' || c == '\n' || c == '\t') {
        state = OUT;
    } else if (state == OUT) {
        state = IN;
        ++nw;
    }

else上面的代码没有任何歧义。

此代码的主要问题是它不是 K＆R的代码。它包括宏定义之后的分号，而本书中没有这些分号，正如其他人指出的那样，它改变了含义。

除非进行更改以尝试理解代码，否则除非您理解它，否则应将其搁置。您只能安全地修改您了解的代码。

这可能只是您的错字，但这确实说明了编程时需要理解和注意细节的需求。

巨集后不应有任何分号，

#define IN   1     /* inside a word */
#define OUT  0     /* outside a word */

它应该是

if (c == ' ' || c == '\n' || c == '\t')

IN和OUT的定义应如下所示：

#define IN   1     /* inside a word  */
#define OUT  0     /* outside a word */

分号引起了问题！解释很简单：IN和OUT都是预处理器指令，从本质上讲，编译器将在源代码中将所有出现的IN替换为1，将所有出现的OUT替换为0。

由于原始代码在1和0之后有分号，因此当代码中替换IN和OUT时，数字后的多余分号产生了无效代码，例如以下行：

else if (state == OUT)

最终看起来像这样：

else if (state == 0;)

但是您想要的是：

else if (state == 0)

解决方案：删除原始定义中的数字后的分号。

如您所见，宏存在问题。

GCC可以选择在预处理后停止。（-E）此选项对查看预处理结果很有用。实际上，如果您使用c / c ++中的大型代码库，则该技术是一项重要的技术。通常，makefile的目标是在预处理后停止。

快速参考：SO问题涵盖了选项- 在Visual Studio中进行预处理后如何查看C / C ++源文件？。它以vc ++开头，但下面还提到了gcc选项。

这不完全是问题，但是的声明main()也已过时，应该像这样。

int main(int argc, char** argv) {
    ...
    return 0;
}

编译器将假定函数的int返回值为w / o one，我确信编译器/链接器将解决缺少argc / argv的声明和缺少返回值的问题，但是它们应该在那里。

尝试在代码块周围添加大括号。在K＆R风格可以是不明确的。

看第18行。编译器告诉您问题出在哪里。

    if (c == '\n') {
        ++nl;
    }
    if (c == ' ' || c == '\n' || c == '\t') { // You're missing an "=" here; should be "=="
        state = OUT;
    }
    else if (state == OUT) {
        state = IN;
        ++nw;
    }

一个简单的方法是使用括号如{}每个if和else：

if (c == '\n'){
    ++nl;
}
if (c == ' ' || c == '\n' || c == '\t')
{
    state = OUT;
}
else if (state == OUT) {
    state = IN;
    ++nw;
}

正如其他答案所指出的那样，问题在于#define分号。为了尽量减少这些问题，我总是更喜欢将数字常量定义为const int：

const int IN = 1;
const int OUT = 0;

这样，您就可以摆脱许多问题和可能出现的问题。它受到两方面的限制：

1. 您的编译器必须支持const-在1988年通常不是真的，但是现在所有常用的编译器都支持它。（AFAIK const是从C ++“借用的”。）

2. 您不能在某些需要类似字符串的常量的特殊地方使用这些常量。但我认为您的程序并非如此。