C：试图将指针值复制到另一个指针中，得到可修改的左值错误

Question

我正在学习 C 并且遇到困难。基本上我声明了 2 个数组，我想通过使用指针将一个数组的所有 char 值复制到另一个数组中。我正在利用数组名称是指向其第一个元素的指针这一事实。有人可以帮忙看看吗？谢谢。

#include<stdio.h>
#include<string.h>
char *mystrcopy(char*p, char*q);   //function prototype

int main(void)
{
    char s1[20] = "Hello\n";   
    char s2[20] = "Bye\n";

    puts(s1);    //print Hello

    mystrcopy(s1, s2);  //call function using pointers s1 and s2

    puts(s1);   //I want this to print Bye after function has run

    system("pause");   //"press any key to continue...."

    return 0;
}

char *mystrcopy(char*s1, char*s2)
{
    int i;
    for (i = 0; s1 + i; i++)    //for loop continues as long as s1+1 not 0
        s1 + i = s2 + i;   //error says s1 is not a modifiable lvalue

    return s1;   //I know this could be a void function but I choose not to
}

我认为这很好，因为 (s1 + i) 是指向数组 s1 中的第 i 个元素的指针，并将其替换为 s2 中的第 i 个元素。

我尝试过使用

*(s1+i) = *(s2+i)

相反，将 s2 的地址复制到 s1 中，但我得到了同样的错误。

编辑：

上面的for循环中的代码确实有效

*(s1+i) = *(s2+i)

我认为这不起作用，因为循环一直在进行。不过，感谢大家回答我关于可修改 l 值的问题。

Answer 1

这样的事情会让你开始。请注意，我们必须取消引用副本中的 char*。这假定字符串长度固定，因此不需要使用 std 库中的 malloc 或 strlen 函数。不理想，但您可以开始使用。

#include<stdio.h>
#include<string.h>

    char *mystrcopy(char* p, char* q);   //function prototype

    #define LENGTH 20

    int main(void){

    char s1[LENGTH] = "Hello\n";   
    char s2[LENGTH] = "Bye\n";

    puts(s1);    //print Hello

    mystrcopy(s1, s2);

    puts(s1);   //I want this to print Bye after function has run

   // system("pause");   //"press any key to continue...."

    return 0;
}

char *mystrcopy(char* s1, char* s2)
{
    int i;
    for (i = 0; i < LENGTH; i++, ++s1, ++s2)    //for loop continues as long as s1+1 not 0
    *s1 = *s2;   //error says s1 is not a modifiable lvalue

    return s1;   //I know this could be a void function but I choose not to

}

Answer 2

由于s1 是一个指针，那么s1 + 1 也产生一个指针（只是一个地址已被移动的临时指针。

如果我们放置一些中间变量，那么您的循环实际上将等效于：

for (i = 0; s1 + i; i++)
{
    char* t1 = s1 + i;
    char* t2 = s2 + i;
    t1 = t2;
}

需要注意的一个主要因素是它不会改变指向的数据中的任何内容。它只是将一个指针的值分配给另一个。

现在回顾s1 + 1 = anything 时，它试图将某些内容分配给临时指针（而不是临时指针指向的内容）。所以编译器抱怨作为一种保障，因为试图分配给一个临时的总是一个错误。

所以知道你真正想要做什么（同时注意s1 + 1的循环条件不是读取指向的值，代码需要调整。对于这种情况，类似数组的语法允许使用指针比尝试增加指针本身要方便得多。因此你最终会得到类似的东西：

for (i = 0; s1[i]; i++)
{
    s1[i] = s2[i];
}

---

作为额外考虑，您的逻辑从s2 复制到s1，但循环以s1 为条件。那么如果s1 中的初始数据比s2 中的数据短，会发生什么？也许你的循环条件应该是s2。

如果指向s1 和s2 的数组大小不同，未来的考虑是什么。会出现什么样的问题？也许您的函数可以采用一个额外的参数来指定s1 的最大大小。

最后一个考虑因素是，std::string 比使用 char 数组自己做更容易、更安全。