sertarus/programmingtask1

programmingtask1's Issues

Incorrect .gitgnore

На данный момент кто-то, кто скачает Ваш проект, не сможет запустить тесты (а в дальнейшем, и воспользоваться функциональностью).
Gitgnore - это хорошо, но файлы с расширением .iml нужны, чтобы управлять зависимостями от библиотек, если Вы решили использовать внутренний формат Идеи.

Test coverage

Постарайтесь покрыть как можно больше возможных путей исполнения функций; пока, если верить Run with coverage, рассмотрены не все возможные ситуации (соответствующие строки в редакторе будут подсвечиваться красным).

Javadoc comments

Пожалуйста, оформляйте комментарии к разработанному коду в соответствии с нотацией Javadoc (в дальнейшем, на её основе легко может быть сгенерирована документация к проекту).

Logic issues &code accuracy

• не совсем понятно, зачем Вы храните highestDegree для полинома, если она у Вас напрямую соответствует длине списка, и используется попеременно то она, то вычисление длины (не проще ли возвращать в геттере результат соответствующих вычислений?);

• в реализации plus для цикла дополнения if под циклом - избыточен;

• дублирование кода в divide/ remainderOfTheDivision - плохой стиль; например, можно создать общую private функцию и public-обёртки к ней;

• с учётом того, что реализуется вычисление остатка от деления, генерация исключения в случае, если степень делимого меньше степени делителя не вполне ясна;

• для сокращения реализации toString, возможно, имеет смысл воспользоваться Map-ом;

• наименее важно, но всё же - обращение к функциям Math с использованием имени пакета - затрудняет читаемость Вашего кода.

JFYI

• для инициализации списка, возможно, более удобным решением будет такой синтаксис;

• для тестирования выкидывания исключений используется, например, метод fail;

• будьте, пожалуйста, в дальнейшем аккуратны с null-значениями (сейчас Ваш код можно сломать, положив такое в конструктор).