#include<stdio.h>
#include<string.h>
void ma(char *p);
void main()//完成小写字母转换成大写字母功能
{
    char a[100];
    printf("请输入字符：");
    scanf("%s",a);
    ma(a);
    printf("转换后的字符：%s\n",a);
}
void ma(char *p)
{
    char s[100];
    int i,j;
    p=s;
   
    for(j=0;j<10;j++)
    {    if(*p>='a'&&*p<='z')
            *p=*p-32;
             p++;
    }
}

#include <stdio.h>
 main()
 {
     char a[50],*p;
     printf("请输入字符串：");
     scanf("%s",a);
     for(p=a;*p!='\0';p++)
     {
         if(*p>='a'&&*p<='z')*p=*p-32;
     }
     printf("新字符串为:%s",a);
 }

void ma(char *p)
{
    for(;*p;p++)
        if(*p>='a'&&*p<='z')
            *p=*p-32;
}

请问我的代码是错在哪呢？是指针的问题吗？

问题在这一句 p=s; 你只是对空的数组s[100]做了转换，去掉这句就好了