Показать сообщение отдельно
Старый 05.07.2015, 00:54
miramax вне форума Посмотреть профиль Отправить личное сообщение для miramax Посетить домашнюю страницу miramax Найти все сообщения от miramax
  № 13  
Ответить с цитированием
miramax
 
Аватар для miramax

Регистрация: Oct 2005
Адрес: Борисоглебск
Сообщений: 1,702
Отправить сообщение для miramax с помощью ICQ Отправить сообщение для miramax с помощью AIM Отправить сообщение для miramax с помощью MSN Отправить сообщение для miramax с помощью Yahoo Отправить сообщение для miramax с помощью Skype™
Сорри, за прямоту - сам не люблю когда делают код ревью мне )
Не стал бы заострять внимание на e или event и прочей лабуде с табуляциями.

Сразу вижу что:
1. мало опыта в МВЦ, не очень логично распределены обязанности. Всё свалено в кучу.
- разделить на модель, вид , контроллер - дать описания каждому паблик методу в формате ас док на английском
2. Не понимает , что такое интерфейсы. Избыточно заявленые интерфейсы. Впечатление , что разработчик - смутно себе представляет зачем они нужны - типа "так правильно"
- Убрать интерфейсы вовсе, либо заморочиться и впилить инъекции (пусть интервьюер сам курит роботлегс например , покажите что-то интересное=)) и имплементировать в них обычные дисплей объекты вместе со старлингом, приправив парочкой геном2д объектов с шаред 3Дконтекстом.

3. Не гуглит - не заюзал всем известные решения типа isolib, и получилась пародия - которая точно будет багать с объектами на близких координатах и разных слоях.

4. А где ресайз ? А где фпс на 60 ? Где обработка ошибок ? Возможно разработчик никогда не сталкивался с этим. Варианта 2 - либо "лепил мини-игры-баннеры", либо флексил на простом уровне.
ФПС 24, а зачем 40 ?
Код AS3:
new MovieClip(atlas.getTextures("boom_"), 40);
Тестовое задание - убойное, честно говоря. Но ту реализацию, что есть в исходнике, - реально за 1 день можно сделать.
Ничего из дополнительных тасков не реализовано.
__________________
AS3 | www.FLAPS.ru | Русские флэшеры самые умные флэшеры в мире. ©