统计字符1的个数

下面的程序意图在于统计字符串中字符1的个数，可惜有瑕疵：

#include <stdio.h>#define maxn 10000000+10int main(){    char s[maxn];    scanf("%s", s);    int tot = 0;    for(int i = 0; i < strlen(s); i++)        if(s[i] == 1)            tot ++;    printf("%d\n", tot);    return 0;}该程序至少有3个问题，其中一个导致程序无法运行，另一个导致结果不正确，还有一个导致效率低下。你能找到它们并改正吗？

发现的错误：

1. 数组过大，需要声明在main方法外面。
2. 是s[i] == ‘1’，是字符1。
3. scanf应该包在while里面，符合一般输入输出要求

猜测的错误：
- strlen虽然很高效，但是每次都需要计算一次，当数组非常大时，可能会影响效率。
改正如下：

#include <stdio.h>#include <string.h>#define maxn 10000000+10char s[maxn];int main(){    int tot, len;    while(scanf("%s", s) == 1 ){        tot = 0;        len = strlen(s);        for(int i = 0; i < len; i++)            if(s[i] == '1')                tot ++;            printf("%d\n", tot);    }    return 0;}