下面的程序意图在于统计字符串中字符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个问题,其中一个导致程序无法运行,另一个导致结果不正确,还有一个导致效率低下。你能找到它们并改正吗?
发现的错误:
- 数组过大,需要声明在main方法外面。
- 是s[i] == ‘1’,是字符1。
- 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;
}
时间: 2024-10-28 21:48:52