Замечания по реализации паттерна Фабрика

vstdio / ood

Репозиторий для лабораторных работ по дисциплине "Объектно-ориентированное проектирование"

MIT License

0 stars 0 forks source link

Замечания по реализации паттерна Фабрика #5

Open alexey-malov opened 6 years ago

alexey-malov commented 6 years ago

[x] Используйте умные указатели на интерфейсы direct2d: CComPtr/CComQIPtr (из ATL) либо _com_ptr

alexey-malov commented 6 years ago

    auto r = GetClientRect(hWnd, &rect);
    assert(r == TRUE);

[x] Не сравнивайте с TRUE. Проверяйте на неравенство FALSE

alexey-malov commented 6 years ago

[x] Работа с окном сделана достаточно "коряво", сделайте оконное приложение с использованием WTL или MFC

alexey-malov commented 6 years ago

[ ] CoInitialize/CoUninitialize следует вызывать не внутри холста, а внутри самого приложения. Рекомендую сделать RAII-обертку над этими функциями и поместить внутрь Application

alexey-malov commented 6 years ago

    ::CoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
    auto hr = m_pFactory.CoCreateInstance(__uuidof(ID2D1Factory));
    assert(hr != S_FALSE); // Не возвращает S_OK

    hr = m_pRenderTarget.CoCreateInstance(__uuidof(ID2D1HwndRenderTarget));
    assert(hr != S_FALSE);

    hr = m_pBrush.CoCreateInstance(__uuidof(ID2D1SolidColorBrush));
    assert(hr != S_FALSE);

    hr = D2D1CreateFactory(D2D1_FACTORY_TYPE_SINGLE_THREADED, &m_pFactory);
    assert(hr == S_OK);

[ ] В CoCreateInstance надо передавать идентификатор класса, а не идентификатор интерфейса
[ ] Объекты Direct2D создаются через фабрику, а не напрямую. А фабрика создаётся через функцию D2D1CreateFactory

alexey-malov commented 6 years ago

D2DCanvas::~D2DCanvas()
{
    ::CoUninitialize();
}

[ ] CoUnitialize надо вызывать только после того, как вы освободили все ссылки на интерфейсы, полученные в данном потоке. Тело деструктора вызывается ДО того, как деструкторы полей класса D2DCanvas освободят свои интерфейсы

alexey-malov commented 6 years ago

[ ] В тесты фабрики добавить проверку обработки ошибочных ситуаций (когда передаётся недостаточное количество параметров или типы параметров не соответствуют ожиданиям