自己写的一个关于复制函数的问题？？

kexinqw

#include "stdafx.h"
#include "stdlib.h"
char *strcpy(char *a)
{
        char         *c= (char *)malloc(sizeof(*a));
        char         *b;
        b=c;
        if(*a !='\0' )
                *(c++) = *(a++);       
        *c= '\0';
        return b;       
}
int main()
{
        char a[80],*p;
        scanf("%s",a);
        p=strcpy(a);
        printf("%s",p);
        return 0;
}
运行后，为什么只复制第一个字符？请问问题出在什么地方啊

KBTiller

if(*a !='\0' )
                *(c++) = *(a++);

hellioncu

没用循环，内存也没释放

kexinqw

感谢楼上两位提醒，下面是修改后的代码：

#include "stdafx.h"
#include "stdlib.h"
char *strcpy(char *a)
{
        char         *c= (char *)malloc(sizeof(*a));
        char         *b;
        b=c;
        while(*a !='\0' )
                *(c++) = *(a++);        
        *c= '\0';
                char free(*c);
        return b;        
}
int main()
{
        char a[80],*p;
        scanf("%s",a);
        p=strcpy(a);
        printf("%s",p);
        return 0;
}

kexinqw

#include "stdafx.h"
#include "stdlib.h"
char *strcpy(char *a)
{
        char         *c;
        char         *b;
                b=c=NULL;
        while(*a !='\0' )
                {
                        c= (char *)malloc(sizeof(char));
                        if(b==NULL)
                                b=a;
            *(c++) = *(a++);
                }
        *c= '\0';
        return b;        
}
int main()
{
        char a[80],*p;
        scanf("%s",a);
        p=strcpy(a);
        printf("%s",p);
            free(p);
        return 0;
}


以上代码注释掉free(p);是正确的，放上去会出错，这是为什么？

snriyt

代码是有问题滴,乱啊,

hellioncu

#include "stdafx.h"
#include "stdlib.h"
char *strcpy(char *a)
{
        char         *c;
       ...
kexinqw 发表于 2011-05-13 09:23

    你释放的不是你申请的，strcpy返回值有问题

KBTiller

while(*a !='\0' )
                {
                        c= (char *)malloc(sizeof(char));

nizvoo

#include "stdafx.h"
#include "stdlib.h"
char *strcpy(char *a)
{
        char         *c= (char *)malloc(sizeof ...
kexinqw 发表于 2011-05-13 07:47

    1， malloc分配的長度不夠
    2， 要用循環判斷 *a是否是'\0'

nizvoo

1. 
   
2. #include <stdio.h>
   
3. #include <stdlib.h>
   
4. 
   
5. char *strcpy(char *a)
   
6. {
   
7.         char         *c= (char *)malloc(strlen(a) + 1);
   
8.         char         *b;
   
9.         b=c;
   
10.         while (*a !='\0' )
    
11.                 *(c++) = *(a++);      
    
12.         *c= '\0';
    
13.         return b;      
    
14. }
    
15. int main()
    
16. {
    
17.         char a[80],*p;
    
18.         scanf("%s",a);
    
19.         p=strcpy(a);
    
20.         printf("%s",p);
    
21.         return 0;
    
22. }
    

复制代码