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

[alexey-malov]

Избавляйтесь от длинных цепочек вызовов

Не компилируется сравнение line segment-ов

[alexey-malov]

Используй заголовочные файлы с forward declaration классов и интерфейсов

[alexey-malov]

http://stackoverflow.com/questions/2832714/header-files-inclusion-forward-declaration

[alexey-malov]

Рисование лучше запускать внутри методов Visit, либо обеспечить механизм "накопления" информации о посещении. Это сделает возможным введение групповых фигур, которые внутри себя содержат вложенные фигуры

[alexey-malov]

подсказка по 5 лабе:

void ExpectStringContent(const CMyString& str, const char * arr,])
{
    BOOST_REQUIRE_EQUAL(str.GetLength(), len - 1);
    BOOST_CHECK_EQUAL(memcmp(str.GetData(), arr, len), 0);
}

template <size_t len>
void ExpectStringContent(const CMyString& str, const char (&arr)[len])
{
    ExpectStringContent(str, arr, len);
}
...

ExpectStringContent(myStr, "Hello\0world");

[alexey-malov]

В CShapeManager::CreateShape слишком много кода. И он будет увеличиваться при введении доп. фигур Разбить на подфункции, сделать возможность легкого добавления новых типов фигур.

typedef function<unique_ptr<IShape>(const vector<string> & params)> ShapeCreator;
map<string, ShapeCreator> m_shapeCreators;

ShapeCreator MakeCreator(<Указатель на метод класса CShapeManager> p)
{
    return bind(p, this, std::placeholders::_1);
}

m_shapeCreator["circle"] = MakeCreator(&CShapeManager::CreateCircle);
m_shapeCreator["rectangle"] = bind(&CShapeManager::CreateCircle, this, _1);

[alexey-malov]

range-based for компактнее чем for_each

[alexey-malov]

string ShapeToString(const shared_ptr<IShape> & shape)
{
    return shape->ToString();
}

// Вывод фигур в cout через \n
boost::transform(shapes, ostream_iterator<cout>("\n"), ShapeToString);