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

Question

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

从下面的例子可以看出，我只对结构中可以改变大小的元素使用 malloc，即字符串。我尝试将 malloc 用于 int 或 double 变量（结构的元素），但在我尝试对它们使用函数 printf 之前一切正常。

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

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

是否可以预期甚至可以将 malloc 和 realloc 用于 int 或 double 结构成员/元素？以后如何在它们上使用 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) 也可能有助于改善用户体验。