Question

我创建了一个程序，该程序通过键盘从学生信息中加载数据库，之后我试图创建一个选项以向该数据库添加学生的新注册

结构由

组成

typedef struct {
    long unsigned int aem;
    char name[64];
    short unsigned int lessonsToPass;
} Registration;

我已经在主函数上创建了它的指针

int i, sizeOfDatabase;
Registration *database
scanf("%d", &sizeOfDatabase);
database = (Registration*) malloc(sizeOfDatabase * sizeof(Registration));
for(i = 0; i < sizeOfDatabase; ++i){
    scanf("%lu%63s%hu", &(database+i)->aem, (database+i)->name, &(database+i)->lessonsToPass);
    for(tmp = (database+i)->name; (*tmp=toupper(*tmp)); ++tmp);
}

所以基本上我是从键盘上获取数据库并将其名称更改为大写

之后，我要调用一个函数以添加新的注册

void add(char *aem, char *name, char *lessonsToPass, int currentDatabaseSize, Registration **database){
    char *tmp;
    int newSize = currentDatabaseSize + 1;
    *database = (Registration*) realloc(*database, newSize * sizeof(Registration));
    for(tmp = name; (*tmp=toupper(*tmp)); ++tmp);
    (*database + newSize)->aem = atoi(aem);
    strcpy((*database + newSize)->name, name);
    (*database + newSize)->lessonsToPass = atoi(lessonsToPass);
}

char *名称从不大于64，并且我仍然收到编译错误malloc(): corrupted top size

我想念什么？

Answer 1

您选择了一些不寻常的习惯用法来表示您要尝试做的事情。这是更易于理解并解决问题的代码。注意我假设您有一个现代的C编译器。您的代码样式为C89。那是1989年。如果可以选择，则应该使用较新的C语言功能。

首先创建一个字符串大写方法以防止重复代码：

void strtoupper(char *s) {
  for (char *p = s; *p; ++p) *p = toupper(*p);
}

剩下的：

int sizeOfDatabase;
Registration *database;

scanf("%d", &sizeOfDatabase);
database = malloc(sizeOfDatabase * sizeof *database); // NO CAST HERE!
for (int i = 0; i < sizeOfDatabase; ++i){
  Registration *r = database + i;
  scanf("%lu%63s%hu", &r->aem, r->name, &r->lessonsToPass);
  strtouper(r->name);
}

您应该将 pointer 传递给数据库大小，以便add方法可以更新它。以正确的方式调用realloc时，效率很低，因为每次调用都需要与整个数据库大小成正比的时间。如果它变大了，您会后悔的。但这是我们忽略的一个好点。

void add(char *aem, char *name, char *lessonsToPass,
  Registration **database, int *dbSize) {
  int i = (*dbSize)++; // Increase db size by 1 and get new element's index.
  Registration *db = *database = realloc(*database, *dbSize * sizeof **database);
  Registration *r = db + i;
  strcpy(r->name, name);  // DON'T MODIFY THE INPUT.
  strtoupper(r->name);
  r->aem = atoi(aem);
  r->lessonsToPass = atoi(lessonsToPass);
}

我希望这是有道理的。

纯C-传递结构体指针的引用

1 个答案: