Попробовал написать программу с динамическим массивом структур данных. Программа компилируется и работает. Хотелось бы что бы более опытные товарищи оценили насколько оптимально

[Тимох]

Изучаю сейчас структуры данных. Попробовал написать программу с динамическим массивом структур данных. Программа компилируется и работает. Хотелось бы что бы более опытные товарищи оценили насколько оптимально написана программа и нет ли утечки памяти(высвобождение памяти от динамического массива). Вот исходный код программы:

/*
Задача
Создай коллекцию автомобилей через структуру данных
*/

#include <iostream>
#include <conio.h>

using namespace std;

//создаю структуру(данные об автомобиле)
struct mashina
{
	char *marka;
	char *model;
	int god_vypuska;
};

void zapolnenie(mashina &a);
void vyvod(const mashina *a);

int main()
{
	setlocale(LC_ALL,"RUS");

	cout<<"\t\t КОЛЛЕКЦИЯ АВТОМОБИЛЕЙ \n\n";
	int a=0;
	cout<<"Введи количество автомобилей: ";
	cin>>a;
	cout<<endl;
	//создаю динамический массив структур
	mashina *kollekcijа = new mashina[a];
	//заполняю динамический массив структур
	for(int i=0; i<a; i++)
	{
		zapolnenie(kollekcijа[i]);
		cout<<endl;
	}
	system("cls");
	cout<<"\t\t КОЛЛЕКЦИЯ АВТОМОБИЛЕЙ \n\n";
	//вывожу данные из массива структур
	for(int i=0; i<a; i++)
	{
		cout<<"Автомобиль "<<(i+1)<<endl;
		vyvod(&kollekcijа[i]);
		cout<<endl;
	}
	for(int i=0; i<a; i++)
	{
		delete [] kollekcijа[i].marka;
		delete [] kollekcijа[i].model;
	}
	//освобождаю динамический массив структур
	delete [] kollekcijа;
	cout<<endl;
	cout<<"Нажми Enter для завершения программы";
	_getch();
	return 0;
}

//функция заполнения данными структуры
void zapolnenie(mashina &a)
{
	a.marka = new char[10];
	a.model = new char[10];
	cout<<"Введи марку автомобиля: ";
	cin>>a.marka;
	cout<<"Введи модель автомобиля: ";
	cin>>a.model;
	cout<<"Введи год выпуска автомобиля: ";
	cin>>a.god_vypuska;
}
void vyvod(const mashina *a)
{
	cout<<a->marka<<a->model<<endl;
	cout<<"Год выпуска: "<<a->god_vypuska<<endl;
}

[Stilet]

насколько оптимально написана программа

Я бы описал класс с приватным полем типа vector для коллекции.
Ну а если говорить о твоем коде - циклы ввода вывода в отдельную процедуру.