描述：
给你一个字母不重复的字符串，再给你一个字母，请你编写程序输出该字母在字符串中第一次出现时候的位置，如果不在该字符串中，则输出-1 ；本题区分大小写。
输入：
一个长度不大于20的字符串，和一个字母，它们中间用一个空格格开
输出：
输出要查找的字母在字符串中第一次出现时候的位置，最后输出一个回车
输入样例：
English g
输出样例：
3

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
    char s[20];
    char a;
    int i,j=0;
    for(i=0;i<strlen(s);i++)
    {
        scanf("%c",&s[i]);
    }
    scanf("%c",&a);
    for(i=1;i<=strlen(s);i++)
    {
        if(s[i]==a)
        {
            printf("%d\n",i);break;
        }
        else
        {
            j++;
        }
    }
    if(j==strlen(s))
    {
        printf("-1\n");
    }
    return 0;
}
运行时无错误，但是出不来正确结果。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
    char s[20];
    char a;
    int i,j=0;
    for(i=0;i<strlen(s);i++)   /* strlen(s)此时是一个不确定值，仔细看一下strlen(s)用法 */
    {
        scanf("%c",&s[i]);    

    }
    scanf("%c",&a);       /* 空格怎么处理的？？*/
    for(i=1;i<=strlen(s);i++)        /* i=1，难道s[0]你不判断了？*/
    {
        if(s[i]==a)
        {
            printf("%d\n",i);break;
        }
        else
        {
            j++;                /*这个变量没有必要*/
        }
    }
    if(j==strlen(s))            /*判断i==strlen(s)即可*/
    {
        printf("-1\n");
    }
    return 0;
}

问题不少，我改了一下，供参考

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
    char s[20];
    char a;
    int i;
    scanf("%s",s);
    getchar();     /* 吸收分隔的空格 */
    scanf("%c",&a);
    for(i=0;i<strlen(s);i++)
    {
        if(s[i]==a)
        {
            printf("%d\n",i+1);break;
        }
    }
    if(i==strlen(s))
    {
        printf("-1\n");
    }
    return 0;
}

[ 本帖最后由 share32 于 2012-4-9 02:29 编辑 ]

你的代码没有对用户输入的字符串长度做限制，以后编程时要注意异常情况对程序的破坏。

改她现有的错误而已,

如果认真一些, 因该通过子函数完成校验,计算过程.

不过这个不适她现在关注的重点吧.

既然把完整修改的代码贴出来了，就尽量完美，一点建议，别介意

你补充吧