Массивы структур?

Question

[Mag1str]

Задача:

Известны данные о стоимости каждой из 15 моделей автомобилей и о их тип (легковой или грузовой). Найти среднюю стоимость легковых автомобилей.

Вот что я пока что написал, как далее расчитать не знаю

#include <iostream>
#include <string.h>

using namespace std;

struct Auto
{
	char marka[15];
	int tup, price;
};

void Auto_desh(struct Auto *m, int n);

int main()
{
	const int n=2;
	Auto M[n];

	string marka [15]{
	"BMW", "Audi"};
	
	string tup [15]{
	"Vaz", "Leh"};
	
	string price [15]{
	"1000", "500"};
	
	cout<<"E taki auto\t";
	for (int i = 0; i < n; i++) {
	cout<<marka[i]<<"\t"<<tup[i]<<"\t"<<price[i]<<endl;
	}
return 0;
}

Comments

[edward_freedom]

почему у тебя цена строка?

[Mag1str]

edward_freedom, ну а как считать среднюю цену?

[edward_freedom]

Mag1str, сложить все цену и поделить на количество всех цен. Но я про то, почему у тебя цена строка, а не число? Как ты математические действия со строкой будешь делать

[Mag1str]

edward_freedom, ошибка

[Mag1str]

int price [15]{
	1000, 500};

Answer 1

[edward_freedom]

#include "stdafx.h"
#include <iostream>
#include "list"
#include <algorithm>
#include <sstream>
#include <string>
using namespace std;

class Car {
public:
	enum Type
	{
		Passenger,
		Cargo
	};
	int Price;
	string Model;
	Type TypeAuto;
	

	Car(int price, string model, Type type) {
		this->Model = model;
		this->Price = price;
		this->TypeAuto = type;
	}
	string ToString() {

		std::stringstream result;
		result << "Model: " << this->Model <<
			", Price: " << this->Price <<
			", Type: " << this->TypeAuto;
		return result.str();
	}
};

class FilterCars {
public:
	list<Car> Cars;
	int AveragePrice;
	FilterCars(list<Car> cars, int averagePrice) {
		this->Cars = cars;
		this->AveragePrice = averagePrice;
	}
};

void ShowCars(list<Car> cars) {
	for (auto  car : cars) {
		std::cout << car.ToString() << endl;;
	}
}

FilterCars AveragePriceByType(list<Car> cars, Car::Type type) {
	list<Car> selectCars;
	std::copy_if(cars.begin(), cars.end(), std::back_inserter(selectCars), [type](Car car) {
		return car.TypeAuto == type;
	});
	auto prices = 0;
	for (auto car : selectCars) {
		prices += car.Price;
	}
	auto average = prices / selectCars.size();
	return FilterCars(selectCars, average);
}



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

	auto cars = {
		Car(40000, "Tesla", Car::Type::Passenger),
		Car(34000, "Chevrolet", Car::Type::Passenger),
		Car(324000, "Mercedes ", Car::Type::Passenger),
		Car(114000, "Volvo ", Car::Type::Cargo),
		Car(222000, "BMW ", Car::Type::Cargo),
	};

	

	auto cargo = AveragePriceByType(cars, Car::Type::Cargo);
	auto passenger = AveragePriceByType(cars, Car::Type::Passenger);
	
	cout << "Вывод грузовых автомобилей: " << endl;
	ShowCars(cargo.Cars);
	cout << "Средняя цена грузовых автомобилей: " << cargo.AveragePrice << endl << endl;

	cout << "Вывод легковых автомобилей: " << endl;
	ShowCars(passenger.Cars);
	cout << "Средняя цена легковых автомобилей: " << passenger.AveragePrice << endl << endl;

	system("pause");


    return 0;
}

Comments

[Антон Жилин]

Это не Питон, по умолчанию стоит использовать vector, а не list

[Антон Жилин]

При создании cars стоит указать тип vector<Car>, иначе это будет initializer_list, который несколько раз (!) будет копироваться в нормальный контейнер. Ну и параметры нужно в большинстве случаев передавать в функцию по ссылке (кроме всяких int-ов и случаев, когда важно создание копии)

[Mag1str]

Что за библиотека?
#include "stdafx.h"
У меня ошибка.

[Mag1str]

Я в Dev C++ пишу код.

[edward_freedom]

Mag1str, удали его, он тебе не надо, если не в студии пишешь

[Евгений Шатунов]

Антон Жилин,

При создании cars стоит указать тип vector, иначе это будет initializer_list, который несколько раз (!) будет копироваться в нормальный контейнер.

Просто на заметку.
Да, cars будет иметь тип std::initializer_list<Car>, но его копирование - это не проблема.
std::initializer_list является легковесной надстройкой над статическим массивом на стеке. Внутренняя реализация этого типа не стандартизирована, но практически всегда его размер не превышает двух регистров процессора.

[Антон Жилин]

Евгений Шатунов, это понятно. Под копированием имел в виду, что при передаче его в функции дважды выполнялась неявная конвертация его в std::list, с копированием всех данных. Это копирование и один-то раз делать жалко.