c语言 - Malloc 和 Realloc 不起作用 (C11 - CLion)

Question

#include <stdio.h>
#include <stdlib.h>
typedef struct Ogrenciler {
int no;
char adi[50];
char soyadi[50];
double vize;
double final;
double notu;
} Ogr;
int ogrenciSayisi = 0;
void KayitEkle(Ogr *ogrenci) {
int simdikiOgr = ogrenciSayisi;
if (ogrenciSayisi == 0) {
ogrenciSayisi++;
ogrenci = (Ogr *) malloc(ogrenciSayisi*sizeof(Ogr));
} else {
ogrenciSayisi++;
ogrenci = (Ogr *) realloc(ogrenci, ogrenciSayisi * sizeof(Ogr));
}
printf("No:");
scanf("%d", &ogrenci[simdikiOgr].no);
printf("Adi:");
scanf("%s", ogrenci[simdikiOgr].adi);
printf("Soyadi:");
scanf("%s", ogrenci[simdikiOgr].soyadi);
printf("Vize:");
scanf("%lf", &ogrenci[simdikiOgr].vize);
printf("Final:");
scanf("%lf", &ogrenci[simdikiOgr].final);
ogrenci[simdikiOgr].notu = (ogrenci[simdikiOgr].vize * 0.4) + (ogrenci[simdikiOgr].final * 0.6);
printf("Notu: %lf", ogrenci[simdikiOgr].notu);
printf("nn");
printf("Adi: %snNo: %dnVize: %lfnFinal: %lfNotu: %lfn",
ogrenci[simdikiOgr].adi, ogrenci[simdikiOgr].no, ogrenci[simdikiOgr].vize, ogrenci[simdikiOgr].final,
ogrenci[simdikiOgr].notu);
}
int main() {
int c;
while (c != 5) {
printf("n1-tYeni Kayit Eklen2-tKayit Siln3-tKayitlari Listelen4-tOrtalama Hesaplan5-tCikisn");
scanf(" %d", &c);
Ogr *ogrenci;
switch (c) {
case 1:
KayitEkle(ogrenci);
break;
case 2:
KayitSil(ogrenci);
break;
case 3:
KayitListele(ogrenci);
break;
case 4:
OrtHesapla(ogrenci);
break;
case 5:
printf("Cikiliyor");
break;
default:
printf("Gecerli bir girdi yapinizn");
break;
}
}
return 0;
}

如您所见，我使用 malloc() 和 realloc() 作为我的 typedef 结构体，我只能输入一个条目。当我尝试添加新条目(切换情况：1)时，它不起作用并在本节之后崩溃：

printf("No:");
scanf("%d", &ogrenci[simdikiOgr].no);

起初，我尝试使用calloc(ogrenciSayisi*10*sizeof(Ogr))，但它只创建了一个空间。之后，在 realloc 之后的调试器 (CLion) 部分中，ogrenci 指针变为空指针。

编辑： 我不是在尝试返回值。据我所知(int a)等于(int a[ ])，所以KayitEkle(ogrenci)和void KayitEkle(Ogr ogrenci)对我来说似乎是合法的。我的 ogrenci 首先应该是空的，所以 (Ogr *ogrenci=NULL) 正如你说的那样是正确的吗？

编辑2： 在malloc第10节是一个错误。我修好了。我正在尝试一些东西，但我忘记删除它。

Answer 1

你ogrenci指针按值传递给KayitEkle()，你修改它里面的值，但不返回它的修改值给main()。您需要使用指针传递ogrenci值(即。KayitEkle(&ogrenci)) 或将新值返回给被调用的 (即。ogrenci = KayitEkle(ogrenci))。下面的示例是后者。ogrenci指针位于 while 循环内，因此每次循环运行时都会重新初始化它，可能您打算将其放在循环之外，以便保留其值。 局部变量具有未定义的(读取：任何)值，无需初始化，因此如果需要，您需要将 ogrenci 显式初始化为 NULL。请参阅初始化。ogrenci == NULL时不需要检查ogrenciSayisi == 0，因为realloc(NULL, ...)等于malloc(...)。请参阅重新分配。

#include <stdio.h>
#include <stdlib.h>
typedef struct Ogrenciler {
int no;
char adi[50];
char soyadi[50];
double vize;
double final;
double notu;
} Ogr;
int ogrenciSayisi = 0;
// or void KayitEkle(Ogr **ogrenci) and then use *ogrenci
Ogr *KayitEkle(Ogr *ogrenci) {
int simdikiOgr = ogrenciSayisi;
ogrenciSayisi++;
ogrenci = realloc(ogrenci, ogrenciSayisi*sizeof(Ogr));
printf("No:");
scanf("%d", &ogrenci[simdikiOgr].no);
printf("Adi:");
scanf("%s", ogrenci[simdikiOgr].adi);
printf("Soyadi:");
scanf("%s", ogrenci[simdikiOgr].soyadi);
printf("Vize:");
scanf("%lf", &ogrenci[simdikiOgr].vize);
printf("Final:");
scanf("%lf", &ogrenci[simdikiOgr].final);
ogrenci[simdikiOgr].notu = (ogrenci[simdikiOgr].vize * 0.4) + (ogrenci[simdikiOgr].final * 0.6);
printf("Notu: %lf", ogrenci[simdikiOgr].notu);
printf("nn");
printf("Adi: %snNo: %dnVize: %lfnFinal: %lfNotu: %lfn",
ogrenci[simdikiOgr].adi, ogrenci[simdikiOgr].no, ogrenci[simdikiOgr].vize, ogrenci[simdikiOgr].final,
ogrenci[simdikiOgr].notu);
return ogrenci;
}
int main() {
int c = 0;
Ogr *ogrenci = NULL;
while (c != 5) {
printf("n1-tYeni Kayit Eklen2-tKayit Siln3-tKayitlari Listelen4-tOrtalama Hesaplan5-tCikisn");
scanf(" %d", &c);
switch (c) {
case 1:
ogrenci = KayitEkle(ogrenci);
break;
case 2:
ogrenci = KayitSil(ogrenci);
break;
case 3:
ogrenci = KayitListele(ogrenci);
break;
case 4:
ogrenci = OrtHesapla(ogrenci);
break;
case 5:
printf("Cikiliyor");
break;
default:
printf("Gecerli bir girdi yapinizn");
break;
}
}
// it's nice to free things
free(ogrenci);
return 0;
}

Answer 2

有很多错误。如果要使用函数动态分配内存。必须使用指针指针，例如 realloc 行上的指令也不正确，因为如果重新分配失败，则会覆盖旧的内存地址，并且指针采用与 NULL 相同的值。

而且没有跑题。如果您输入的格式预期内容(例如字符而不是数字，反之亦然)，您还应该对 scanf 函数采取预防措施，这肯定会使程序的行为不确定，因此您应该预测这种情况。