传递指针和复制值

Question

我想我在这里做了一些愚蠢的事情，请帮助我理解它是什么，我遇到了崩溃：

char *data="xyz";
int EEIndex=0;
int getEEPROMindex();
void updateEEPROMindex();
void getCmd(char *cmd);
void getcmdAtIndex(int index,char *cmd);


        int main(int argc, const char * argv[]) {

        getCmd(data);
        printf("%s  ",data );
        return 0 ;
    }

        void getCmd(char *cmd)
        {
            getcmdAtIndex(EEIndex, cmd);
         }

        void getcmdAtIndex(int index,char *cmd)
        {
            char *EEPROM[]={"A","E","C","D"};

            strcpy(cmd, EEPROM[index]);
            EEIndex=index+1;

        }

Answer 1

习惯上将字符串文字设为只读。那就是：

char *data="xyz";

当您的代码尝试更改字节 x、y 和 z 时，操作系统会崩溃。

要使其可写，请将指针替换为数组；您还应该指定数组的大小。最简单的方法是：

char data[] = "xyz"; // will hold a maximum of 3 bytes (size is implicit)

或

char data[20]; // will hold a maximum of 19 bytes plus an end-of-string byte

你也可以使用strdup:

char* data;
...
strdup(cmd, EEPROM[index]); // instead of strcpy

但这涉及动态内存分配，我认为您不想打扰。

Answer 2

你strcpy正在进入静态内存空间，即

char *data="xyz";

也就是说，您创建了一个字符串xyz，它在编译时存储在保留的内存位置。您无法写入该内存位置。您想分配新空间，例如 malloc。

Answer 3

char *data="xyz"; 定义了一个指向“xyz”的字符指针，它位于只读区域，因此您无法写入该位置。哪一点你还坚持？

将其更改为 char data[] = "xyz" 确保此文字被复制到堆栈 - 所以它是可修改的。

另外，getcmdAtIndex 函数并不安全。超出索引位置怎么办？