我这样做对吗？ - C代码

Question

当我运行这个程序时，它说大小是 4，但实际上是 6。它在这里这样做：

printf("String Size: %u\n", sizeof some_string.basic_string);

我是 C 内存分配的新手，以前从未使用过 malloc。我使用 malloc 对吗？

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <string.h>

typedef struct String String;

struct String {
  char *basic_string;
};

String String_New(char basic_string[]) {
  String temp;
  temp.basic_string = (char *) malloc(sizeof basic_string);
  strcpy(temp.basic_string, basic_string);
  return temp;
}

void String_Delete(String *string) {
  free(string->basic_string);
  string->basic_string = NULL;
}

int String_GetSize(String string) {
  int i = 0, s = 0;
  while (string.basic_string[i] != '\0') {
    i++;
    s++;
  }
return s;
}

int main(int argc, char *argv[]) {
  String some_string = String_New("hello");
  printf("String Literal: %s\n", some_string.basic_string);
  printf("String Size: %u\n", sizeof some_string.basic_string);
  printf("String Length: %d\n", String_GetSize(some_string));
  String_Delete(&some_string);

  if (some_string.basic_string == NULL) {
    return 0;
  }

  return 1;
}

Answer 1

char *basic_string; 是一个指向某个内存的指针，该指针的大小（在 32 位系统上）是 32bits = 4bytes。 Sizeof 不知道您在此地址保留的内存大小。

通常没有办法获得 malloc 保留的内存大小 - 尽管您的系统可能有一些私有调试功能来执行此操作。

Answer 2

不，String_New 中的 malloc 应该使用字符串的实际运行时长度，例如

String String_New(char basic_string[]) {
  String temp;
  int length = strlen(basic_string);
  temp.basic_string =
     (char *) malloc(length+1); /* 1 more byte for terminating \0 */
  strcpy(temp.basic_string, basic_string);
  return temp;
}

，但您应该只使用 strdup 例如temp.basic_string = strdup(basic_string);

Answer 3

在你的辅助函数 String_New() 中，当你说“sizeof(basic_string)”时，你要求的是指针的大小而不是字符串的大小，这就是为什么你得到 4 而不是 6。我认为你想要的是this:-

String String_New(const char* basic_string) {
  String temp;
  temp.basic_string = (char *) malloc(strlen(basic_string)+1);
  strcpy(temp.basic_string, basic_string);
  return temp;
}

sizeof 在编译时测量类型的大小；您的字符串参数将是运行时变量。

Answer 4

我认为您期望 sizeof 执行与 strlen() 相同的操作，或者您的 String_GetSize() 执行的操作。这不是它的目的。

您正在对 char* 指针执行 sizeof，即“这个 char* 的大小是多少”，这是一个与“这个char*指向的字符串有多长”。在大多数（如果不是全部）32 位平台上，sizeof(char*) 实际上是 4。

Answer 5

在 C 中，“字符串”不是真正的数据类型。 sizeof 运算符采用数据类型或具有“类型”作为操作数的对象。在您的情况下，对象是 some_string.basic_string ，其类型为 char*，系统上指针的大小为 4。

解决方案是将 String 结构定义为有一个大小成员：

struct String {
  char *basic_string;
  size_t length ;
};

并在 String_New() 中分配时存储大小。这将简化并提高 String_GetSize() 函数的效率（由于 s == i，该函数已经过于复杂）。

还要注意，在 String_New() 中，basic_string 参数也是一个指针（尽管其签名中使用了“数组语法”）。我会避免使用这种语法，它具有误导性，因为在 C 中，除非将数组嵌入到结构中，否则无法通过复制传递数组；当作为参数传递时，数组总是“降级”为指针。此外，在任何情况下调用者都可以传递指针而不是数组。因此，在大多数情况下，您分配的内存太少（4 字节）。您应该使用 strlen() 或最初在 String_GetSize() 中使用的方法来确定长度。

String String_New(char* basic_string) 
{
  String temp;
  temp.length = strlen( basic_string ) ;
  temp.basic_string = (char *) malloc( temp.length + 1 );

  strcpy(temp.basic_string, basic_string);
  return temp;
}

size_t String_GetSize(String string) 
{
    return string.length ;
}

Answer 6

我认为你的函数String_New有问题，当你将basic_string传递给String_New时，它实际上是一个指针，即char *。所以，在32位机器中，sizeof(char *)是4