#include<stdio.h>
#include<string.h>
int main()
{
    char *q, *p;
    scanf("%s%s",q,p);
    printf("%s",strcpy(q,p));
     return 0;
}

char *strcpy(char *strDest,char *strSrc)
{
    char *address=strDest;
    while(*strSrc!='\0')
    {
        *strDest=*strSrc;
        strDest++;
        strSrc++;
    }
    *strDest='\0';
    return address;
}

char *strcpy(char *strDest, const char *strSrc)
{
    char *address=strDest;
    while (*address++ = *strSrc++)
        ;
    return strDest;
}

“写了输入语句，从来不想办法去看看输入效果是否与愿望相符的，太多了，已经算是论坛常见毛病。编程的良好习惯没培养好，都不知师从何家的。其实排错也是从检查输入了什么开始呀。”    －－－引自下岗又上岗的版主

我感觉不是这里的错吧，应该是输入出错了

函数中的局部变量是不能作为函数的返回值返回的

只看3楼就行了

char *q, *p;
    scanf("%s%s",q,p);

这里的p q 接收到的数据（假设能够接收到） 往哪里放呢  没有仓库啊

/* #include<string.h> */

你不是编写strcpy吗？？？那你直接上<string.h>,用strcpy什么意思啊？？？用现成的函数还用写什么串拷贝啊！！！再说了，你怎么还用关键字当函数的名字啊！！！你这个编程的习惯方式比我还差！！！我都够菜鸟了，你这是还没有孵化的蛋啊（玩笑）！！！建议你练习这么简单的东西时还是用定义一个字符串方式来吧！！！上来就指针啊，慢慢来吧！！！我改了下，你看看是你想实现的意思不（本人菜鸟，说的不一定对，你就挑对的用吧）：
#include<stdio.h>
//#include<string.h>
//char *strcpy(char *strDest,char *strSrc)
char *strcp(char *strDest,char *strSrc)
{
    char *address=strDest;
    while(*strSrc!='\0')
    {
        *strDest=*strSrc;
        strDest++;
        strSrc++;
    }
    *strDest='\0';
    return address;
}
int main()
{
    char a[5],b[6];
    char *q=a, *p=b;
    scanf("%s%s",q,p);
    //printf("%s",strcpy(q,p));
     printf("%s",strcp(q,p));
     return 0;
     
}

strcpy不是关键字。

#include<stdio.h>
//#include<string.h>    //这句不需要

char *strcpy(char *strDest,char *strSrc)
{
    char *address=strDest;
    while(*strSrc!='\0')
    {
        *strDest=*strSrc;
        strDest++;
        strSrc++;
    }
    *strDest='\0';
    return address;
}

int main()
{
    char *q, *p;    //这里只是定义了2个指针变量，并没有分配空间。
    char cq[100],cp[100];
    q=cq;
    p=cp;    //分配空间
    //scanf("%s%s",q,p);    //不需要2个都输入
    scanf("%s",p);

    strcpy(q,p);
    printf("%s",q);    //输入p，输出q，更能体现字符串的复制结果
    return 0;
}