Замечания по паттерну "Декоратор" - потоки

[alexey-malov]

    if (argc < 3)
    {
        std::cerr << "Invalid arguments count..." << std::endl;
        std::cerr << "Usage: Streams.exe [options] <input file> <output file>" << std::endl;
        return 1;
    }

• [x] Следует вывести более полную инструкцию

[alexey-malov]

• [x] Код функции Decorate труден для понимания

[alexey-malov]

CompressOutputStreamDecorator::~CompressOutputStreamDecorator()
{
    Flush();
}

• [x] Обернуть в try/catch тело. Деструктор не должен бросать исключений

[alexey-malov]

• [x] Структура Chunk дублируется в компрессоре и декомпрессоре. Устранить дублирование кода

[alexey-malov]

        for (uint8_t i = 0; i < chunk->count; ++i)
        {
            m_decompressedBuffer.push_back(chunk->byte);
            ++unpacked;
        }

• [x] В deque есть метод insert, позволяющий вставить несколько одинаковых элементов

[alexey-malov]

• [x] В тестах можно не создавать реальные файловые потоки можно не создавать. Использовать Memory

[alexey-malov]

• [ ] Применение декораторов можно оформить в виде функций

using InputStreamDecorator = function<unique_ptr<IInputStream>(unique_ptr<IInputStream>&& s);
using OutputStreamDecorator = function<unique_ptr<IOutputStream>(unique_ptr<IOutputStream>&& s);
vector<InputStreamDecorator> inputDecorators;
vector<OutputStreamDecorator> outputDecorators;

inputDecorators.emplace_back([](auto&& s){
    return make_unique<Decompress>(move(s));
});
inputDecorator.emplace_back([key](auto&& s){
    return make_unique<Decrypt>(move(s));
});

for (auto&& d : inputDecorators)
{
    s = d(move(s));
}

либо

InputStreamDecorator  applyDecompressor;
for (auto&& option : options)
{
    if (option == "--decompress") 
        applyDecompressor = [](auto&& s){
            return make_unique<Decompress>(move(s));
        };
    ...
}

if (applyDecompressor)
{
    s = applyDecompressor(move(s));
}
if (applyDecryptor)
{
    s = applyDecryptor(move(s));
}