Как в C++ создать массив структур неизвестной длины внутри функции?

Question

[YermolaevG]

В теле main() я создаю переменную-указатель на структуру, которая передается в функцию, где вводится количество элементов массива и создается массив. При попытке ввода поля в первый элемент массива программа вылетает. Где у меня ошибка, и возможно ли вообще так создать этот массив?

struct publisher{
    int year;
    char name[maxL], city[maxL];
};

struct book{
    publisher pubhouse;
    char name[maxL], auth[maxL];
    int price, page_n;
};

void input (book *books, int *n){
    int i;
    printf("Введите количество книг: ");
    scanf("%d", n);
    books = new book[*n];
    printf ("\nВведите данные о книгах:");
    for (i = 0; i < *n; i++){
        printf("\nКнига #%d\n", i + 1);
        printf("Введите название книги: ");
        scanf("%s", *books[i].name);
        printf("\nВведите имя автора: ");
        gets(books[i].auth);
        printf("\nВведите цену книги: ");
        scanf("%d", books[i].price);
        printf("\nВведите количество страниц: ");
        scanf("%d", books[i].page_n);
        printf("Введите имя издательства: ");
        gets(books[i].pubhouse.name);
        printf("Введите имя города: ");
        gets(books[i].pubhouse.city);
        printf("\nВведите год издания: ");
        scanf("%d", books[i].pubhouse.year);
    }
}

int main()
{
    setlocale(LC_ALL, "RUS");
    int i, j, n, k;
    book *books;
    input(books, &n);
    delete [] books;
    return 0;
}

Comments

[YermolaevG]

Спасибо за указание на ошибку с вводом/выводом, но проблема, как я посмотрел, в том, что "books = new book[*n];" создает только одну структуру, а не их массив. Поясните, чем это вызвано

[Миша Кринкин]

@YermolaevG а с чего вы это взяли? new [] выделяет память именно под массив объектов. Возможно у вас просто в n оказалась 1?

Answer 1

[Rsa97]

Ошибка:
scanf("%s", *books[i].name); Вы вместо указателя на имя передаёте значение первого символа, правильно:
scanf("%s", books[i].name);
Ошибка:

scanf("%d", books[i].price);
scanf("%d", books[i].page_n);
scanf("%d", books[i].pubhouse.year);

Вы вместо адреса переменной передаёте её значение, правильно:

scanf("%d", &(books[i].price));
scanf("%d", &(books[i].page_n));
scanf("%d", &(books[i].pubhouse.year));

Answer 2

[Илья Попов]

Во-первых, ваш код - это С а не С++. В С++ стоило бы использовать std::vector и что-то более безопасное вместо scanf и gets.

По вашему коду: обратите внимание на строчки со scanf. Вторым аргументом должен быть указатель на переменную, однако в строке
scanf("%d", books[i].price);вы передаёте туда саму цену, а не указатель на неё. Ещё раз перепроверьте все эти строки.

И компилируйте с макимальным уровнем предупреждений (например в GCC это -Wall -Wextra), современные компиляторы способны определять несоответствие типов в printf/scanf.

Comments

[YermolaevG]

К сожалению, я пока еще с vector обращаться не умею. Аргумент в scanf исправил, но проблема была не в этом.

[DancingOnWater]

Не путайте человека в С нет new

Answer 3

[Миша Кринкин]

Кроме упомянутых выше ошибок с использованием scanf (и сомнительности использования gets), есть еще одна ошибка связанная с передачей указателя в функцию input - он передается по значению, и изменяется версия внутри функции, но не значение переменной books из main, соответственно, delete [] books - делает совсем не то, что вы ожидаете.

Answer 4

[Сергей Протько]

Почему бы вам не использовать списки?

А ответ довольно простой - память под массив вы выделили, а структуру не инициализировали.

Comments

[YermolaevG]

Не подскажите, как это сделать в данном конкретном случае?

[Сергей Протько]

вам нужно инициализировать books[i] перед тем как с ним работать.