Замечания по программе Crypt

[alexey-malov]

• [ ] Программа шифрует саму себя, и получается файл меньшей длины. Убедитесь, что файл открываете в бинарном режиме

  Crypt.exe crypt Crypt.exe Crypt.exe.crypted 42

[alexey-malov]

• [ ] Прикрутить автоформатирование при помощи clang-format

[alexey-malov]

https://github.com/venera13/oop/blob/0744c8f7c2a1993504ad84588aed509374ab90ec/lab1/Crypt/Crypt.cpp#L134-L137

• [ ] InitCrypt - неудачное имя.
• [ ] ; после блока не ставят

[alexey-malov]

https://github.com/venera13/oop/blob/0744c8f7c2a1993504ad84588aed509374ab90ec/lab1/Crypt/Crypt.cpp#L79

• [ ] Тут args не нужно заворачивать в Optional

[alexey-malov]

https://github.com/venera13/oop/blob/0744c8f7c2a1993504ad84588aed509374ab90ec/lab1/Crypt/Crypt.cpp#L9-L15

• [ ] В args строками должны быть только имена файлов. Ключ должен быть целым числом, а режим обработки - enum-ом. Задача функции ParseArgs выполнить необходимые преобразования

[alexey-malov]

https://github.com/venera13/oop/blob/0744c8f7c2a1993504ad84588aed509374ab90ec/lab1/Crypt/Crypt.cpp#L110-L120

• [ ] При обработке каждого байта программе незачем проверять режим работы. Можно воспользоваться std::function положив туда функцию кодирования либо декодирования.

  
  void TransformStream(istream& input, ostream& output, std::function<char(char)> transform);

if (args.mode == Mode.Crypt) { TransformStream(input, output, [key = args.key](char ch) { return Encrypt(char, key); }); } else { TransformStream(input, output, [key = args.key](char ch) { return Decrypt(char, key); }); }

либо даже:
```c++
#include <iterator>
std::transform(
    std::istream_iterator<char>(input), (istream_iterator<char>()),
    std::ostream_iterator<char>(output),
     [key = args.key](char ch) {
      return Decrypt(char, key);
  });