llday / solar-system Goto Github PK

в ряде случаев что-то пошло не так с настройками области видимости полей/ методов; как минимум, не вполне понятно, зачем нужны public методы для классов, обладающих package private конструкторами;

в TimeTakt delta(...) и time почти дублируют друг друга - стоит вынести вспомогательную функцию;

методы получения Painter'a для классов Sun и Planet также практически идентичны - следовательно, их можно вынести выше по иерархии, для получения изменяемого значения задав, например, переопределяемый по-своему наследниками абстрактный метод;

SpaceObjectListenerJComponent - скорее не Listener, а Publisher, т.к. он сам никого не слушает, но публикует информацию для подписанных компонентов;

fyi - проверку на то, пустая ли строка, можно реализовывать с помощью метода isEmpty();

(Math)

jfyi - при работе с NPE дополнительно упростить себе жизнь можно с использованием аннотаций.