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

MaxMartST / OOP

Курс по объектно ориентированному программированию

1 stars 0 forks source link

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

1>The contents of <optional> are available only with C++17 or later.
2>The contents of <optional> are available only with C++17 or later.

[ ] Либо включите в проекте поддержку C++17, либо не подключайте заголовочный файл <optional>

alexey-malov commented 4 years ago

[ ] pch.h следует подключать только из .cpp файлов. Его не надо подключать из заголовочных файлов

alexey-malov commented 4 years ago

[ ] GetListChannels должен быть константным
[ ] Передавать примитивные типы (такие как int) по константной ссылке смысла нет

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Отсутствие имени у канала не должно являться исключительной ситуацией. Можно вернуть либо optional, либо пустую строку в этом случае
[ ] Если у вас есть итератор, не равный end, то используйте его для получения значения, а не метод at(), который будет повторно выполнять поиск

alexey-malov commented 4 years ago

[ ] Невозможность найти канал по его имени вряд ли должно являться исключительной ситуацией. Можно вернуть либо optional, либо 0.
[ ] Если у вас есть итератор, не равный end, то используйте его для получения значения, а не метод at(), который будет повторно выполнять поиск

alexey-malov commented 4 years ago

[ ] Если у вас есть итератор, не равный end, то используйте его для получения значения, а не метод at(), который будет повторно выполнять поиск

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Исключения следует ловить по константной ссылке, а не по значению, чтобы избежать срезки (slicing). В редких случаях можно ловить по неконстантной ссылке, если требуется модифицировать пойманный объект исключений перед перевыбросом его.