Качество кода.

[in4core]

in4core, хорош уже ко мне цепляться

А к кому еще, у меня больше друзей нет ))))

SuriTheAngel - это тебе не AS. Книжка-раскраска тут тебе не сильно поможет, а вот годы тренировок да, по Unity еще можно что то найти.

А с чего такое рвение? Не уж то ли half-life 3 писать решил ?

[ZackMercury]

half-life 3

full-life минус 1.

[Rzer]

Как я не люблю эти подчёркивания... без них гораздо читабельней код.

Если их использовать, то можно вместо:

_onClick = onClick

писать:

this.onClick = onClick

А если у тебя FlashDevelop, то просто выбирать аргумент функции и жамкать ctrl+shift+1

А вот это:

if(mouseY <= _drag.height / 2)
   _drag.y = 0;
else
   _drag.y = _height - _drag.height;

Как по мне лучше записать:

drag.y = (mouseY <= drag.height / 2) ? 0 : height - drag.height

Про вот это:

_background = new Sprite();
_background.graphics.lineStyle(1, 0x0d3d1f, 0.7);
_background.graphics.beginFill(0x415046, 0.6);
_background.graphics.drawRect(0, 0, App.SCR_W - 1, CONSOLE_HEIGHT - 1);
_background.graphics.endFill();

Старайся избегать двух точек в одной строке (это и полезно - вдруг рисовать нужно в другом объекте, менять нужно только одну строку):

background = new Sprite();
var canvas:Graphics = background.graphics;
canvas.lineStyle(1, 0x0d3d1f, 0.7);
canvas.beginFill(0x415046, 0.6);
canvas.drawRect(0, 0, App.SCR_W - 1, CONSOLE_HEIGHT - 1);
canvas.endFill();

Не понятно, зачем в LiteButton.as используешь слово super.

private function type(e:KeyboardEvent):void
{
   if(e.keyCode == Keyboard.ENTER)
   {
      prompt(_promptField.text);
      _promptField.text = "";
   }
}

Я бы написал:

private function type(e:KeyboardEvent):void{
 
    if(e.keyCode != Keyboard.ENTER) return;
 
    prompt(promptField.text);
    promptField.text = "";
 
}

Получается более эстетично, всё под одним отступом. Вообще не люблю вложенности, стараюсь чтобы везде была максимум одна, иначе дробить на более мелкие смысловые блоки.

Ну вот ещё фрагмент:

if(command.indexOf(" ") > -1)
   operator = command.substring(0, command.indexOf(" ")).replace("/", "");
else
   operator = command.replace("/", "");

дважды выполняется поиск по массиву, не бойся выносить результаты во временные переменные:

 
operator = command;
 
var anIndex:int = operator.indexOf(" ");
 
if(anIndex != -1){
   operator = operator .substring(0, anIndex);
}
 
operator = operator .replace("/", "");

[illuzor]

drag.y = (mouseY <= drag.height / 2) ? 0 : height - drag.height

А по мне, эта конструкция плохо читабельна.
Я б записал так:

_drag.y = 0;
if(mouseY > _drag.height >> 1) 
   _drag.y = _height-_drag.height;

[MikroAcse]

_drag.height >> 1

Это что за магия такая?

[illuzor]

MikroAcse, справка для чего существует?

[ZackMercury]

А по мне, эта конструкция плохо читабельна.

Да вроде норм, тернарные операторы делают код компактней и читабельней, по крайней мере для меня.

Добавлено через 51 секунду

справка для чего существует?

>>1 всегда делит число напополам?
И эта штука по вашему читабельней? ...

Добавлено через 37 минут

... Не понравилось именование. ... class S ...

Зато вы не поверите, как легко стало импортировать этот класс.
Прям лишь одну букву написать. И то же самое при использовании цветов, текстформатов.

[illuzor]

>>1 всегда делит число напополам?

<sarcasm>Нет, только иногда</sarcasm>

И эта штука по вашему читабельней?

Да.

Зато вы не поверите, как легко стало импортировать этот класс.
Прям лишь одну букву написать. И то же самое при использовании цветов, текстформатов.

Классам нужно давать осмысленные названия. Насчёт импортов - бред. В нормальных ide есть автокомплит и автоимпорт.

[ZackMercury]

Во флешдевелопе автоимпорт не всегда срабатывает, например, у меня не хотел нивкакую находить класс Keyboard, пока я его не указывал в импорте.
Сам проект приложения, со всеми его классами у меня хранится в формате Flash IDE, так как там моя библиотека с битмапами и кнопками, а она не хочет автоимпортить то, что мне нужно. А автокомплитить не хочет то, что не заимпорчено. Поэтому всё вручную.

[caseyryan]

Во флешдевелопе автоимпорт не всегда срабатывает, например, у меня не хотел нивкакую находить класс Keyboard, пока я его не указывал в импорте.
Сам проект приложения, со всеми его классами у меня хранится в формате Flash IDE, так как там моя библиотека с битмапами и кнопками, а она не хочет автоимпортить то, что мне нужно. А автокомплитить не хочет то, что не заимпорчено. Поэтому всё вручную.

Такое бывает если где-то, например, не поставлена точка с запятой в конце строки, или переменные по-конски объявлены через запятую. Или еще что-нибудь сделано коряво. В FD нет никакой магии, чистая логика. Глюки конечно бывают (как и везде), но очень редко
Так что не надо катить бочку на FD ) У меня она все на раз два импортирует и автодополняет