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

IrinaZagaynova / oop

0 stars 0 forks source link

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

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/7b56d8a92e3f92746f9268fa012ac218ac09480d/lab3/TVSet/TVSet/TVSet.h#L3-L25

[ ] объекты следует передавать по константной ссылке, если функция не модифицирует их состояние. Примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению
[ ] typedef лучше объявить внутри класса, если к нему не нужно обращаться снаружи (можно даже в private-области)

alexey-malov commented 4 years ago

https://github.com/IrinaZagaynova/oop/blob/7b56d8a92e3f92746f9268fa012ac218ac09480d/lab3/TVSet/TVSet/TVSet.cpp#L46-L48

[ ] Используйте std::swap

alexey-malov commented 4 years ago

[ ] В тестах телевизора (установка имени канала) нет проверки ограничений на номер канала