Question

这是我练习经典“ C编程语言”的1-14的解决方案。应该返回输入中字符的频率。它可以这样做，但是会一直打印完整的频率列表，直到编译器停止为止。我试图找到无限循环的源头。在哪里？

#include <stdio.h>

char j;
int i;
int tally;
char input[100];
int k;

int main ()
{
for (k = 0; k < 100; k++)
{
    input[k] = getchar();
}

for (j = 0; j <= 127; j++)
{
    tally = 0;
    for (i = 0; i < 100; i++)
    {
        if (input[i] == j)
        { 
            tally++;
        }
    }
    if (tally != 0)
    {
        printf ("%c : %d times\n", j, tally);
    }
}
return 0;
}

Answer 1

问题出在您的变量j上，其类型为char：

for (j = 0 ; j <= 127 ; j++)

在已签名char的系统上，char的最大值为127，这意味着j <= 127条件始终为真。

尽管您可以通过将j声明为unsigned char来解决此问题，但是还有一种更好的计数字符的方法：制作一个包含128个计数器的数组，然后遍历input[]一次：

unsigned char input[100];
int tally[128] = {0};
for (int i = 0 ; i < 100 ; i++) {
    if (input[i] < 128) {
        tally[input[i]]++;
    }
}
for (int c = 0 ; c != 128 ; c++) {
    if (tally[c] != 0) {
        printf("%c : %d times\n", c, tally[c]);
    }
}

我的角色频率程序似乎有一个无限循环

1 个答案: