[SOLVED] bullet collision killing all aliens instead of one

[TheKingofJello]

when my bullet hits one alien it will automatically kill all aliens to the right of the and above it (starting with the aliens to the right, ending with the actual alien hit).

if you need more info, ask.

[airstruck]

I think you forgot a condition in your collision detection, don't you need something like `and b.y > e.y`?

[bobbyjones]

I didn't take a look, but I know I followed tutorials for space invaders and the way I stored the enemies in the table and the way I used table.remove caused me to have a lot of bugs with killing enemies. That might be something worth considering.

[TheKingofJello]

I think you forgot a condition in your collision detection, don't you need something like `and b.y > e.y`?

i didn't see that really do anything different then the script it was before I added your suggestion, but thanks for trying to help anyway.

I solved my problem when I realized I was using some iterators and table.remove()s wrong in my collision script.

[airstruck]

Yeah, I suppose if your bullets are always coming from the bottom and the enemies are always coming from the top then the bullet's Y coordinate will always be greater than the enemy's Y coordinate when the X check passes. It still seems strange to leave that out, though (you wouldn't be able to reuse that collision code for much, anyway).

[Beelz]

I didn't take a look, but I know I followed tutorials for space invaders and the way I stored the enemies in the table and the way I used table.remove caused me to have a lot of bugs with killing enemies. That might be something worth considering.

I had an issue like this before and I found counting backwards fixed it... Not sure why, maybe it was a fluke.

Code: Select all

for i=#objects.enemy,1,-1 do
     checkCollision(objects.player,objects.enemy[i])
end

[bobbyjones]

Yeah that is a common solution. Many people make a custom iterator for it called reverse ipairs or ripairs I see some people name it as. It because of the way table.remove works.