char数组的动态内存分配

Question

我已经手动给出了数组大小，如下所示：

int main(int argc, char *argv[] )
{
    char buffer[1024];
    strcpy(buffer,argv[1]);
    ...
}

但是如果参数中传递的数据超过这个大小，它可能会产生问题。

这是动态分配内存的正确方法吗？

int main(int argc, char *argv[] )
{
    int length;
    char *buffer;
    length = sizeof(argv[1]); //or strlen(argv[1])?
    buffer = (char*)malloc(length*sizeof(char *));
    ...
}

Answer 1

sizeof 告诉你char* 的大小。你想要strlen 而不是

if (argc < 2) {
    printf("Error - insufficient arguments\n");
    return 1;
}
length=strlen(argv[1]);
buffer = (char*)malloc(length+1); // cast required for C++ only

我在这里提出了一些其他更改

• 您需要为空终止符在buffer 添加一个额外的字节
• 您应该检查用户是否传入了argv[1]
• sizeof(char *) 在计算字符串所需的存储空间时不正确。 C字符串是chars的数组，所以需要sizeof(char)，保证为1，所以不需要乘以它

或者，如果您在与 Posix 兼容的系统上运行，您可以简化操作并改用 strdup：

buffer = strdup(argv[1]);

最后，当你完成它时，一定要free这个内存

free(buffer);

Answer 2

正确的方法是使用std::string，让C++为你完成工作

#include <string>

int main()
{
    std::string buffer = argv[1];
}

但如果你想以艰难的方式做到这一点，那么这是正确的

int main()
{
    int length = strlen(argv[1]);
    char* buffer = (char*)malloc(length + 1);
}

不要忘记为 C 样式字符串中使用的空终止符 +1。

Answer 3

在 C++ 中，您可以这样做以使您的论点具有良好的数据结构。

const std::vector<std::string>(argv, argv + argc)

Answer 4

length= strlen(argv[1]) //not sizeof(argv[1]);  

和

//extra byte of space is to store Null character.    
buffer = (char*)malloc((length+1) * sizeof(char));

由于sizeof(char) 总是一，你也可以使用这个：

  buffer = (char*)malloc(length+1);                       

Answer 5

首先，如果您使用 C++，我认为最好使用 new 而不是 malloc。

其次，你的 malloc 大小是错误的：buffer = malloc(sizeof(char) * length); 因为你分配的是 char 缓冲区而不是 char* 缓冲区。

第三，您必须为字符串的末尾多分配 1 个字节并存储 '\0'。

最后，sizeof 只获取类型的大小而不是字符串，必须使用 strlen 获取字符串大小。

Answer 6

您需要添加一个额外的字节来保存字符串的终止空字节：

length=sizeof(argv[1]) + 1;

那应该没问题。