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

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/38ea31de104b2b132f5ea8931e3a7c33926f5c8e/lab_01_04_01/lab_01_04_01.cpp#L66-L98

[x] Сложно. Выделите контекст


struct CompressionContext// CompressionContext
{
char currentChar;
uint8_t counter = 0;
};

bool CompressChar(CompressionContext& context, char c, ostream& output); bool FlushContext(CompressionContext& context, ostream& output;

// внутри функции компрессии CompressionContext& ctx; while(input.get(currentChar)) { CompressChar(ctx, currentChar, output); } FlushContext(ctx, output);



- [x] Аналогично сделайте с распаковкой

alexey-malov commented 4 years ago

[x] Print - неудачное название для операции вывода

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/38ea31de104b2b132f5ea8931e3a7c33926f5c8e/lab_01_04_01/lab_01_04_01.cpp#L134-L163

[x] Можно вывернуть наизнанку код:

using Transformer = function<bool(istream& input, ostream& output)>;
bool TransformFile(const string& inputFileName, const string& outputFileName, const Transformer& transformer);

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

...
// Для запаковки
TransformFile(args.inputFileName, args.outputFileName, Pack);
// Для запаковки
TransformFile(args.inputFileName, args.outputFileName, Unpack);

arina-kaa commented 4 years ago

Замечания исправлены.

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/0b2db1ad58b73acc3fabc1c59014881406d50d27/lab_01_04_01/lab_01_04_01.cpp#L35-L48

[x] Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.
[x] getCurrentMode - неудачное имя функции

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/0b2db1ad58b73acc3fabc1c59014881406d50d27/lab_01_04_01/lab_01_04_01.cpp#L69-L75

[x] Возможно, лучше принимать CompressionContext и сбрасывать сбрасывать счётчик внутри функции

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/0b2db1ad58b73acc3fabc1c59014881406d50d27/lab_01_04_01/lab_01_04_01.cpp#L121-L130

[x] DecompressionContext не меняется, поэтому можно принимать его по константной ссылке

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/0b2db1ad58b73acc3fabc1c59014881406d50d27/lab_01_04_01/lab_01_04_01.cpp#L135-L141

[x] DecompressionContext лучше считывать прямо в структуру. Однако советую обернуть ее в #pragma pack (см пример input-output в моем репозитории)

arina-kaa commented 4 years ago

Замечания исправлены.

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/952cb2f8a1fc86a1ef1ad3fe6d1ba6a26039e802/lab_01_04_01/lab_01_04_01.cpp#L72-L82 https://github.com/arina-kaa/OOP/blob/952cb2f8a1fc86a1ef1ad3fe6d1ba6a26039e802/lab_01_04_01/lab_01_04_01.cpp#L104-L112

[ ] Зачем передавать в FlushPackResult символ? Задача этой функции сбросить буфер и успокоиться. А то код запаковки потока выглядит так, что мы дважды последний символ потока прогоняем - сперва мы его записали, а потом в конце еще и Flush делаем. Неверно разделен функционал между FlushPackResult и CompressChar

alexey-malov commented 4 years ago

https://github.com/arina-kaa/OOP/blob/952cb2f8a1fc86a1ef1ad3fe6d1ba6a26039e802/lab_01_04_01/lab_01_04_01.cpp#L104-L112