Тестовое задание.

[caseyryan]

Ну, в плане оптимизации здесь это не особо критично. Просто видимо хотели проверить знания по разным нюансам.
К примеру, длину массива не нужно просчитывать в цикле каждый раз.
То есть лучше написать так:

var length:int = planets.length;
for (var i:int = 0; i < length; i++) {
   planets[i].update(target);
}

а не так:

for (var i:int = 0; i < planets.length; i++) planets[i].update(target);

Плюс у цикла должно быть тело, чтобы было удобнее читать. Ведь этот код не для себя пишется.

Второе. Известно, что доступ к константам всегда медленнее, чем доступ к локальным переменным.
Поэтому можно было часто используемые константы внутри функций "прилепить" к локальным переменным.
К примеру не использовать везде в функции вызов Math.PI
а сначала присвоить ее значение локальной переменной
var pi:Number = Math.PI;
а потом уже использовать ее.

По-поводу оператора switch. Здесь уже были темы с таким обсуждением, и приходили к выводу, что в ас3, if / else работает быстрее, чем switch. В отличие от некоторых других языков. Так что свич я бы тоже выкинул.

Так же есть такая мелочь как вид записи

if(moon)

неявное приведение к типу Boolean.
По логике, должна быть проверка типа, и после этого приведение к типу Boolean

if(moon != null)

А так должна сразу приводить к Boolean
Во многих языках первый тип записи вообще не позволит программе скомпилироваться.


Это так, то, что можно сказать на вскидку