用C++动态分配结构数组

用C++动态分配结构数组,c++,arrays,dynamic-memory-allocation,records,C++,Arrays,Dynamic Memory Allocation,Records,我正在尝试动态分配一组记录。当我使用./a.out运行程序时,它似乎工作正常,但当我尝试运行该程序时,从.txt文件输入数据。/myProg

我正在尝试动态分配一组记录。当我使用./a.out运行程序时,它似乎工作正常,但当我尝试运行该程序时,从.txt文件输入数据。/myProg<它会重复记录整个数组中的第一行文本并崩溃,最终不会打印出所选的书籍。我的猜测是我没有正确创建数组,但我似乎无法找出我的问题所在

struct Book {
    char *title; //entered with no spaces
    int date; // in the form ddmmyy
};

Book *createRecord(int);
void input(Book *, int);
void display(Book *, int, int);
void destroyRecord(Book *);

int main() {
    int arrN = 0;
    int n = 0;

    cout << "Enter size of array: ";
    cin >> arrN;

    Book *bookArr;  
    bookArr = new Book[arrN];

    bookArr = createRecord(arrN);

    input(bookArr, arrN);

    cin.ignore();
    cout << "Book: ";
    cin >> n;

    display(bookArr, arrN, n);  
    destroyRecord(bookArr);

    return EXIT_SUCCESS; 
}


Book *createRecord(int arrN){
    struct Book *bookArr;
    bookArr = new Book[arrN];
    return bookArr;
}

void input(Book *bookArr, int arrN) {
    for(int i = 0; i < arrN; i++){
        char arrFirst[20];
        cin.ignore();
        cout << "Name: ";
        cin.getline(arrFirst, 20);
        strcpy((bookArr[i]).title = new char, arrFirst);
        cout << "Score: ";
        cin >> (bookArr[i]).date;
    }
}


void display(Book *bookArr, int arrN, int n) {
    if (0 <= n && n <= arrN){
        cout << (bookArr[n-1]).title << " " << (bookArr[n-1]).date << endl;
    }
}

void destroyRecord(Book *bookArr) {
   delete [] (bookArr)->title;
   delete bookArr;
}

首先,您要分配两个阵列:

bookArr = new Book[arrN]; // <-- leaked
bookArr = createRecord(arrN);
这就是分配一个字符,您试图将整个字符串复制到其中。这可能是:

// two lines please!
bookArr[i].title = new char[20];
strcpy(bookArr[i].title, arrFirst);
第三:

if (0 <= n && n <= arrN){

首先,您要分配两个阵列:

bookArr = new Book[arrN]; // <-- leaked
bookArr = createRecord(arrN);
这就是分配一个字符,您试图将整个字符串复制到其中。这可能是:

// two lines please!
bookArr[i].title = new char[20];
strcpy(bookArr[i].title, arrFirst);
第三:

if (0 <= n && n <= arrN){
问题1:

修复显示函数中的数组索引问题:

void display(Book *bookArr, int arrN, int n) {
    if (0 <= n && n <= arrN){
        cout << (bookArr[n - 1]).title << " " << (bookArr[n - 1]).date << endl;
    }
}

问题1:

修复显示函数中的数组索引问题:

void display(Book *bookArr, int arrN, int n) {
    if (0 <= n && n <= arrN){
        cout << (bookArr[n - 1]).title << " " << (bookArr[n - 1]).date << endl;
    }
}


不确定输入问题,但您肯定没有按应有的方式创建数组。在代码中,您正在创建一个图书数组,然后在createRecord函数中分配一个新的图书数组,并用它替换原始数组。与预期的情况不同,您最终会得到一系列未初始化的书籍指针

这是你应该做的

// Allocate an array of books
Book *bookArr; 
bookArr = new Book[arrN];

// Preallocate the records
for ( int iBook = 0; iBool < arrN; iBook++ )
{
  bookArr[ iBook ] = createRecord();
}

...


Book *createRecord()
{
  // Allocate one new book
  struct Book *pbook = new book;
  return pbook;
}

不确定输入问题,但您肯定没有按应有的方式创建数组。在代码中,您正在创建一个图书数组,然后在createRecord函数中分配一个新的图书数组,并用它替换原始数组。与预期的情况不同,您最终会得到一系列未初始化的书籍指针

这是你应该做的

// Allocate an array of books
Book *bookArr; 
bookArr = new Book[arrN];

// Preallocate the records
for ( int iBook = 0; iBool < arrN; iBook++ )
{
  bookArr[ iBook ] = createRecord();
}

...


Book *createRecord()
{
  // Allocate one new book
  struct Book *pbook = new book;
  return pbook;
}