c - 结构体的动态内存分配，但它的 int 或 double 成员呢

Question

我将非常感谢您查看我的代码并告诉我，如果我创建一个为其动态分配内存的结构，我是否也应该为该结构的 int 或 double 元素动态分配内存？

从下面的示例中可以看出，我仅对结构中那些大小可以更改的元素(即字符串)使用 malloc。我尝试对 int 或 double 变量(结构的元素)使用 malloc，但一切正常，直到我尝试对它们使用 printf 函数。

它拒绝工作，因为 printf("%d", [struct]->[int element]) 不正确。

所以我更改了代码，并使用 malloc 和 realloc 函数处理 int 和 double 结构成员。这种做法正确吗？

是否可以预期甚至可能对 int 或 double 结构成员/元素使用 malloc 和 realloc？稍后如何对它们使用 printf ？

#include<stdlib.h>
#include<stdio.h>
#include<string.h>
#define LICZBA 2
#define SIZE 256

typedef struct ksiazka{
   char * imie;
   char * nazwisko;
   char * tytul;
   int numerkat;
   double cena;
}rekord;

rekord * rekordnew(){
   rekord *r;
   r=malloc(sizeof(rekord));
   r->imie=NULL;
   r->nazwisko=NULL;
   r->tytul=NULL;
   r->numerkat=0;
   r->cena=0;
   return r;
}

void rekordfree(rekord *r){
   if (r->imie) free (r->imie);
   if (r->nazwisko) free (r-> nazwisko);
   if (r-> tytul) free (r-> tytul);
   //if (r-> numerkat) free (r->numerkat);
   //if (r-> cena) free (r-> cena); 
   free (r);
}

void wpiszimie(rekord *pr, char im[SIZE]){
   (*pr).imie=realloc(pr->imie, strlen(im)+1);
   strcpy(pr->imie, im);
}

void wpisznazwisko(rekord *pr, char nazw[SIZE]){
   pr->nazwisko=realloc(pr->nazwisko, strlen(nazw)+1);
   strcpy(pr->nazwisko, nazw);
}

void wpisztytul(rekord *pr, char tyt[SIZE]){
   pr->tytul=realloc(pr->tytul, strlen(tyt)+1);
   strcpy(pr->tytul, tyt);
}

void wpisznumerkat(rekord *pr, int numer){
 //  pr->numerkat=realloc(pr->numerkat, sizeof(int));
   pr->numerkat=numer;
}

void wpiszcena(rekord *pr, double cen){
 //  pr->cena=realloc(pr->cena,sizeof(double));
   pr->cena=cen;
}


int main(){
  rekord *r[LICZBA];
  char bufor[SIZE];
  int i, number;
  double number2;

  for(i=0;i<LICZBA;i++){
     r[i]=rekordnew();
  }

  for(i=0;i<LICZBA;i++){
    printf("Podaj tytuł książki");
    scanf("%s", bufor);
    wpisztytul(r[i],bufor);
    printf("Podaj imię autora książki");
    if (scanf("%s", bufor)==1)wpiszimie(r[i],bufor);
    printf("Podaj nazwisko autora książki");
    if (scanf("%s", bufor)==1)wpisznazwisko(r[i],bufor);
    printf("Podaj numer katalogowy książki");
    if (scanf("%d", &number)==1)wpisznumerkat(r[i],number);
    printf("Podaj cenę książki");
    if (scanf("%lf", &number2)==1)wpiszcena(r[i],number2);
    }

    for(i=0;i<LICZBA;i++){
    printf("Tytuł książki");
    printf("%s", r[i]->tytul);
    printf("Imię autora książki");
    printf("%s", r[i]->imie);
    printf("Nazwisko autora książki");
    printf("%s", r[i]->nazwisko);
    printf("Numer katalogowy książki");
    printf("%d", r[i]->numerkat);
    printf("Cena książki");
    printf("%lf", r[i]->cena);

  } 
 return 0;
}

Answer 1

我看不到你的代码中有任何严重错误，但如果你用英语编写它，对我来说推理会更容易。

当您执行malloc(sizeof(rekord))时，您会分配内存来保存rekord的所有成员。这是三个 char * 指针、一个 int 和一个 double。您可以立即写信给这些成员，无需进一步分配。当然，写入 char * 成员并不是真正有用，除非您为其分配一个指向某个 char 缓冲区的指针，而您又可以通过 malloc 获取该缓冲区。对于 int 和 double 来说，这既不需要也不可能。所以你的代码在这里是正确的。

不过，有一些小评论:

• 您应该检查malloc的返回值。如果失败，将返回NULL。您应该检查此(罕见)事件并在发生时处理错误。通常，终止程序是适当的操作。
• 您忘记在最后调用 rekordfree。
• 在程序源代码中使用非 ASCII 字符可能有效，也可能无效。在可移植程序中应避免使用它。
• 使用 scanf 读取字符串是一个潜在的安全漏洞，因为它可能会溢出输入缓冲区。由于无论如何您都会将字符串存储在 malloc()ed 缓冲区中，因此我建议您使用 getline作为一种方便且安全的替代方案。
• 考虑在输出中添加更多空格(例如通过 \n 换行)。我想即使我理解这种语言，输出也不会很容易阅读，因为它都在一行上。在输入提示末尾打印空格或制表符 (\t) 也可能有助于改善用户体验。