统计字符1的个数

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

#include <stdio.h>

#define maxn 10000000+10


int 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+10
char 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;
}