Замечания по Shapes

[alexey-malov]

• [x] Сделать пример, иллюстрирующий проблему из-за отсутствия виртуального деструктора.

[alexey-malov]

    //double GetArea() const override = 0;
    //double GetPerimeter() const override = 0;

• [x] переопределять методы, чтобы объявить их чисто виртуальными, избыточно.

[alexey-malov]

• [x] Разработать приложение, считывающее фигуры и обрабатывающее их согласно заданию. Предоставить файлик с входными данными, чтобы можно было ввод перенаправить при защите

[alexey-malov]

• [x] Окружность должна конструироваться путем задания радиуса. Таково пожелание заказчика

[alexey-malov]

#include "IShape.h"
#include "Circle.h"

void DoSomething(const IShape & shape)
{
    (void)&shape;
}

int main()
{
    CCircle circle({100, 100}, 100, "red", "green");
    DoSomething(circle);
    return 0;
}

• [x] Проблема с ромбовидным наследованием

  1>c:\teaching\2016\ips\lihachev\oop2016\lab4\shapes\shapes\main.cpp(13): error C2594: 'argument': ambiguous conversions from 'CCircle' to 'const IShape &'

[alexey-malov]

1>c1xx : fatal error C1083: Cannot open source file: 'ShapesFactory.cpp': No such file or directory

• [x] программа должна компилироваться без ошибок

[alexey-malov]

            CCircle c1(CPoint(0, 0), 0, "", "");
            auto sp = make_shared<CCircle>(CPoint(0, 0), 0, "", "");
            shared_ptr<CCircle> sp2(new CCircle(CPoint(0, 0), 0, "", ""));

            auto sp1 = make_shared<CCircle>(c1); // копия c1
            shared_ptr<CCircle> sp2(new CCircle(c1)); // копия c1

вот так в куче можно создать объект

[alexey-malov]

        switch (typeIt->second) {
        case ShapeTypeEnum::circle:
            try{
                auto sp = make_shared<CCircle>(CPoint(0, 0), 0, "", "");
                strm >> *sp;
                shapesVector.push_back(sp);
            }
            catch (std::exception e)
            {
                cout << e.what() << endl;
            }
            break;
        case ShapeTypeEnum::rectangle:
            try {
                auto sp = make_shared<CRectangle>(CPoint(0, 0), CPoint(0, 0), "", "");
                strm >> *sp;
                shapesVector.push_back(sp);
            }
            catch (std::exception e)
            {
                cout << e.what() << endl;
            }
            break;
        case ShapeTypeEnum::triangle:
            try {
                auto sp = make_shared<CTriangle>(CPoint(0, 0),
                    CPoint(0, 0), CPoint(0, 0), "", "");
                strm >> *sp;
                shapesVector.push_back(sp);
            }
            catch (std::exception e)
            {
                cout << e.what() << endl;
            }
            break;
        }

• [ ] Устранить дублирование кода

[alexey-malov]

    auto less_area = [&](shared_ptr<IShape> const& sh1, shared_ptr<IShape> const& sh2) {
        return sh1->GetArea() < sh2->GetArea();
    };

• [ ] Для именования переменных использовать lowerCamelCase, для функций - UpperCamelCase

• [ ] В лямбде не нужно захватывать параметры по ссылке, если она не использует переменные из внешней области видимости.

• [ ] Параметры лямбды можно принимать с использованием auto:

auto lessArea = [](auto & sh1, auto & sh2){
    return sh1->GetArea() < sh2->GetArea();
};