求“ 警告：比较指针和整数”的解决方法

pc521

1. 
   
2. #include<stdio.h>
   
3. #include<ctype.h>
   
4. main()
   
5. {
   
6.         char s[]="12as056;^*&";
   
7.         int i;
   
8.         for(i=0;s[i]!=NULL;i++)
   
9.         {
   
10.                 if(isalpha(s[i]))
    
11.                 {
    
12.                         printf("%c is a number or character.\n",s[i]);
    
13.                 }
    
14.         }
    
15. }
    

复制代码

编译时出现以下警告，怎么解决阿？
practice24.c: In function ‘main’:
practice24.c:7: 警告：比较指针和整数

jtkk

比较指针和整数是因为这句：if(isalpha(s))，isalpha需要一个int参数，你却传了数组指指针；
for循环是个死循环！

daybreakcx

我都怀疑你是不是插入代码部分命令没打全，导致显示那个方括号i被隐掉了

okocha-jay

for那里应该是 s[i] != '\0' 吧。

proge

首先动态字串和字符指针一样，都需要分配、回收空间。所以这样写
char s[]="12as056;^*&";
应该会导致Segment Fault。
其次，循环中没有改变s，所以要么是死循环（假设前面正确的对s赋了值），要么不会进入循环。

我猜你需要的程序应该这样写：

1. #include <stdio.h>
   
2. #include <ctype.h>
   
3. 
   
4. main()
   
5. {
   
6.   char s[11] = "12as056;^*&";
   
7.   char* p = s;
   
8.   char c;
   
9.   for(c=*p;c;c=*++p)
   
10.   {
    
11.     printf("%c\n", c);
    
12.     if(isalpha(c))
    
13.     {
    
14.       printf("%c is a character.\n", c);
    
15.     }
    
16.   }
    
17. }

复制代码

在Ubuntu 9.10 x64 GCC 4.3上测试过。

proge

另外再补充一点，象身份证号到底应该定义为char s[19]，还是char s[18]？可以就拿这个程序做个试验。定义为char s[18]错了？这个问题我和某个面试我的牛X争论过，他说还要一位来存"\0"……后来还出口成脏……

解释如下：
C/C++中数组下标是以零开始的，所以char s[18]实际上定义了19个元素的数组，并由系统自动补零。
但是如果使用char s[]或者char *s，都需要s = malloc(19)。

人可以无知，但不要无理。技术争论是有意义的，但到了人身攻击就无聊了。

starwing83

char s[] = "...."是可以的，这个是C的一个特例：如果给出了初始化序列，则可以不写数组大小。

另外，这段代码显然是从论坛复制的，论坛[i]是斜体的意思，从而丢失了这几个字符……

w_anthony

回复 6# proge


    我觉得你需要面壁思过下了……

想飞的蜗牛

char s[18]实际上定义了19个元素的数组