Removing event listener on collision

Solar2D General Questions/Discussion
#1

Hello, I’m not sure which way is the right one so I’m gonna ask:

Is this one right?

nloDrop = function()    nlo.collision = nloCollision     nlo:addEventListener("collision", nlo) end function nloCollision(self, event)     self:removeEventListener("collision", self) -- THIS PART     self:removeSelf()     self = nil end  

Or is this one right?

nloDrop = function()    nlo.collision = nloCollision     nlo:addEventListener("collision", nlo) end function nloCollision(self, event)     self:removeEventListener("collision", nlo) -- THIS PART     self:removeSelf()     self = nil end  

I think both ways work, but I don’t know which one is better.

Thanks.

#2

Hi @kizzwiz,

Both should be fine. I suppose I’d go with #1 because it doesn’t need to up-reference “nlo”, wherever that is declared… but in terms of performance, the difference would be imperceptible.

Brent

#3

Thanks, Brent!

#4

Hi @kizzwiz,

Both should be fine. I suppose I’d go with #1 because it doesn’t need to up-reference “nlo”, wherever that is declared… but in terms of performance, the difference would be imperceptible.

Brent

#5

Thanks, Brent!

#6

Thnx kizzwiz , for sharing this one

#7

Thnx kizzwiz , for sharing this one