问题描述
为什么 gcc 不对这段代码给出任何警告?
#include <stdio.h>
void printDigits(int *arr,int arrsize);
int main() {
int number[] = { 1,7,8,3,6,5,4,2 };
size_t arraysize = (sizeof(number) / sizeof(number[0]));
printDigits(number,arraysize);
return 0;
}
void printDigits(int *arr,int arrsize) {
int i;
for (i=0; i<arrsize,i++;) {
printf("%d ",arr[i]);
}
}
特别是关于printDigits 函数中的for 循环。它的:
for(i=0; i<arrsize,i++;)
虽然它真的应该是 for(i=0; i<arrsize; i++)
Gcc 没有给我任何警告,我花了一段时间才弄清楚为什么数组没有打印出来。
解决方法
有一个警告。 i<arrsize
被计算但随后被丢弃,因此 gcc 警告:
warning: left-hand operand of comma expression has no effect [-Wunused-value]
(由 -Wall
启用)
这个for循环
for(i=0; i<arrsize,i++;){
printf("%d ",arr[i]);
}
是一个有效的 C 结构。在循环条件中使用逗号运算符
i<arrsize,i++
其中的值是其第二个操作数的值。因此循环体将不会被执行,因为表达式 i++
的值为 0(即变量 i
在其递增之前的值)。
来自 C 标准(6.5.17 逗号运算符)
2 逗号运算符的左操作数被评估为空 表达;在它的评估和那个之间有一个序列点 正确的操作数。然后评估正确的操作数;结果 有它的类型和值
注意for循环中的任何表达式都可以省略。例如你甚至可以这样写
for ( ; ; )
{
//...
}
此外,您至少应该将第二个函数参数的类型从 int
更改为 size_t
,因为您传递的是 size_t
类型的参数。
函数应该像这样声明
void printDigits( const int *arr,size_t arrsize );
,
这是一个有效的 C。这个 for 循环将执行直到 i
不为零