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

alexey-malov commented 4 years ago

[ ] Программа при попытке скомпрессировать произвольный двоичный файл и распаковать его выдаёт отличный результат

alexey-malov commented 4 years ago

https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L8-L13

[ ] Режим компрессии парсером командной строки должен возвращаться в виде enum-а, а не в виде строки

alexey-malov commented 4 years ago

https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L108-L124

[ ] До строчки с проверкой input и flush-ем выходного файла управление не дойдёт

alexey-malov commented 4 years ago

https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L52 https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L30

[ ] Следует зависеть от istream& и ostream&, а не от ifstream/ofstream

alexey-malov commented 4 years ago

https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L68-L77

[ ] Выделите функцию, выполняющую компресию очередного байта. Это упростит цикл компрессии

struct RLEChunk
{
    char currentChar;
    uint8_t counter;
};

// Компрессирует очередной байт, обновляя chunk. При необходимости сбрасывает в Output
bool CompressChar(RLEChunk& chunk, char ch, ostream& output);

// Сбрасывает блок на диск, обнуляет счётчик
bool FlushChunk(RLEChunk& chunk, ostream& output);

alexey-malov commented 4 years ago

https://github.com/Dekart-hub/oop/blob/87cc30b6dd43bf1fddd3272715393936eeeeca9f/RLE/RLE/RLE.cpp#L147-L148

[ ] Нет проверки ошибок

alexey-malov commented 4 years ago

[ ] В тестах сравнивать файлы надо в бинарном режиме:
```
fc /B file1 file2
```

Dekart-hub / oop

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