MikroAcse ActionScript 3.0 Library .5 [TEST]

[ChuwY]

Сообщение от -De-

n неплохо бы uint сделать.

Спасибо. Исправил. Бес попутал.

Сообщение от -De-

В этом вашем FisherYatesRandomizer надо вместо len + 1 просто len

А вот тут вас бес попутал.

len при первом же вхождении в цикл на самом деле (_currentArray.length - 1), а Math.random() единицу не возвращает.
Поэтому, чтобы захватить крайний элемент, прибавляем единицу.

Добавлено через 45 часов 21 минуту
Тк тоже сел писать свой твин, решил посмотреть все, что под руку попадутся.

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

*
* CALL
*/
private function callFunction(func:String):void {
  var arr:Array=this['on'+func+'Params'];
  if(arr){
    if(arr[1].length>0)
      if(arr[2]==true) arr[0](arr[1], this);
      else arr[0](arr[1]);
    else
      if(arr[2]==true) arr[0](this);
      else arr[0]();
  }
}

Кроме того, что прочитать невозможно, неверно вызываете коллбеки.
Советую почитать про apply/call.

А если вы и собираетесь передавать массив, то лучше об этом сообщить в документации.
"дополнительный параметр" тоже невнятный.
Кстати, вот список тегов для асдоков.
Примеры лучше оформлять с тегом code хотя бы. Или атрибутом example.

Добавлено через 46 часов 19 минут
К чести создателя, работает довольно шустро.

[MikroAcse]

Советую почитать про apply/call.

Я же обновление выложил
Там все исправлено, и твин обновлен.
Стал более легче.
Ну и еще подправлены некоторые ошибки.

К чести создателя, работает довольно шустро.

Спасибо, старался.

[ChuwY]

Если уж ты дошел до создания асдоков, то выкладывай все-таки свою библиотеку в публичный репозиторий типа гитхаба, или битбакета, или гуглокода и давай ссылку именно на репозиторий, чтобы не нужно было по форуму искать актуальную версию.

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

private function callFunction(funcName:String):void {
  var arr:Array = this['on' + funcName + 'Params'];
  if (!arr) {
    return;
  }
  var callback : Function = arr[0];
  if(!(callback)){
    return;
  }
  var params       : Array = arr[1]||[];
  var isTweenParam : Boolean = arr[2];
  if(isTweenParam){
    params.push(this);
  }
  callback.apply(null, params);
}

Старайтесь избегать большой вложенности кода. Это сильно затрудняет чтение.
Выход из функции при неверных параметрах предпочтительнее, чем вход в блок при верных, например, в данном случае.

И конструкция выбора параметров для apply переусложнена и неверна.

{
  if (arr[2] == true) {
    arr[0].apply(null, this);
  } 
  else 
  {
    arr[0]();
  }
}

apply принимает вторым параметром именно массив, так что нужно было написать хотя бы

arr[0].apply(null, [this]);

, а еще лучше, примерно как я выше.

[illuzor]

MikroAcse, пара статей в тему http://habrahabr.ru/post/125799/
http://habrahabr.ru/post/141160/

[Котяра]

Я уже советовал в блогах ещё. Не слушается совета.
и ИМХО
https://bitbucket.org/ и mercurial кошернее.