метод getPixels возвращающий ByteArray

[etc]

А зачем их копировать из картинки в массив, чтобы потом в другую картинку вставлять? Не проще напрямую getPixel из одной и setPixel в другую?

[vapes]

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

[etc]

Всё, у меня вывих мозга.

[vapes]

Ух ты убираешь: this.myarray.splice(pixelNumber-1,1);
и производительность в раз 5 возрастает

[vapes]

теперь вопрос:как избежать повторения пикселов?

[MerlinTwi]

Сообщение от vapes

теперь вопрос:как избежать повторения пикселов?

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

// массив с точками
var ar:Array = new Array();
// размеры картинки
var bitmapWidth:Number = 100;
var bitmapHeight:Number = 100;
// собираем в массив координаты всех точек
for (var i:Number=0; i<bitmapHeight; i++) {
	for (var j:Number=0; j<bitmapWidth; j++) ar.push(new Point(j, i));
}

// перетасовываем массив по алгоритму Fisher-Yates
var a:Number = ar.length;
var b:Point, c:Number;
while(a--) {
	c = Math.floor(Math.random()*(a+1));
	b = ar[c];
	ar[c] = ar[a];
	ar[a] = b;
};
  
// Тестируем что получилось

// Сделаем текстовые картинки
var myBitmapData:BitmapData = new BitmapData(bitmapWidth, bitmapHeight, false, 0xFF0000);
var resultBitmapData:BitmapData = new BitmapData(bitmapWidth, bitmapHeight, false, 0xFF);
addChild(new Bitmap(resultBitmapData))
// выводим случайные точки по enterFrame
addEventListener(Event.ENTER_FRAME, onDraw);

function onDraw(e:Event):void {
	var n:int = 200; // выводим 200 точек
	while (n-- && ar.length) {
		// берем следующую точку
		var pixel:Point = ar.pop();
		// рисуем ее
		trace("Draw:", pixel.x, pixel.y);
		resultBitmapData.setPixel(pixel.x, pixel.y, myBitmapData.getPixel(pixel.x, pixel.y));
	}
}

[__i]

Сообщение от MerlinTwi

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

...............
function onDraw(e:Event):void {
	var n:int = 200; // выводим 200 точек
	while (n-- && ar.length) {
		// берем следующую точку
		var pixel:Point = ar.pop();
		// рисуем ее
		trace("Draw:", pixel.x, pixel.y);
		resultBitmapData.setPixel(pixel.x, pixel.y, myBitmapData.getPixel(pixel.x, pixel.y));
	}
}

кхм тоже правильно, но не рационально... именно в этой части

	var n:int = 200; // выводим 200 точек
	while (n-- && ar.length) {
		// берем следующую точку
		var pixel:Point = ar.pop();

лишний поп, лишнее вычисление ar.length, ......

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

var currentPos : number = 0;
........
var countPoint : number  =  ((currentPos+200)> ar.length) ? 200 : ar.length-currentPos
var pixel:Point;
while (countPoint >0) {
 pixel = ar[countPoint + currentPos ];
trace("Draw:", pixel.x, pixel.y);
		resultBitmapData.setPixel(pixel.x, pixel.y, myBitmapData.getPixel(pixel.x, pixel.y));
countPoint--;
}
currentPos+=countPoint ;
if (currentPos>ar.length) trace ("the end");
)

ну а в самом конце просто бахать наш массив и все

[miramax]

юзайте byteArray для хранения больших массивов чисел - там сумасшедшая скорость....

[MerlinTwi]

byteArray действительно быстрее, но в данном случае сомнительно его применение. Как красиво и быстро перетасовать поток байтов я не представляю.
Если говорить об оптимизации, то следует еще при копировании точек лочить битмап:

resultBitmapData.lock();
while (n-- && ar.length) {
 ...
}
resultBitmapData.unlock();

P.S.
Кстати, можно устроить соревнование на создание наиболее быстрого алгоритма для решения этой задачи, есть желающие?

[miramax]

Сообщение от MerlinTwi

P.S.
Кстати, можно устроить соревнование на создание наиболее быстрого алгоритма для решения этой задачи, есть желающие?

Если будут хорошие идеи, обязательно выложу здесь