How to destory object iterated in foreach loop

Question

I am detecting whether the mousepointer is within an ellipse that I have drawn in P5.js. When the mouse is in contact with the ellipse it should be deleted. I can achieve this by iterating over the array that stores the ellipses locations, checking if the mouse pointer is close enough to them to be warranted as inside by accessing the width element.w. This is a way to do it with a simple for loop:

for(var i = 0; i < locations.length; i++){
    var d = dist(mouseX, mouseY, locations[i].x, locations[i].y);
    if(d <= locations[i].w){
        locations.splice(i,1);
    }
}

I am wondering what the forEach loop way to do this would be. I premuse it would be something like the below code, but how do you actually destroy the element?

locations.forEach(function(element) {
    var d = dist(mouseX, mouseY, element.x, element.y);
    if(d <= element.w){
        //Destoy the element??? How do you do that????
    }                   
});

locations.forEach(function(element, i) { look you can an index to foreach developer.mozilla.org/es/docs/Web/JavaScript/Referencia/… and make a splice as demo behind — Álvaro Touzón
– Álvaro Touzón, Commented Oct 16, 2017 at 6:36
check the callback signature developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… function callback(currentValue, index, array) — JEY
– JEY, Commented Oct 16, 2017 at 6:36

Nandu Kalidindi · Accepted Answer · 2017-10-16 06:59:20Z

1

Try this if you don't want to deal with browser compatibility issues for .remove()

function removeDummy() {
    var elem = document.getElementById('dummy');
    elem.parentNode.removeChild(elem);
    return false;
}

The snippet is from this answer https://stackoverflow.com/a/5933167/4724167.

answered Oct 16, 2017 at 6:59

Nandu Kalidindi

6,2901 gold badge28 silver badges38 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Coco · Accepted Answer · 2017-10-16 06:57:05Z

1

use filter function of Array, def variable preserve new array

answered Oct 16, 2017 at 6:57

Coco

5071 gold badge5 silver badges16 bronze badges

Comments

Nitheesh · Accepted Answer · 2017-10-16 06:36:12Z

0

Try using element.remove(), but this has cross-browser compatibility issues.

answered Oct 16, 2017 at 6:36

Nitheesh

20.1k3 gold badges28 silver badges52 bronze badges

Collectives™ on Stack Overflow

How to destory object iterated in foreach loop

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related