尝试在C函数中复制字符串时出现分段错误

时间:2012-06-24 09:42:37

标签: c segmentation-fault strcpy

我在' C'应该实现我自己的strcpy程序。这是我写的。但是,我无法调试分段错误的原因。

#include <stdio.h>
#include <stdlib.h>

char * mystrcpy(char *dest, char *src) {
    char * ptr = dest;
    while (*src != '\0'){
        *ptr = *src;
        ptr++; src++;
        //printf("Dest is \"%s\" and Source is \"%s\"\n",dest,src);
    }
    *ptr = '\0';
    return dest;
}


int main() {
    char str[] = "I rock always";
    char * dest = NULL;
    dest = mystrcpy(dest, str);
    printf("Source String %s and Destination String %s\n", str, dest);
}

有人可以向我解释这种行为吗?

5 个答案:

答案 0 :(得分:4)

您必须为目标字符串分配内存:

int main() {
    char str[] = "I rock always";
    char * dest = (char*)malloc(strlen(str) + 1);
    dest = mystrcpy(dest, str);
    printf("Source String %s and Destination String %s\n", str, dest);
}

当然,最终释放记忆是个好方法:

    free(dest);

答案 1 :(得分:1)

您永远不会为*dest分配任何内存来指向。

它开始指向NULL,然后你尝试:*ptr = *src

答案 2 :(得分:0)

您必须为目标字符串缓冲区分配内存。

内存管理是C的一个重要组成部分...您需要非常小心,因为它在使用时会分配和释放内存,并且会变得过剩。如果不这样做将导致分段错误(无法分配内存)和内存泄漏(无法释放内存)

这里给出的其他答案给出了malloc的例子,所以我不再赘述了。

您应该尽可能使用已经可用的功能,而不是自己编写。这可以避免实现中的错误,因为有人已经为您调试并优化了它。

http://www.cs.cf.ac.uk/Dave/C/node19.html

答案 3 :(得分:0)

分配内存并尝试....

         #include <stdio.h>
         #include <stdlib.h>

         char * mystrcpy(char *dest, char *src) 
         {
                     char * ptr = dest;
                     int index=0;
                     while (*src != '\0')
                     {
                         *(ptr+index) = *(src+index);
                           index++;
                     }
                     *(ptr+index) = '\0';
                     return dest;

          }




          int main() 
          {
                char str[] = "I rock always";
                char * dest = (char*)malloc((strlen(str)+1)*sizeof(char));
                dest = mystrcpy(dest, str);
                printf("Source String %s and Destination String %s\n", str, dest);
                free(dest);
                return 0;
          }

答案 4 :(得分:0)

#include <stdio.h>

    /* this the K&R version of strcpy */
char * mystrcpy(char *dest, char *src) {
    char *ptr = dest;
    while ( *ptr++ = *src++ ) {;}

    return dest;
}


int main(void) {
    char str[] = "I rock always";
    char dest[sizeof str];
    char *result;

    result = mystrcpy(dest, str);
    printf("Source String %s and Destination String %s and result %s\n", str, dest, result);
    return 0;
}