我在更大的C程序中有以下代码。直到刚才我尝试编译它时,我才遇到任何问题;它在 Minix 2.0.4 中运行并使用 cc
进行编译。引发编译错误,如下所示:
line 26: void not expected
第 26 行只是 main()
内部的函数声明:
void initpool(void);
initpool()
本身在程序后面使用以下标头定义:
void
initpool(void)
{
根据我的研究,一切都应该是正确的,gcc
不会引发编译错误。前面的所有行都以 ;
s 结尾,所以这不是问题所在。为什么cc
在编译它时遇到问题?
编辑:根据要求,导致第26行的行如下(从main()
开头开始,第25行为空白):
19: int
20: main(int argc, char *argv[])
21: {
22: int count, inserror;
23: olnode *list, *ptr;
24: list = NULL;
很可能你的程序如下所示:
int main(int argc, char ** argv) {
... // (a)
void initpool(void);
...
initpool();
...
}
用(a)表示的部分必须包含一些非声明声明。在较旧的 C 编译器中,不允许在第一个非声明语句之后进行声明:
void foo() {
int a;
int b;
foo();
int c; // not allowed in old C
}
因此,有两种可能的修复方法:
// the preferred fix for a single file
void initpool(void);
int main(int argc, char ** argv) {
... // (a)
...
initpool();
...
}
void initpool(void) {}
// the technically correct fix
int main(int argc, char ** argv) {
void initpool(void);
... // (a)
...
initpool();
...
}
initpool
的前瞻声明真的不属于main
。为什么?因为你应该让编译器帮助你,而你不应该重复自己。
就冗长而言,当地的声明看起来非常愚蠢:
// Good // Correct but Verbose
void initpool(void); void fun1(void) {
void fun1(void) { void initpool(void);
initpool(); initpool();
} }
void fun2(void) { void fun2(void) {
initpool(); void initpool(void);
} initpool();
}
最后,假设initpool()
是在单独的文件中实现的。然后你可以自由地做任何你想做的愚蠢的事情。例如:
// pool.c
void initpool(void) {
...
}
// main.c
int main() {
void initpool(); // a common typo
initpool(0); // undefined behavior, your hard drive gets formatted
}
您应该在单独的头文件中具有池组件的公共 API:
/// pool.h
void initpool(void);
/// pool.c
#include "pool.h"
void initpool(void) { ... }
/// main.c
#include "pool.h"
int main() {
initpool(); // OK
initpool(0); // the compiler will catch the mistake
}
没关系,老编译器会很乐意接受,例如,这个:
void fun1() {
void initpool(int);
}
void fun2() {
void initpool(void);
}
最后,必须说,在 C 中,并且仅在 C(不是 C++)中,以下声明是兼容的,但这并不能使其安全。行为是实现定义的。例如,这种草率会生成带有 stdcall 的无效程序集。
void bar(); // an unknown, fixed number of arguments
void bar(int,int,int,int);
如果 C 允许这样做,void bar()
类似于void bar(...)
。一些旧的 C 编译器确实允许在没有前面参数的情况下使用省略号。
感谢 Keith Thompson 迫使我更深入地研究事物,并意识到我使用的一些编译器是多么糟糕:)
将问题中的代码片段放在一起,您有以下内容:
int
main(int argc, char *argv[])
{
int count, inserror;
olnode *list, *ptr;
list = NULL;
void initpool(void); /* line 26 */
/* ... */
}
在1999年ISO C标准之前,C不允许在一个块中混合声明和语句。每个块(包括函数定义的外部块)必须包含零个或多个声明,后跟零个或多个语句。
1999 年的标准放宽了此规则(遵循 C++),但许多 C 编译器仍然默认执行 C90 规则。(C90有时被错误地称为"ANSI C"。
您有一个声明:
list = NULL;
后跟一个声明:
void initpool(void);
将声明移到语句上方应该可以更正问题。使用编译器选项使用 C99 或更高版本的标准也应该可以解决此问题,但这可能不可用,具体取决于您使用的编译器。海湾合作委员会有-std=c99
、-std=gnu99
、-std=c11
和-std=gnu11
;有关详细信息,请阅读 GCC 手册。我不知道"cc
"是什么编译器;这是许多不同 C 编译器的通用名称。
顺便说一下,将函数声明放在函数定义中有点不寻常。更常见的做法是将所有函数声明放在文件范围内,或者对于较大的项目,将声明放在由定义函数的.c
文件和包含对函数调用的任何.c
文件#include
的头文件中。不过,显然你的导师坚持这种风格。这并没有错,只要函数声明和定义出现在同一个源文件中,编译器就会诊断任何不一致之处。
如果声明使用空括号,则存在潜在问题:
void initpool();
不幸的是,这与以下条件兼容:
void initpool(int n) { /* ... */ }
但这与声明是否在函数体内无关,并且通过始终使用原型很容易避免。
对于不带参数的函数,请使用 (void)
,而不是 ()
。您已经在使用正确的原型;继续这样做。