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

[alexey-malov]

• [ ] Подключите заголовочный файл optional в Dictionary.h

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/f8413748cf55c7c43a1f21d37b9d96e2a0605d30/lab2_dict/Dictionary.h#L6-L26

• [ ] Класс не предназначен для наследования (у него нет виртуальных методов и виртуального деструктора), поэтому защищенный метод выглядит странно.

• [ ] Возможно, optional здесь избыточен для возвращат переводов. Пустое множество вернуть не будет достаточно, чтобы подчеркнуть отсутствие переводов?

• [ ] Методы Translate* следует объявить констатными

• [ ] Деструктор можно не объявлять. Его компилятор сам сгенерирует. Почитайте про Правило трех

• [ ] Конструктор по умолчанию тоже можно не объявлять. В отсутствие других конструкторов компилятор сам может его сгенерировать

• [ ] По соглашениям о стиле кодирования поля класса должны иметь префикс m_

[alexey-malov]

• [ ] Не надо везде писать this->. С++ - не PHP и не JavaScript

[alexey-malov]

• [ ] Если все же решили использовать наследование, то сделайте это корректно:
• Сделайте деструктор публичным и виртуальным
• Сделайте нужные методы виртуальными. При вызове AddTranslation по ссылке на базовый класс поведение класса-наследника будет некорректным (не выставится флаг isSavedToFile)

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/f8413748cf55c7c43a1f21d37b9d96e2a0605d30/lab2_dict/FileDictionary.h#L16

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

[alexey-malov]

• [ ] Что должно происходить при загрузке словаря из файла, если он был непустой?

[alexey-malov]

• [ ] Вообще класс FileDictionary, унаследованный публично от Dictionary выглядит странно:
• базовый класс не оставляет точек для расширения функционала (нет виртуальных методов)
• загрузка и сохранение выглядят как новая ответственность, а сейчас она является частью класса. Например, что будет, если если вы заходите поддержать несколько разных форматов хранения словаря. Сделаете несколько разных наследников? Но тогда если надо будет загружать словарь в одном формате, а сохранять в другом, то количество классов-наследников будет пропорционально квадрату количества форматов. Наследование делает связь между классами жёсткой, задаваемой в момент написания кода. Во время выполнения программы гибко изменить вы уже не сможете.