c - Malloc() 不起作用 + 字符数组清除

Question

关闭。这个问题需要多问 focused 。目前不接受答案。

---

想要改进此问题吗？更新问题，使其仅关注一个问题 editing this post .

已关闭 7 年前。

Improve this question

第一个问题: 我在 Redis 数据库中有一些键和值。我使用以下命令读取它们:redisReply *reply = redisCommand(c, "HGETALL %s", test_id);然后我想创建一个 char 数组，将它们放入其中，如下所示:“key=value; key2=value2;” ETC。所以我尝试使用 malloc() 为 char 数组分配内存。我执行以下操作:

int token_length;
for (i = 0; i < reply->elements; i = i + 2 ) 
  token_length = strlen(reply->element[i]->str)+strlen(reply->element[i+1]->str)+3;

tokens = (char*)malloc(sizeof(char)*token_length);
memset(tokens, '\0' , sizeof(tokens));

for (i = 0; i < reply->elements; i = i + 2 ) {
      strcat(tokens,reply->element[i]->str);
      strcat(tokens,"=");
      strcat(tokens,reply->element[i+1]->str);
      strcat(tokens,"; ");
  }

我用 token_length 得到了正确的数字，但如果我在 malloc(...) 中使用它，我会得到内存错误等。相反，如果我只是在 malloc() 中输入 150 而不是 token_length ，它会很好用。这是为什么？

第二个问题:如果我不做memset(tokens, '\0' , sizeof(tokens));之后malloc() ，我的 token 数组中有很多“垃圾”。有人应该在分配后总是用 malloc() 清除 char 数组吗？ ？还有另一种也许更好的方法吗？

第三个问题:当我输出数组而不是字符“;”时，我得到\073，这是“;”的八进制表示形式在 ASCII 表中。

Answer 1

存在一些问题 - 我不会单独描述每个错误，而是只发布一个修复版本，您可以将其与原始代码进行比较:

int token_length = 0; // <<< initialise
for (i = 0; i < reply->elements; i = i + 2 ) 
    token_length += strlen(reply->element[i]->str)+strlen(reply->element[i+1]->str)+3; // <<< use += to accumulate length

tokens = malloc(token_length + 1); // <<< need extra char for '\0' terminator
tokens[0] = '\0';                  // (also don't cast result of malloc() in C !)

for (i = 0; i < reply->elements; i = i + 2 ) {
    strcat(tokens,reply->element[i]->str);
    strcat(tokens,"=");
    strcat(tokens,reply->element[i+1]->str);
    strcat(tokens,"; ");
}