Follow-up from "Дорофеев Николай P32101"
The following discussions from !190 (closed) should be addressed:
-
@uvuv643 started a discussion:
Желательно из коммита убрать файлы .idea, .vscode, .o. Собираемые файлы и файлы проекта не рекомендуется добавлять в git. Для этого рекомендуется соответствующим образом настроить .gitignore (или разрешить IDE это сделать самостоятельно)
-
@uvuv643 started a discussion:
Желательно вынести подсчёт padding в отдельную функцию и избавиться от magic number. (За что отвечает 3 и 4?)
-
@uvuv643 started a discussion:
Лучше вынести в #define DEFAULT_BF_TYPE 0x4d42 #define BIT_COUNT 24 и т.д.
-
@uvuv643 started a discussion:
Всё круто, но ещё круче, если выводить все ошибки в stderr: fprintf(stderr, "error");
-
@uvuv643 started a discussion:
Все ошибки лучше передавать как enum из функций возвращаемым значением. Если всё ок - специальное значение enum - OK. Обработку этих ошибок осуществлять в main. Если захотите добавить локализацию, к примеру, вызов printf из множества файлов может стать проблемой.
Все исправил. Спасибо за ревью!