程序崩溃，指针试图使 strcpy 类似

Question

这是我今天的第二个问题，指针让我做噩梦。 我正在尝试制作一个与 strcpy() 函数执行相同操作的程序。 一旦我尝试它..它崩溃了，我 100% 确定这是我的代码中的指针问题。我想是因为有某种未初始化的指针（*复制）..但是我已经为它分配了 NULL ...所以有人能告诉我 Null 分配的确切用途吗？因为我认为我误解了它的用途。请告诉我可以进行哪些修改才能使程序正常运行。

 #include <iostream>
 using namespace std;

 void mycpy(char *b , char *a);

 int main()
 {

    char *original = "this is a text" ;
    char *copied = 0 ;

    mycpy(copied , original);

    for(int i = 0 ; *(copied+i) ;i++) cout << *(copied+i) ;

    return 0;
}

void mycpy(char *b , char *a){

    for(int i = 0 ; *(a+i) ; i++) *(b+i) = *(a+i);

}

提前致谢 。

Answer 1

好吧，你的 mycpy 几乎是正确的（尽管你可以使用括号而不是算术，即 a[i] 而不是 *(a+1)< /代码>）。为了正确打印复制的字符串，最后一个字符必须为零，但最后一个字符不会被您的函数复制。所以它应该像

void mycpy(char *b , char *a)
{
    int i;
    for(i = 0 ; *(a+i) ; i++) *(b+i) = *(a+i);
    *(b+i) = 0; // or "\0", doesn't matter    
}

此外，变量

char *copied = 0 ;

没有指向有效的内存，所以你从内存位置 0 读取，这是非法的。您可以将变量定义为数组

char copied[20];

Answer 2

您必须为复制操作的结果分配一些内存。

在您的情况下，copied 尚未初始化，因此您尝试写入空指针。以下内容将为您分配足够的内存，将original复制到copied

char* copied = new char[strlen(original)+1];

Answer 3

你有指针，但它们没有指向任何内存。您需要分配一些内存才能使其工作。

char *original;

那只是一个指向 char 类型内存的指针。您不能将其设置为“这是文本”，因为它只是一个指针。它没有任何空间来存储“这是文本”。

char original[ 15 ] = "this is a text";
char copied[ 15 ] = "this is a text";

会起作用，或者

char *original;
char * copied;

original = malloc( 15 );
copied = malloc( 15 );

mycpy( original, "this is a text" );
mycpy( copied, original );

这两种方法都会占用 15 个字节来存储文本。第一个示例使用堆栈中的 15 个字节。第二个示例使用堆中的 15 个字节。

Answer 4

固定的：

 #include <iostream>
 using namespace std;

 void mycpy(char *b , char *a);

 int main()
 {

    char *original = "this is a text" ;
    char copied[30]; // you need to actualy allocate space 
    // (this is on stack, you could use new as well, for heap allocation)

    mycpy(copied , original);

    for(int i = 0 ; *(copied+i) ;i++) cout << *(copied+i) ;

    return 0;
}

void mycpy(char *b , char *a){
    int i =0;
    while (*(a+i)) {
        *(b+i) = *(a+i);
        ++i;
    }
    *(b+i) = '\0'; // null termination
}

Answer 5

在这一行中：

for(int i = 0 ; *(a+i) ; i++) *(b+i) = *(a+i);

b+i 的取消引用导致了错误。您正在取消引用 0 (NULL)，这是数字 1 指针错误。