Question

我想将字符串数组的每个元素复制到另一个字符串数组，但是当我复制一个字符串时，整个数组将更改为该字符串。以下是代码集。

这是有问题的代码行：

strcpy(MsgList[i].ga_data, a_Database[i].ga_data);

当我检查内容时 a_Database [i] .ga_data，如下所示

"1240,message 7:War of the worlds"

"1238,message 5:Life of this world"

"1236,message 3:world is not enough"

"1235,message 2:What a world!"

因此，不是在MsgList[i].ga_data中填充相同内容，而是仅仅是“1235，消息2：真是个世界！” 4次，这是复制的最后一个元素。

typedef enum
{
  ACK,
  NACK,
  DELETE
}eMsgStatus_t;

typedef struct _Message
{
  eMsgStatus_t status;
  char *ga_data;
  uint16_t time;
}Message_t;
Message_t MessageList[8]={
  {ACK,"1234,message 1:Hello world",1000},
  {NACK,"1235,message 2:What a world!",1011},
  {NACK,"1236,message 3:world is not enough",1022},
  {ACK,"1237,messsge 4:Cruel world",1033},
  {NACK,"1238,message 5:Life of this world",1044},
  {ACK,"1239,message 6:Around the world in 80 days",1055},
  {NACK,"1240,message 7:War of the worlds",1066},
  {ACK,"1241,message 8:End of World",1077}
}; 

Message_t a_Database[20];
Message_t MsgList[20]= {0};

int main()
{
  for (i = 0; i < idx /* total unread message */; ++i)
  {
      strcpy(MsgList[i].ga_data, a_Database[i].ga_data);
      MsgList[i].ga_data[14] = '\0';
  }
}

uint8_t  GetMessages (Message_t *pg_Message)
{
  char i;uint8_t idx = 0;
   for (i = 0; i < 8 /* total message */; ++i)
  {
    if(NACK == MessageList[i].status)
    {
      pg_Message[idx].status = MessageList[i].status;
      pg_Message[idx].ga_data = MessageList[i].ga_data;
      pg_Message[idx].time = MessageList[i].time;
      idx++;
    }
  }
  return idx;
}

Answer 1

首先，你在这里破坏了记忆：

strcpy(MsgList[i].ga_data, a_Database[i].ga_data);

因为我看不到MsgList [i] .ga_data的分配位置，我认为它不是。

然后用幻数＆＃39; 14＆＃39;制作空终止字符串这里：

 MsgList[i].ga_data[14] = '\0';

也可能导致一些内存损坏。

甚至假设你在某个地方进行分配，你可以在你的MsgList [i] .ga_data中观察完整的字符串是非常奇怪的，因为你在14处进行空终止它应该是这样的：

1235,message 2                                                            
1236,message 3
1238,message 5 
1240,message 7

为什么strcpy无法正常工作

1 个答案: