在 stl 映射中插入新项目时，会覆盖第一个元素

Question

首先，我有以下代码：

map<char*, char*> records;

// id, value are buffer variables to store the text that returns from `strcpy()` 
char *id = (char*)malloc(sizeof(char*) * 20);
char *value = (char*)malloc(sizeof(char*) * 20);

sql->open(dbFile);
sql->query("SELECT * FROM categories");
while(sql->fetch() != SQLITE_DONE){

    // copy the text that returns from `sql->getValue`, then convert that text from `const unsigned char*` to `const char*` 
    strcpy(id, reinterpret_cast<const char*>(sql->getValue(0)));
    strcpy(value, reinterpret_cast<const char*>(sql->getValue(1)));

    // insert the values that in `id`,`value` as new item
    records.insert(pair<char*, char*>(id, value));           
}
free(id);
free(value);

前面代码的想法是使用strcopy()复制返回此方法sql->getValue(0)的文本，然后将该文本从const unsigned char*转换为const char*，然后将这些文本存储在id，@ 987654327@ 变量，然后插入id、value 作为新项目。 前面的步骤发生在从数据库获取的每个数据上。

进一步澄清这个想法：
例如，从数据库中提取了三行（fruits、vegetables、sweets）。
自然，将这三个数据插入地图容器时，容器中的项目数将是三个项目first(fruits)，second(vegetables )、第三(sweets)。

在我的情况下，当检查项目数量时，我发现只有一项，项目是sweets。

现在，我很惊讶，剩下的物品在哪里？

Answer 1

您将覆盖相同的内存块 3 次，并将指向这些块的指针存储 3 次。就地图而言，每次都是相同的值，因为指针永远不会改变。

看来您真正想要的是拥有一个字符串映射，而不是指针。如果你使用std::string，这会容易得多：

#include <string>

...

map<string, string> records;

string id, value; // size of allocation will be handled by the string class.

sql->open(dbFile);
sql->query("SELECT * FROM categories");
while(sql->fetch() != SQLITE_DONE){
  // string has an assignment operator from C-style strings
  id    = reinterpret_cast<const char*>(sql->getValue(0));
  value = reinterpret_cast<const char*>(sql->getValue(1));

  // more readable way of writing the insert. id, value will be copied into
  // the map.
  records[id] = value;
}

// freeing unnecessary; string class handles it when id, value go out of scope.

这还具有使代码异常安全（至少是字符串）的优点，而无需 try-catch 块。无论作用域如何离开，std::string 的析构函数都会处理清理工作。