这两个版本都不是好的编码;两者都忽略了在读取空间之前到达 EOF 的可能性,并且都忽略了在读取空间之前溢出缓冲区的可能性。
第一个在其他方面是正确的。它包含一个任务:
x[num++] = getchar()
赋值的结果是赋值。因此,如果getchar()
返回“X”,则赋值结果为“X”。然后将赋值结果与空白进行比较;它们是不同的,循环重复。如果getchar()
返回一个空白,那么赋值的结果也是一个空白,空白等于空白,所以循环终止。
第二个是完全错误的。因为赋值的优先级低于!=
,所以结果就像代码读取一样:
while (x[num++] = (getchar() != ' '))
;
也就是说,一个字符被读取getchar()
并与空白进行比较,如果字符不是空白,则生成值 1,如果是空白,则生成值 0。这个 0 或 1 分配给x[num++]
,然后作为逻辑条件进行评估。如果结果为 0,则循环终止;如果结果为 1,则循环继续。请注意,读取的字符未记录在此版本中 - 这就是编译器发出警告的原因。这几乎总是一个错误。在那些奇怪的情况下,如果它不是错误,你可以告诉编译器,通过提供额外的括号,让编译器和你的观众,其他将阅读你的代码的人清楚地知道你的意图。(请记住:如果您在 6 个月甚至 6 周后回来,您将成为另一个人,并且可能很难记住这些微妙之处。让每个人都明白。在过度括号之间有一个很好的平衡和括号内的代码。)
代码可能应该是:
int c;
int max = sizeof(x) - 1;
int num = 0;
while ((c = getchar()) != EOF && num < max && (x[num++] = c) != ' ')
;
请特别注意,它c
必须是 aint
而不是 a char
。