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

[alexey-malov]

1>C:\teaching\oop-labs\ler\OopLabs\lab4\Bodies\Bodies\UserInterface.cpp(13,15): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
1>C:\teaching\oop-labs\ler\OopLabs\lab4\Bodies\Bodies\UserInterface.cpp(31,15): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data

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

[alexey-malov]

https://github.com/Sancene/OopLabs/blob/f88292f91e572123df67eae9d17c7e52a0893d61/lab4/Bodies/Bodies/CBody.cpp#L17-L20

• [ ] Не надо умножать и делить на 1000. Вычисления должны выполняться с максимльной точностью и лишь только при выводе выполнять округление

[alexey-malov]

https://github.com/Sancene/OopLabs/blob/f88292f91e572123df67eae9d17c7e52a0893d61/lab4/Bodies/Bodies/CCone.cpp#L21-L24

• [ ] Не надо умножать и делить на 1000. Значение числа PI можно взять поточнее.

[alexey-malov]

https://github.com/Sancene/OopLabs/blob/f88292f91e572123df67eae9d17c7e52a0893d61/lab4/Bodies/Bodies/CBody.h#L17-L18

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

[alexey-malov]

https://github.com/Sancene/OopLabs/blob/f88292f91e572123df67eae9d17c7e52a0893d61/lab4/Bodies/Bodies/CCone.cpp#L11-L19

• [ ] Для доступа к полям достаточно указать их имя.

[alexey-malov]

https://github.com/Sancene/OopLabs/blob/f88292f91e572123df67eae9d17c7e52a0893d61/lab4/Bodies/Bodies/UserInterface.cpp#L21-L37

• [ ] А что если вектор будет пустой?

• [ ] А что если все фигуры будут иметь массу в воде > 999999999?

[alexey-malov]

• [ ] Класс UserInterface не обладает состоянием и содержит в себе разношерстный набор функций.

[alexey-malov]

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

[alexey-malov]

• [ ] Не реализован метод ToString()