Review #1

[knyazsergei]

• прочитать по подход RAII

class CGdiPlus
{
    CGdiPlus()
    {   
        ULONG_PTR token;
        GdiplusStartupInput input;
        GdiplusStartup(&token, &input, 0);
    }
    ~CGdiPlus()
    {
        GdiplusShutdown(token);
    }
    // ...
}

{
    CGdiPlus initGdi;   
    throw 1;
}

• использовать данный подход для работы с памятью в соответствующих местах.
• можно использовать только wide char (std::wstring)
• стандартные хедеры в общий заголовочник stdafx
• мы пишем на с++ где есть классы! и исключения! C+11!!
• создать консольный проект - статическую либу. liblazyloader
• создать консольный проект - консольный exe для тестов liblazyloader (boost).
• создать класс и тесты для них
• класс определять в namespace

CImage 
  Bitmap& GetBitmap() const;

CImageCollection (size_t imageSize = 0)
   size_t Size() const;
   CImage const& GetImage(size_t i) const;
   void EmplaceImage(CImage& image); // movable
   void Clear();

CImageLoader (std::wstring const& path)
    size_t GetImagePageSize() const; // return 30;
    void Load(CImageCollection& images, size_t startIndex);

CException : public std::runtime_exception

• при возникновении ошибки необходимо бросать исключение с соответствующим сообщением об ошибке например: if !GdiPlus::Status::Ok else throw CException("can't load image");

liblazyloader
    liblazyloadertest
        data
            test1
                image1.png
                image2.png
            test2
                image1.png
                image1.txt // throw CException("can't load image");
            ...
        ... //      

[evkalinin]

• код должен компилироваться!!!!!
• следить за code convensions
• написать тесты. для каждого нового класса новый тест сьюит
• примерно что имелось ввиду:

CGdiPlus init; // setup

CImageLoader loader(path);
loader.SetImagePageSize(10);

CImageCollection c;
loader.Load(collection, 3);

// CImageLoader.Load()
{
    for (all images in path)
    {
        try {
            auto bitmap = Load(path);
            //lload::CImage image(bitmap);
            c.EmplaceImage(lload::CImage(bitmap));
        }
        catch (CLLException const& e)
        {
            // ...
        }
    }
}

ASSERT(c.GetSize() == 10);

[evkalinin]

• тесты разнести отдельными файлам
• перенести конструкторы копирования и перемещения в приватную область, сделать метод Clone возвращающий копию c новыми размерами std::unique_ptr Clone() const; std::unique_ptr Clone(unsigned int width, unsigned height) const;
• добавить интерфейс изображения IImage, продумать у него базвый функционал
• в коллекции CImageCollection достаточно дефолтного конструтора, а содержит она коллекцию (продумать какую) unique_ptr
• изображение добавляется в коллекцию чз ее методы в начало и в конец
• контроллер знает индексы текущего скрола, и подгружает изображения ЧЕРЕЗ ЛОАДАЕР и смещает текущее положение
• покрыть это тестами.

[evkalinin]

Проект должен представлять из себя:

• графическое приложение отображающее список изображений, пример:
• скрол списка должен быть плавным на большом количестве изображений (и больших разрешений)
• использовать MVC, и нормальный стиль программирования
• тесты

после выполенения вышеуказанного

• выбор папки с изображениями
• открытие полноразмерной картинки при нажатии на миниатюру

согласовано с Сергеем.