Относительно задания — там явно прописано, что должны быть кнопки "+" и "-", у автора это "add" и "minus" (еще и total есть). Это мелочь, конечно, но всё-таки, мне кажется, что нужно подходить очень внимательно к подобным вещам.
По коду:
Код AS3:
private var planets:Vector.<Planet>;
— я бы посчитал большим недочетом. С моей точки зрения нужно было так:
Код AS3:
private var _planets:Vector.<Planet>;
"onClick", "onMove" и т.д. я бы переписал на "mouseClickHandler" или типа того. Хотя это не ошибка, да.
Про "for (var i:int = 0; i < planets.length; i++)" уже говорили.
В Universe свойство graphics вполне можно закэшировать.
Думаю, что вполне неплохо выполненное тестовое задание. Но само задание так себе, не показательно, как мне кажется.
Добавлено через 2 минуты
А, и еще:
Цитата:
|
Причина: не оптимизированный код.
|
Мне кажется, это довольно-таки, нехорошо в таком формате отвечать на выполненные тестовые задания. В хорошем случае я (как соискатель) ожидаю получить нормальный развёрнутый ответ.