dvede / table Goto Github PK

1. Что произойдёт в методе inRange, если таблица будет пустой?

2. Конструкции вида:
   if (toAdd == null) throw new NullPointerException();
   в add(..)/ removePoint(..) - бессмысленны, так как Вы выбрасываете всё тот же неинформативный NPE. Есть следующие альтернативы:

• оставить NPE и документировать, в каких случаях он выбрасывается / кастомизировать сообщение или тип исключения;

• добавить аннотации - см. файл на гуглдиске;

• возвращать осмысленные специальные значения.

3. if (table.add(toAdd)) return table.add(toAdd); else return false;
   Нельзя ли это упростить?

4. Пожалуйста, называйте пакеты в соответствии с одной из распространённых нотаций - например, Google Java style guide.

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

1. В тестах не должно быть методов, в которых Вы не реализуете проверки с применением assert, sout - точно не лучшее решение в отладке кода, при необходимости лучше воспользоваться дебаггером;

2. В данный момент не проверяются ветки true для addPoint(..)/ removePoint(..), и выкидываемые Вами NPE;

3. Дублирующийся код стоит выносить во вспомогательные функции.