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

[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;
	}

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

[alexey-malov]

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

[alexey-malov]

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

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

[alexey-malov]

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

[alexey-malov]

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

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

[alexey-malov]

• В тестах можно не создавать реальные файловые потоки можно не создавать. Использовать 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));
}