Как удалить объект из массива

[Vansimar]

Есть массив пулек, если пулька улетает за границу экрана или попадает в цель ,то я ее удаляю из массива так:

InGame.bullets.splice(InGame.bullets.indexOf(this), 1);

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

[Akopalipsis]

Странно.. Как такое может быть? Вы передали пульку на удаление и только в методе определился индекс, ход выполнения прерывается и в этот метод попадает другая пулька. Индекс пересчитывается. но старый где-то сохраняется... Даже представить сложно, как такое возможно.

[Vansimar]

Ну я убедился в этом! я сам офигел! в итоге я переписал немного и все заработало, в условии , где у меня колизится пулька с врагами и проверяется вылет пульки за границы экрана я ставлю свойство isDead в true,
а потом в цикле апдейта я проверяю isDead и там же удаляю если в тру это значение

private function updateBullets():void{
var i:int = bullets.length;
while(i--)
{
if(bullets[i].isDead == true){
bullets.splice(bullets.indexOf(bullets[i]), 1);
}else
bullets[i].update();
}
}

[Akopalipsis]

Так получше будет.

private function updateBullets():void{
var i:int = bullets.length;
var bullet:Bullet;
while(i--)
{
bullet = bullets[i];
if(bullet.isDead == true){
bullets.splice(bullets.indexOf(bullet), 1);
}else
bullet.update();
}
}

И я был не прав в предыдущем сообщении, я не подумал

[dimarik]

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

Так получше будет.

if(bullet.isDead == true)

Я думаю, что получше будет сначала поставить пробел после if и перед его скобкой, потом набраться мужества и почитать справку по if, а потом придет прозрение, что дело можно усугубить дальше. И поржать, конечно.

if ( ((((true === true) == true) == true) == true) == true) ...)

спойлер. Ты и так проверяешь boolean bullet.isDead.

[Akopalipsis]

я на это даже внимания не обратил))) А лучше я употребил про присвоение элемента свойству.

[Isfet]

я бы все таки делал это статичной функцией , аля InGame.removeBullet(this);

[caseyryan]

спойлер. Ты и так проверяешь boolean bullet.isDead.

Как бы да, но по этому вопросу можно и холивар развернуть)
Многие намеренно пишут явную проверку, чтобы проще было читать. Понятно, что в этом случае для программы разницы нет, но вот при чтении человеком, будет сразу понятно, что там булево значение, а не приведение к типу Boolean.
Что бы ты подумал, если бы увидел вот такой кусок г-кода? (не обращая внимания на то, что тут названия не по конвенциям)

if (someObject.hasOne) {
   // some code
}

Что hasOne это Boolean? Или может это проверка на null с неявным приведением к Boolean, например?

я бы все таки делал это глобальной функцией , аля InGame.removeBullet(this);

Это не глобальная функция

[Psycho Tiger]

InGame.bullets.splice(InGame.bullets.indexOf(this), 1);

indexOf возвращает -1, если объект не найден.
splice при удалении с -1 удаляет последний элемент.
Без if'а не обойтись.

[Akopalipsis]

но вот при чтении человеком, будет сразу понятно, что там булево значение

так оно и так понятно, ведь isЖе

Добавлено через 3 часа 59 минут

Что бы ты подумал, если бы увидел вот такой кусок г-кода?

Feathers писал ***-кодер, там есть строчка -

var hasNameInitializer:Boolean = false;