Замечания по программе Bodies

[alexey-malov]

1>C:\teaching\oop-labs\ler\OopLabs\lab4\Bodies\Bodies\UserInterface.cpp(13,15): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
1>C:\teaching\oop-labs\ler\OopLabs\lab4\Bodies\Bodies\UserInterface.cpp(31,15): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data

• Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

[alexey-malov]

OopLabs/lab4/Bodies/Bodies/CBody.cpp

Lines 17 to 20 in f88292f

	double CBody::GetMass()
	{
	return round(GetDensity() * GetVolume() * 1000) / 1000;
	}

• Не надо умножать и делить на 1000. Вычисления должны выполняться с максимльной точностью и лишь только при выводе выполнять округление

[alexey-malov]

OopLabs/lab4/Bodies/Bodies/CCone.cpp

Lines 21 to 24 in f88292f

	double CCone::GetVolume() const
	{
	return round((3.14 * baseRadius * baseRadius * height) / 3 * 1000) / 1000;
	}

• Не надо умножать и делить на 1000. Значение числа PI можно взять поточнее.

[alexey-malov]

OopLabs/lab4/Bodies/Bodies/CBody.h

Lines 17 to 18 in f88292f

	double density;
	string type;

• Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.

[alexey-malov]

OopLabs/lab4/Bodies/Bodies/CCone.cpp

Lines 11 to 19 in f88292f

	double CCone::GetBaseRadius() const
	{
	return CCone::baseRadius;
	}
	double CCone::GetHeight() const
	{
	return CCone::height;
	}

• Для доступа к полям достаточно указать их имя.

[alexey-malov]

OopLabs/lab4/Bodies/Bodies/UserInterface.cpp

Lines 21 to 37 in f88292f

	shared_ptr<CBody> UserInterface::FindMinMassInWater(const vector<shared_ptr<CBody>>& bodies)
	{
	double minMass = 99999999.9;
	int pointer = 0;
	for (size_t i = 0; i < bodies.size(); ++i)
	{
	CBody* temp = bodies[i].get();
	double tempMass = (temp->GetDensity() - 1000) * 10 * temp->GetVolume();
	if (minMass > tempMass)
	{
	pointer = i;
	minMass = tempMass;
	}
	}
	shared_ptr<CBody> minMassBody = bodies[pointer];
	return minMassBody;
	}

• А что если вектор будет пустой?

• А что если все фигуры будут иметь массу в воде > 999999999?

[alexey-malov]

• Класс UserInterface не обладает состоянием и содержит в себе разношерстный набор функций.

[alexey-malov]

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

[alexey-malov]

• Не реализован метод ToString()