If a value comes through an if/then statement remove element from array - Javascript

Question

I am looping through elements in an array. If a value is not within a specific range I want to remove the element (that the value is associated with).

Here's what i have so far (see code below) - I am using the pop method to remove the element as it comes through but the last element in the array always gets removed and NOT the element/value within the if/then statement. I've tried the splice method as well but I can't get that to work. Any idea on how to do this?

    var h = [
    	["29","Verbena St", "500", "2", "2,702"],
    	["36", "Quitman St", "400", "2", "1,700"],
    	["32", "Alan Dr", "500", "2", "2,408"],
    	["34", "Newton St", "300", "2", "1,954"],
    	["30", "Soth Pl", "400", "2", "1,509"]
    ];

    var hs = [
     	["Verbena St"],
    	["Quitman St"],
        ["Alan Dr"],
     	["Newton St"],
     	["Soth Pl"]
    ];


function Location (){

    for (var r = 0; r <= h.length; r++){
        var p = h[r][0];
	    var address = h[r][1]; // Get address
        if (p >= 21 && p <= 33 && address == hs[r]){
            console.log(address);
        }
        else {
            console.log(address + " - OVER 33");
            h.pop(address);
            console.log(address + " - REMOVED");
        }
   }
};

Location();

because the pop() method removes the last element from an array — yvoytovych
– yvoytovych, Commented Sep 3, 2017 at 18:10

Djaouad · Accepted Answer · 2017-09-03 18:30:33Z

It's not good practice to remove elements from an array while iterating through it, instead, you can filter it using filter(), like this :

var h = [
  ["29","Verbena St", "500", "2", "2,702"],
  ["36", "Quitman St", "400", "2", "1,700"],
  ["32", "Alan Dr", "500", "2", "2,408"],
  ["34", "Newton St", "300", "2", "1,954"],
  ["30", "Soth Pl", "400", "2", "1,509"]
];

var hs = [
  ["Verbena St"],
  ["Quitman St"],
  ["Alan Dr"],
  ["Newton St"],
  ["Soth Pl"]
];

function Location (){
  h = h.filter(function(r, i){ // r = value, i = index
    var p=r[0];
    var address = r[1]; // Get address
    return p >= 21 && p <= 33 && address == hs[i];
  });
};

Location();
console.log(h);

Or, in case you want to display messages, you can use another array and only push the elements you want to it (better use a forEach) :

var h = [
  ["29","Verbena St", "500", "2", "2,702"],
  ["36", "Quitman St", "400", "2", "1,700"],
  ["32", "Alan Dr", "500", "2", "2,408"],
  ["34", "Newton St", "300", "2", "1,954"],
  ["30", "Soth Pl", "400", "2", "1,509"]
];

var hs = [
  ["Verbena St"],
  ["Quitman St"],
  ["Alan Dr"],
  ["Newton St"],
  ["Soth Pl"]
];

function Location (){
  var g = [];
  h.forEach(function(r, i){ // r = value, i = index
    var p = r[0];
    var address = r[1]; // Get address
    if (p >= 21 && p <= 33 && address == hs[i]){
    	console.log(address);
      g.push(r); // push it
    }
    else{
      console.log(address + " - OVER 33 - REMOVED"); // do nothing
    }
  });
  h = g;
}

Location();
console.log(h);

Nina Scholz · Accepted Answer · 2017-09-03 18:20:09Z

For removing elements of an array, you could loop from the end and splice if necessary, because the next smaller index is not affected.

function updateLocation() {
    var address, p, r = h.length;
    while (r--) {
        p = h[r][0];
        address = h[r][1]; // Get address
        if (p >= 21 && p <= 33 && address == hs[r]) {
            console.log(address);
        } else {
            console.log(address + " - OVER 33");
            h.splice(r, 1);
            console.log(address + " - REMOVED");
        }
    }
}

var h = [["29", "Verbena St", "500", "2", "2,702"], ["36", "Quitman St", "400", "2", "1,700"], ["32", "Alan Dr", "500", "2", "2,408"], ["34", "Newton St", "300", "2", "1,954"], ["30", "Soth Pl", "400", "2", "1,509"]],
    hs = [["Verbena St"], ["Quitman St"], ["Alan Dr"], ["Newton St"], ["Soth Pl"]];

updateLocation();
console.log(h);

Collectives™ on Stack Overflow

If a value comes through an if/then statement remove element from array - Javascript

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related