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

ImbaCow / OOP

Repo for object oriented programming

0 stars 0 forks source link

Open alexey-malov opened 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

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

alexey-malov commented 5 years ago

void encode(const std::function<uint8_t(uint8_t, uint8_t)>& modeFunc, std::istream& input, std::ostream& output, uint8_t key)

[ ] Функция encode не должна принимать ключ. Она должна принимать функцию с одним аргументом (текущий обрабатываемый байт). Преобразование байта (в т.ч. с использованием ключа) должна выполнять эта функция. Можно превратить функцию из 2х аргументов в функцию с одним.

alexey-malov commented 5 years ago

посмотрите у меня в репозитории примеры про лямбды и про bind

Почитайте серию статей про функциональное мышление https://habr.com/ru/company/microsoft/blog/433402/