Замечания по объёмным телам

[alexey-malov]

• [ ] Файлы предварительно скомпилированных заголовков следует подключать только в .cpp файлах (самым первым #include-ом)

https://docs.microsoft.com/en-us/cpp/build/creating-precompiled-header-files?view=vs-2019

[alexey-malov]

class CSolidBody : public CBody
{
public:
    CSolidBody(const std::string& type, double density);

    double GetDensity() const override;
    virtual double GetVolume() const = 0;

• [ ] В CSolidBody не надо повторно объявлять чисто виртуальный метод GetVolume, потому что он уже был объявлен чисто виртуальным в базовом классе

• [ ] То же самое касается метода AppendProperties

[alexey-malov]

double CCompound::GetDensity() const
{
    return GetMass() / GetDensity();
}

• [ ] Тут бесконечная рекурсия

[alexey-malov]

double CCompound::GetVolume() const
{
    double volume = 0;
    for (shared_ptr<CBody> body : m_children)
    {
        volume += body->GetMass();
    }
    return volume;
}

• [ ] Перебор указателей огранизовать по константной ссылке

• [ ] Почему к объёму прибавляется масса?

[alexey-malov]

bool CCompound::AddChildBody(shared_ptr<CBody> child)
{
    /*if (make_shared<CBody>(this) == child)
    {
        return false;
    }*/

    try
    {
        m_children.push_back(child);
    }
    catch (const exception& e)
    {
        std::cout << e.what() << '\n';
        return false;
    }
    return true;
}

• [ ] Зачем мы здесь перехватываем исключение?

• [ ] Нет защиты от добавления объекта внутрь себя самого (прямо или косвенно).

[alexey-malov]

• [ ] Покрыть тестами

[alexey-malov]

• [ ] Добавить вывод массы в команде ShowMax и вывод веса в воде в команде ShowLight