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

ASTeterin / OOP

0 stars 0 forks source link

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

Open alexey-malov opened 5 years ago

alexey-malov commented 5 years ago

[ ] Сделайте функции, принимающие имена файлов и открывающие их. Внутри main должен быть только высокоуровневый код

alexey-malov commented 5 years ago

void CompressFile(istream &input, ostream &output)
{
    char ch = '\0';
    char next_ch = NULL;

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

alexey-malov commented 5 years ago

void CompressFile(istream &input, ostream &output)
{
    char ch = '\0';
    char next_ch = NULL;
    uint8_t count = 0;
    while (input.get(ch))
    {
        next_ch = input.peek();

[ ] Есть риск потерять значение EOF

alexey-malov commented 5 years ago

[ ] Символ с кодом 255 ошибочно принимается за EOF. Надо использовать int для хранения результата вызова peek
[ ] Добавить тест для этой ситуации

alexey-malov commented 5 years ago

[ ] Пусть функция ParseCommandLine еще и принимает out-параметры (режим работы, входной/выходной файлы)