执行while或for循环后没有代码执行C

Question

我正在使用Xcode 8学习C，编译器在执行while或for循环后不运行任何代码。这是一个错误？我该如何解决它？

在下面printf("code executed after while-loop");提供的例子中从未执行

#include <stdio.h> 

int getTheLine(char string[]); 

int getTheLine(char string[]) { 

    char character; 
    int index; 
    index = 0; 

    while ((character = getchar()) >= EOF) { 

    string[index] = character; 
    ++index; 
    } 

    printf("code executed after while-loop"); 
    return index; 
} 

int main(int argc, const char * argv[]) { 

    char string[100]; 
    int length = getTheLine(string); 
    printf("length %d\n", length); 

    return 0; 
} 

Answer 1

getchar返回int不是char，和比较EOF应与!=操作者而不是操作者>=来完成。

... 
int character; // int instead of char 
int index; 
index = 0; 

while ((character = getchar()) != EOF) { // != instead of >= 
... 

Answer 2

这是>= EOF，这将让病情始终true。其原因是，getchar()“有效”的结果将是一个正整数，和一个“非有效”导致像文件结束-将EOF，其是负（参见getchar()）：

EOF ... int类型和负值

因此的整数常量表达式，从getchar任何有效的结果将是>EOF，而结束文件的结果将是==EOF，使得>= EOF将总是匹配。

改为写!= EOF。

此外，值得注意的是你不串终止字符'\0'，使得在​​使用string像一个字符串（例如，将产生不确定的行为（崩溃或别的东西，可能不需要）终止您的字符串。 所以写至少：

while ((character = getchar()) != EOF) { 
    string[index] = character; 
    ++index; 
} 
string[index]='\0'; 

然后还有，你可以写出界，例如如果一个人在你的例子进入100多个字符，但检查这现在是超出了实际问题，这是对的问题。无限循环

Answer 3

符号常数EOF是整数常数，类型为int。它（通常）被定义为一个宏，如-1。

的问题是，作为（32位）的值-1int具有值0xffffffff和作为（8位）char相同的值将是0xff。这两个值不相等。这反过来意味着你的循环条件将永不为假，导致无限循环。

解决此问题的方法是，所有读取字符的标准函数都将其作为int返回。这意味着您的变量character也需要具有该类型。

重要注意事项：如果普通char是有符号或无符号类型，这是一个编译器实现细节。如果它是有符号的，那么与int的比较将导致符号扩展当char值在比较中被提升时。这意味着signed char与值0xff将扩展到int值0xffffffff。这意味着如果char已签名，那么比较就会起作用。

这意味着您的编译有char为unsigned char。所以推广到int后unsigned char值0xff将为0x000000ff。

---

至于为什么值-1变得0xffffffff是因为如何负数的计算机上通常表示，其中一种叫two's complement。

---

您的代码中还有另外两个缺陷。

首先是因为循环是无限的，你会去方式出string阵列的界限，导致未定义行为（以及可能的崩溃迟早）。解决方法是添加一个条件，以确保index永远不会达到100（在数组的特定情况下，应该真正作为参数传递）。

第二个问题是，如果你打算用string数组作为实际字符串，你需要终止它。 C中的字符串实际上被称为null 终止字符串。该终结符是字符'\0'（等于整数0），并且需要放在每个要传递给处理此类字符串的标准函数的字符串的末尾。拥有这个终结符意味着一个100字符数组只能有99个字符，以便能够匹配终结符。这对解决上述问题具有影响。至于如何添加终止符，只需在循环后执行string[index] = '\0';（如果index在当然范围内）。