Правильно удаление мувиков (в массивах)

[artemst]

Проблема оказывается в том, что я не удалял элементы из самого массива. Как собственно это сделать грамотно? Есть 3 метода, pop, splice, shift - но они все удаляют или начало или конец или несколько, а как удалить именно тот элемент который столкнулся? метод delete тоже не действует, так как он заменяет элемент на undefined, но не удаляет его

[Zebestov]

delete действует ) вот так он и действует
не поверишь:

var arr:Array = new Array();
trace(arr[123]); // выдаст undefined 0_о

так что в цикле просто проверяй на undefined... в идеале 2 раза - в наружном и во внутреннем.

А еще лучше:

public function hitsp():void {
        var i:uint;
        var j:uint;
        for each (i in listvr) {
        for each (j in listvz) {
             if (listvr[i].hitTestObject(listvz[j])) {
                  removeChild(listvr[i]);
                  removeChild(listvz[j]);
                  delete listvr[i];
                  delete listvz[j];
                  break;
             }
        }
        }
}

[artemst]

Zebestov - ваша функция вызывает ошибку при столкновении и вообще ничего не удаляется ))

[Zebestov]

Теперь должно работать — надо было поменять местами delete и removeChild )

[artemst]

менял, эффект такой что элемент всё-равно остаётся под видом undefined, так что просто вид ошибки меняется на "TypeError: Error #1010: A term is undefined and has no properties.".

[Zebestov]

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

public function hitsp():void {
        // здесь будем отмечать кандидатов на удаление
        var vrArr:Array = new Array();
        var vzArr:Array = new Array();
 
        var i:uint;
        var j:uint;
 
        // сначала фиксируем все столкновения
        for each (i in listvr) {
        for each (j in listvz) {
             if (listvr[i].hitTestObject(listvz[j])) {
                  vrArr[i] = 1;
                  vzArr[j] = 1;
             }
        }
        }
 
        // а потом лишь зачисточка
        for each (i in listvr) {
             if (vrArr[i] == 1) {
                  removeChild(listvr[i]);
                  delete listvr[i];
             }
        }
        for each (j in listvz) {
             if (vzArr[j] == 1) {
                  removeChild(listvz[j]);
                  delete listvz[j];
             }
        }
}

[artemst]

В любом случае вы сначала присваиваете элементу значение 1, а потом если оно равно одному - присваиваете undefined (меодом delete) - соответственно элемент таки не удаляется, а ошибка всё та же. (кстати в этих двух функциях, что вы дали удаляется как раз 1 элемент при столкновении с несколькими, а код который я писал до этого с for... удаляет как и надо - все столкнвшиеся элементы), но спасибо попытку помочь

[Zebestov]

Так погоди! как это не удаляется о_0 у тебя этот элемент еще где-то кроме массива живет? в массиве не единственная ссылка на него? если нет, то:

1. removeChild
2. удаление ссылки на него (там теперь живет undefined)
3. отсутствие слушателей (об этом ты не упоминал)

ведет к тому, что элемента больше нет (нет, из памяти он потом удалится) и обратиться к нему больше никак нельзя.

вот вопрос - а ты как определяешь, что элемент остался?

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

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

...а код который я писал до этого с for... удаляет как и надо - все столкнвшиеся элементы

ххех ) так потому и "удаляет", что он их не удаляет (по прежнему оставляет в массиве), а лишь убирает с экрана (removeChild)

[artemst]

вобщем вся проблема решилась строчкой

delete listvr.splice(i,1);

Итого конечный код:

public function hitsp():void {
        for(var i:uint=0;i<listvr.length;i++) {
        for(var j:uint=0;j<listvz.length;j++) {
           if (listvr[i].hitTestObject(listvz[j])) {
				removeChild(listvr[i]);
				delete listvr.splice(i,1);
           }
        }
        }
}

а элемент из массива listvz у меня удаляется в другой функции.

[Zebestov]

delete listvr.splice(i,1);

да зачем же delete?

Ваш метод не работает ) при первом же столкновении вы срезаете массив и продолжаете цикл, а listvr[i] — это уже следующий элемент, который "на лету" встал на место удаленного! так что вы по прежнему проверяете лишь первое столкновение.

Другими словами — вы запутываетесь.
Если вы ничего не утаеваете ) то метод, что я предлагал выше - работает как надо, внимательней его разберите; и если смущают пустоты undefined - ну так подставьте в мой метод свои splice вместо моих delete! это ничего не изменит кроме лишнего времени на бессмысленное "смыкание рядов"