1.使用符号常量LEN代替硬编码的魔法值。特别是当你需要在两个地方使用同一个变量时。
1.（不固定）考虑使用unsigned类型（如unsigned或size_t）而不是（有符号）int作为myAsciiList。这更好地捕获了正计数的意图。
1.如果char是signed或unsigned，则它的实现定义。如果它是有符号的，负值将导致缓冲区下溢。您还假设小于128，否则将导致缓冲区溢出。
1.循环之前的代码与循环中的代码相同。这可能是个错误。另外，在循环之前，你没有实现错误处理，这是有问题的（如果它工作的话）。消除重复代码是个好主意。在这种情况下，我可以删除它，其他时候将重复的代码移动到函数中，并根据需要多次调用该函数。

1. scanf()返回EOF。它不会将变量gettingChar设置为它。
   1.如果你想要输出，你需要打印一些东西。
   1.尽可能减少变量的数量和范围。char ch;现在在使用前声明。myIndex被删除，因为它与ch和下面的值相同，我在需要时将其转换为无符号值。
   1.（不固定）打印结果时，请考虑对不可打印字符（如空格、换行符或制表符）的特殊处理。
   你可能需要一些沿着的东西：

#include <stdio.h>

#define LEN 128

int main() {
    int myAsciiList[LEN] = {0};
    for(;;) {
        char ch;
        int rv = scanf("%c", &ch);
        if(rv == EOF)
            break;
        if(!rv) {
            printf("scanf failed\n");
            break;
        }
        if((unsigned) ch >= LEN) {
            printf("ch is too large\n");
            break;
        }
        myAsciiList[(unsigned) ch]++;
    }
    for(size_t i = 0; i < LEN; i++) {
        if(myAsciiList[i])
            printf("%c: %d\n", (char) i, myAsciiList[i]);
    }
}

下面是一个使用源文件作为输入运行的示例：

$ ./count < count.c
        : 41

: 26
 : 38
!: 1
": 8
#: 2
%: 3
&: 1
(: 14
): 14
+: 4
,: 3
.: 1
0: 2
1: 1
2: 1
8: 1
:: 1
;: 14
<: 2
=: 6
>: 2
A: 4
E: 5
F: 1
L: 8
N: 4
O: 1
[: 4
\: 3
]: 4
_: 1
a: 10
b: 3
c: 16
d: 7
e: 11
f: 13
g: 3
h: 8
i: 37
k: 3
l: 3
m: 5
n: 18
o: 5
p: 3
r: 14
s: 15
t: 13
u: 3
v: 3
y: 4
z: 1
{: 6
}: 6