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

PetrovIliya / OOP

OOP

0 stars 0 forks source link

Замечания по программе Car #6

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

[ ] Тесты должны автоматически запускаться во всех конфигурациях и платформах по окончании сборки проекта

alexey-malov commented 4 years ago

[ ] Деструктор писать в классе Car явно не надо, если не планируется особый способ завершения жизненного цикла объекта

alexey-malov commented 4 years ago

1>C:\teaching\oop-ips\petrov\OOP\lw3\Car\Car\StringUtilities.cpp(8,2): warning C4834: discarding return value of function with 'nodiscard' attribute

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

alexey-malov commented 4 years ago

[ ] Неверно заданы настройки автозапуска. При сборке в Debug-конфигурации проекта Car будет проиведен запуск проекта tests из Release-конфигурации, независимо от того, собирались тест или не собирались.

alexey-malov commented 4 years ago

[ ] Не выполнено требование задания:

на задний ход можно переключиться только на нулевой скорости; даже при движении назад на нейтральной передаче не допускается включать заднюю передачу.

[ ] Воспроизвести проблему в тестах и исправить

alexey-malov commented 4 years ago

https://github.com/PetrovIliya/OOP/blob/d358d4df934088b708acb91ee2276077516b5594/lw3/Car/Car/Car.cpp#L1-L9

[ ] Некоторые #include-ы лишние

alexey-malov commented 4 years ago

https://github.com/PetrovIliya/OOP/blob/d358d4df934088b708acb91ee2276077516b5594/lw3/Car/Car/Car.cpp#L110-L121

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

alexey-malov commented 4 years ago

[ ] Если программе встретится EOF во входном потоке, программа зависнет