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

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/main.cpp#L6-L8

• [x] FileLinker - неудачное имя

• [x] Parse

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/ParseArgument.h#L9-L14

• [x] Команду лучше представить в виде enum-а, чтобы на функцию, принимающую решение о кодировании или декодировании не переносить знания о командной строке

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/LinkerBinaryFile.cpp#L9

• [x] простые типы не имеет смысла передавать по ссылке

• [x] Размер передавать незачем. Как eof встретится, так и закончите работу

• [x] принимать лучше потоки istream& и ostream&, а не привязываться жёстко к файловым потокам

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/LinkerBinaryFile.cpp#L15-L44

• [x] Код труден для понимания. Сделайте функцию вроде:

struct RLEChunk
{
    char currentChar;
    uint8_t counter = 0;
};

// обновляет чанк новым символом
// возвращает true в случае успеха
// может вызвать FluchChunk
bool PackNextChar(RLEChunk& chunk, char ch, ostream& output);

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

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/LinkerBinaryFile.cpp#L91

• [x] Исключения следует ловить по константной ссылке, а не по значению, чтобы избежать срезки (slicing). В редких случаях можно ловить по неконстантной ссылке, если требуется модифицировать пойманный объект исключений перед перевыбросом его.

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/LinkerBinaryFile.cpp#L64-L77

• [x] Введите функции bool ReadChunk(RLEChunk&, istream&) и bool UnpackChunk(const RLEChunk&) и упростите с их помощью цикл распаковки

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.4%20rle/lab1.4%20rle/LinkerBinaryFile.cpp#L110-L130

• [x] Сделайте как-то так:

  
  bool TransformFile(const string& inputFileName, const string& outputFileName, const std::function<bool(istream&, ostream&)> transformer);

bool Pack(istream& input, ostream& output); bool Unpack(istream& input, ostream& output);

bool Foo(const Args& args) { switch (args->command) { case Command:Pack: return TransformFile(args.inputFileName, args.outputFileName, Pack); case Command::Unpack: return TransformFile(args.inputFileName, args.outputFileName, Unpack); } }

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/67e9588eec3182d003b936b8d4af62e56358ec76/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L74

• [ ] Тернарный оператор не нужен

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/67e9588eec3182d003b936b8d4af62e56358ec76/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L59-L72

• [ ] Если при чтении произойдёт ошибка, то флаг eof никогда не выставится и программа зависнет

[alexey-malov]

• [ ] Файл и 255 и 254 одинаковых символов в сжатом виде выглядят одинаково. Программа неверно компрессирует

• [ ] Добавьте тесты на проверку сжатия последовтельностей длиной 254, 255, 256 и 257 одинаковых байт

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/67e9588eec3182d003b936b8d4af62e56358ec76/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L91-L100

• [ ] Функция ReadChunk не выполняет чтение чанка

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/987c97d2246105c069cbdb83fd985845e65e6ccd/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L68-L91

• [ ] Выпилить

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/987c97d2246105c069cbdb83fd985845e65e6ccd/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L93-L105

• [ ] Не нужно внутри цикла чтения залезать во внутренности chunk-а. Это должны делать только FlushChunk и PackNextChar

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/987c97d2246105c069cbdb83fd985845e65e6ccd/Lw%201/lab1.4%20rle/lab1.4%20rle/ArchiverBinaryFile.cpp#L44-L61

• [ ] PackNextChunk должен возвращать true в случае успеха, а false в случае ошибки записи (косвенно через FlushChunk). Сейчас же он у вас может вернуть false как в случае ошибки записи, так и в случае, когда мы просто увеличили счётчик. То есть нет возможности отличить ошибку от ее отсутствия