массивы структур

[Tirendus]

Здравствуйте, по учебе поставили задание

Создать структуру ВИДЕОМАГАЗИН со следующими полями:

Название фильма
Режиссер
Жанр
Рейтинг популярности
Цена диска

Реализовать следующие возможности:
Поиск по названию
Поиск по жанру
Поиск по режиссеру
Самый популярный фильм в жанре
Показ всех записей и добавление



Сейчас работаю над той частью, где нужно добавлять записи (по сути, в самом начале), что-то не получается, но что, понять не могу, вот мои наработки.

#include <iostream>
struct videoShop {
	char fn[100];  // film name
	char pr[100];  // producer
	char gr[100];   // genre
	double popr;   // popularity rating
	double price;  // price
};
void add(videoShop* vs, int& size);

void main()
{
	int size_of_vs = 1, index = 0;
	videoShop* vs = new videoShop;
	add(vs, size_of_vs); //тут все проходит гладко
	add(vs, size_of_vs); // НА ЭТОЙ ОПЕРАЦИИ ПРОГРАММА ВЫЛЕТАЕТ
}

void add(videoShop* vs, int& size)
{
	size++;
	videoShop* temp = new videoShop[size];

	for (int i = 0; i < size-1; i++)
	{
		for (int j = 0; j < 100; j++)
		{
			temp[i].fn[j] = vs[i].fn[j];
			temp[i].pr[j] = vs[i].pr[j];
			temp[i].gr[j] = vs[i].gr[j];
		}

		temp[i].popr = vs[i].popr;
		temp[i].price = vs[i].price;
	}
	
	std::cout << "New film name?\n";
	std::cin.getline(temp[size-1].fn, 100);
	system("cls");
	std::cout << "Producer\'s name?\n";
	std::cin.getline(temp[size-1].pr, 100);
	system("cls");
	std::cout << "Genre?\n";
	std::cin.getline(temp[size-1].gr, 100);
	system("cls");
	std::cout << "Popularity rating?\n";
	std::cin >> temp[size-1].popr;
	system("cls");
	std::cout << "Price?\n";
	std::cin >> temp[size-1].gr;
	system("cls");
	std::cout << "New record added\n";

	delete [] vs;
	vs = temp;
	delete [] temp;
}

я закоментировал место с ошибкой крупным шрифтом.

Буду очень благодарен за любые советы.

[profi]

videoShop* vs = new videoShop;

тут ты не создаешь массив. Сделай так:

videoShop* vs = new videoShop[size_of_vs];

А тут советую

		for (int j = 0; j < 100; j++)
		{
			temp[i].fn[j] = vs[i].fn[j];
			temp[i].pr[j] = vs[i].pr[j];
			temp[i].gr[j] = vs[i].gr[j];
		}

копировать массив символов с помощью функции strcpy:

			strcpy(temp[i].fn, vs[i].fn);
			strcpy(temp[i].pr, vs[i].pr);
			strcpy(temp[i].gr, vs[i].gr);

[Tirendus]

В функции strcpy используется strlen, мои строки изначально пусты, потому использование strcpy не православно.

тут ты не создаешь массив. Сделай так:
Код:

videoShop* vs = new videoShop[size_of_vs];

сделал, не помогло, есть еще варианты?

[Пепел Феникса]

тогда можно memcpy

[Tirendus]

Спасибо за совет, а есть какие-то мысли почему не работает программа?

[profi]

Ты же при выходе из функции вызываешь операторы delete

	delete [] vs;
	delete [] temp;

которые уничтожают память, а потом снова пытаешь записать в уже не существующий массив данные. Попробуй убрать эти два оператора.

[mrChester]

Тебе надо для начала определиться в каком виде мудут храниться данные: в виде списка или массива, если массива, тогда в главной функции

videoShop* vs = new videoShop[count_vs];

где count_vs максимальное количество элементов массива, после его уже нельзя менять. Если count_vs не изменяется ДО тогда лучше будет вообще объявить так

videoShop vs[count_vs];

где count_vs константа.
PS. В случае динамического выделения памяти лучше хранить данные в виде списка

[MaTBeu]

add(vs, size_of_vs);

если вы передаете ссылку, то передавайте ссылку, а не копию.

add(vs, &size_of_vs);

У вас не изменяется размер структуры, хотя должен. Следовательно вы присваиваете данные в непредназначенные для этого места.

[Tirendus]

Большое спасибо за советы, со списками я еще не работал, задачу так поставили, хотя не говорили про то, что память нужно выделять динамически, попробую константный массив структур, а потом буду насиловать преподавателя с динамикой

[profi]

Вот, что быстренько набросал:

#include <iostream.h>
#include <windows.h>

struct videoShop {
	char fn[100];  // film name
	char pr[100];  // producer
	char gr[100];   // genre
	double popr;   // popularity rating
	double price;  // price
};

void add(videoShop* vs, int i);
void printShop(videoShop* vs, int i);

void main()
{
	int size_of_vs = 2, index = 0;
	videoShop* vs = new videoShop[size_of_vs];

	for (int i = 0; i < size_of_vs; i++)
		add(vs, i); 

	for (int j = 0; j < size_of_vs; j++)
		printShop(vs, j); 

	delete []vs;
}

void add(videoShop* vs, int i)
{
		cout << "New film name?\n";
		cin.get(vs[i].fn, 100).get();

		cout << "Producer\'s name?\n";
		cin.get(vs[i].pr, 100).get();

		cout << "Genre?\n";
		cin.get(vs[i].gr, 100).get();

		cout << "Popularity rating?\n";
		cin >> vs[i].popr;
		cin.get();

		cout << "Price?\n";
		cin >> vs[i].gr;
		cin.get();
		system("cls");

}

void printShop(videoShop* vs, int i){

		cout << "New film name: " << vs[i].fn << "\n";
		cout << "Producer\'s name: " << vs[i].pr << "\n";
		cout << "Genre: " << vs[i].gr << "\n";
		cout << "Popularity rating: " << vs[i].popr << "\n";
		cout << "Price: " << vs[i].gr << "\n\n";
}