Замечания по RLE

IrinaZagaynova / oop

0 stars 0 forks source link

Замечания по RLE #2

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/ce096e47503b885bdb2521ec77927da0f15f4a0d/lab1/RLE/RLE/RLE.cpp#L187-L201

[ ] Не нужно использовать argv[] в main. Задача разбора массива argv - компетенция ParseArgs
[ ] Слетело форматирование

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/ce096e47503b885bdb2521ec77927da0f15f4a0d/lab1/RLE/RLE/RLE.cpp#L166-L170

[ ] Режим работы тоже должен разбираться функцией Parse и в виде enum-а находиться в этой структуре

alexey-malov commented 4 years ago

[ ] Устранить дублирование кода между функциями PackFile и UnpackFile. Это можно сделать за счёт выделения кода обработки потока (фактическая упаковка и распаковка) в отдельные функции и передачу function:
```
using StreamProcessor = function<bool(istream& input, ostream& output)>;
bool ProcessFiles(string const& inputFileName, string const& outputFileName, StreamProcessor processStreams)
{
// открыть файлы
..
if (!processStreams(input, output))
{
    return false;
}

// закрывать файлы
}
```

alexey-malov commented 4 years ago

[ ] Из кода RLE-компрессии выделите функцию, которая упаковывает

struct RLEChunk
{
    char ch;
    uint8_t counter;
};
bool FlushChunk(RLEChunk& chunk, ostream& output);
bool CompressChar(char ch, RLEChunk& chunk, ostream& output);

alexey-malov commented 4 years ago

[ ] Добавьте тесты для проверки граничных условий - компрессия последовательностей длиной 255-257 байт