杭电2043哪里错了？http://acm.hdu.edu.cn/showproblem.php?pid=2043

Question

#include<stdio.h>
#include<string.h>
main()
{
int a,b,c,d,i,j,k,e,q,w;
char s[51];
scanf("%d",&a);
while(a)
{
b=0;c=0;d=0;k=0;e=0;
a--;
gets(s);
getchar();
for(i=0;i<strlen(s);i++)
{
if(s[i]>='A'&&s[i]<='Z')
b=1;
if(s[i]>='a'&&s[i]<='z')
c=1;
if(s[i]>=0&&s[i]<=9)
d=1;
if(s[i]==33||s[i]==126||s[i]==64||s[i]==35||s[i]==35||s[i]==37||s[i]==94)
e=1;
}
k=b+c+d+e;
if(k>=3)
printf("YES\n");
else
printf("NO\n");
}
}

Answer 1

#include<stdio.h>
#include<string.h>
main()
{
int a,b,c,d,i,j,k,e,q,w;
char s[51];
scanf("%d",&a);
while(a)
{
b=0;c=0;d=0;k=0;e=0;
a--;
scanf("%s",s); // 用scanf比较安全, gets难处理.
if(strlen(s)>16||strlen(s)<8) // 你忘了检测长度
{
printf("NO\n");
continue;
}
for(i=0;i<strlen(s);i++)
{
if(s[i]>='A'&&s[i]<='Z')
b=1;
if(s[i]>='a'&&s[i]<='z')
c=1;
if(s[i]>='0'&&s[i]<='9') // 0到9是字符, 不是数值
d=1;
if(s[i]=='~'||s[i]=='!'||s[i]=='@'||s[i]=='#'||s[i]=='$'||s[i]=='%'||s[i]=='^') // 如1楼建议, 不要直接ASCII
e=1;
}
k=b+c+d+e;
if(k>=3)
printf("YES\n");
else
printf("NO\n");
}
}

Answer 2

if(s[i]==33||s[i]==126||s[i]==64||s[i]==35||s[i]==35||s[i]==37||s[i]==94)
怎么有两个35？某个ASCII错了吧
其实没必要特意去查他们的ASCII
直接写成
if(s[i]=='~'||s[i]=='!'||s[i]=='@'||s[i]=='#'||s[i]=='$'||s[i]=='%'||s[i]=='^')
多方便，是吧~