Показать сообщение отдельно
Старый 17.11.2013, 03:29
Wolsh вне форума Посмотреть профиль Отправить личное сообщение для Wolsh Найти все сообщения от Wolsh
  № 17  
Ответить с цитированием
Wolsh
Нуб нубам
 
Аватар для Wolsh

модератор форума
Регистрация: Jan 2006
Адрес: Бердск, НСО
Сообщений: 6,445
Поработай над названиями, может станет понятней что это.
Функции не должны называться существительными. Функция что-то "делает", она глагол или деепричастие.
Функция не может называться state(). Может быть checkState(), или isFinish(), если она Boolean-причастие (но тогда может есть смысл делать паблик геттер animationFinished и уж точно нужно событие об окончании).
Что такое _isY ? Игрек — что?
Почему класс называется ...Container, но не наследует DOC? Назвал бы ScaleAnimator например...
Почему метод называется add(), если он не Добавляет (еще один элемент к остальным), а устанавливает Единственный, да еще и возвращает новый экземпляр того же класса, у экземпляра которого мы этот метод вызвали (голова кругом)? Это для сахара scaleMaster.add().add().add().add()? Необходимость в каждом add() указывать целевой объект превращает такой сахар в соль моих слёз. Что это за сахарная цепочка, в середине которой может полностью поменяться объект, с которым работаем?
На вскидку, я бы наверное сделал статический метод класса createScaleAnimator() нормальный параметризированный конструктор, которому бы отдавал целевой объект раз и навсегда, и получал экземпляр ScaleAnimator'a, уже привязанный к конкретному объекту. И вот ему уже можно было бы делать инъекции add().add().add() (возвращающие ссылку на этот же экземпляр, а не создающие новые) с параметрами анимации и целевой трансформации.
Ну это так, первое что в глаза бросилось, и не заостряя внимание на смысле всего этого действия))
__________________
Reality.getBounds(this);


Последний раз редактировалось Wolsh; 17.11.2013 в 15:59. Причина: зачеркнул ночной бред