#include<stdio.h>
#include<string.h>
int countchars(char *s1, char ch);
int countchars(char *s1, char ch)
{
        int i,s=0;
        for(i=0;s1[i]!='\0';i++){
            while(ch>='a'||ch<='Z'){
            if(ch=s1[i])
                s=s+1; }}
        return s;

}        
main()
{
    char *s1,ch;
    printf("Input ：",*s1);
    printf("Please input :",ch);
    scanf("%c",&ch);
    printf("%d",countchars( s1,ch));
}

int countchars(char *s1, char ch); 这个函数已经写在main()函数前面了,不用再申明了吧,会不会是再申明就多余了.

我又修改了一下：main()
{
    char *s1,ch;
    printf("Input ：");
    gets(s1);
    printf("Please input :");
    scanf("%c",&ch);
    printf("%d",countchars(s1,ch));
}
但还是不行啊。。

我是新手，觉得以下两点错误:一,S=S+1后面有两个}，到此函数结束，return语句没包含在函数里。二,gets(s1)这句，对一个指针变量赋值以字符串不对吧

[ 本帖最后由 sala0127 于 2011-11-17 09:10 编辑 ]

我也是一个新手，有两点说明：

1.你的书写格式不正确，正确格式应该是：

        int main(void)
        {
             //语句
             return 0;
         }
2.接着，你编写的countchars函数是一个死循环函数，虽然不知道你函数的用意，但是：
  
         for(i=0;s1[i]!='\0';i++){
            while(ch>='a'||ch<='Z'){
            if(ch=s1[i])
                s=s+1; }}

按照你的代码就，if括号内的表达式是一个赋值语句（=是赋值的意思），把s1[i]的值赋给ch，所以只要s1[i]的值不为0，if括号内表达式的值恒为非零，执行s = s + 1.而while内的判定语句是针对ch的，所以只要任何一个s1[i]满足>='a'或者<='Z'则ch也满足循环条件，while循环就会一直进行下去。我想这才是不出结果的主要原因吧

哦哦。。知道错误了，谢谢大家！！

直接对指针变量赋值字符串不可以

格式貌似不正确了。。。
 printf("Input ：",*s1);//你的意思是提示输入，你确没有执行读入的代码
 一般字符没这样用的，写得有点像数字类型了。
还有你没有定义数组的长度，你那个循环是终止不了的