Как я могу оптимизировать этот метод?

Что ж, чтение проекта и поиск того, что можно оптимизировать, наверняка потребует времени.

Начните с удаления нескольких вызовов rectanglePacker.getRectangle(i) внутри циклов.

Например :

    itemName = itemNames[rectanglePacker.getRectangleId(i)];
    itemTexture = itemTextures[rectanglePacker.getRectangleId(i)];
    rect = rectanglePacker.getRectangle(i);

возможно, могло быть:

    rect = rectanglePacker.getRectangle(i);
    itemName = itemNames[rect];
    itemTexture = itemTextures[rect];

Если getRectangle действительно просто "получает прямоугольник" и ничего не устанавливает.

Я думаю, что более серьезная проблема заключается в том, почему, почему вы ДОЛЖНЫ делать это во время выполнения, в ситуации, когда это не может занять больше времени? Это экспансивная операция, независимо от того, насколько сильно вы ее оптимизируете, вы, вероятно, в конечном итоге получите около 40 мс или около того при выполнении в AS3.

Вот почему такие операции следует выполнять во время компиляции или во время «загрузочных экранов» или других «переходов», когда частота кадров не критична и когда вы можете себе это позволить.

В качестве альтернативы создайте другую систему на С++ или каком-либо другом языке, который действительно может обрабатывать числовые вычисления, которые дают вам конечный результат.

Кроме того, когда дело доходит до проверки производительности, да, вся функция занимает 53 мс, НО, где используются эти миллисекунды? 53 мс ничего не говорит и является всего лишь «профилированием служебных данных», где вы нашли виновника, вам нужно разбить его на более мелкие фрагменты, чтобы собрать достоверную информацию о том, что НА САМОМ ДЕЛЕ требует времени внутри этой функции.

Я имею в виду, что внутри этой функции у вас есть 3 цикла for, несколько вызовов других классов, приведения, удаления, создания. Это не похоже на то, что вы делаете что-то одно, эта функция, вероятно, приводит к ~ 500 строкам кода и миллиарду операций процессора. И вы понятия не имеете, где это используется. Я бы предположил, что именно rectanglePacker.packRectangles(); занимает 60% этого времени, но без профилирования мы с вами не знаем, что оптимизировать, у нас просто недостаточно данных.

Если вам НЕОБХОДИМО сделать это во время выполнения в AS3, я бы порекомендовал сделать это в течение нескольких кадров и равномерно распределить рабочую нагрузку в течение 10 кадров или около того. Вы также можете сделать это с помощью другого потока и рабочих. Но больше всего это похоже на ошибку дизайна, поскольку это, вероятно, можно было бы сделать в другое время. А если нет, то на другом языке, который лучше справляется с такими операциями.

Самый простой способ профилировать это — добавить пару меток времени, похожих на:

var timestamps:Array = [];

Затем нажмите getTimer() в разных местах кода, а затем распечатайте их, когда функция будет выполнена.

Как говорили другие, маловероятно, что причиной плохой производительности является неоптимизированный код AS. Вывод из профилировщика (например, Scout) будет очень полезен. Однако, если вашей целью является просто добавление новых текстур, я могу предложить несколько оптимизаций:

1. Зачем вам каждый раз заново генерировать весь атлас (вызывая Assets.getTextures() и создавая новую текстуру рендеринга)? Почему бы вам просто не добавить новые элементы в существующий атлас? Создание нового RenderTexture (и, соответственно, новой текстуры в памяти GPU) — очень затратная операция, так как требует синхронизации между CPU и GPU. С другой стороны, отрисовка в RenderTexture выполняется полностью внутри GPU, поэтому занимает гораздо меньше времени.

2. Если вы размещаете каждый элемент в сетке, вы можете избежать использования RectanglePacker, поскольку все ваши прямоугольники могут иметь одинаковые размеры, соответствующие размерам сетки.

Изменить:

Уточню, некоторое время назад у меня была аналогичная проблема: мне приходилось регулярно добавлять новые элементы в существующий атлас. И производительность этой операции была вполне приемлемой (около 8 мс на iPad3 при использовании динамической текстуры 1024x1024). Но я использовал тот же объект RenderTexture и тот же объект Sprite, которые содержали элементы моего динамического атласа. Когда мне нужно добавить новый элемент, я просто создаю новый Image с нужной текстурой (отдельно или из другого статического атласа), затем помещаю его в контейнер Sprite, а затем перерисовываю этот контейнер в RenderTexture. Аналогично с удалением/модификацией элемента.