Removing elements from multi dimensional array using value in Javascript

Question

I have a multidimensional array is :-

serialNos = [{id:'12',serial:'DAS245'},{id:'13',serial:'DAS246'},{id:'15',serial:'DAS247'}]

the ids and serials are random and i use push function to push them inside the array as i get those.

serialNos.push({id:'16',serial:'DAS248'});

Now, the thing is at times I have to remove elements from this array but all i get is serial not the id i.e., DAS248. How do I delete it in easy steps? I dont want to go thru iteration method as it takes too much memory and is not efficient. Also I want to restore order i mean indexes of serialNos in ascending order without any gaps.

NOTE : ids and serials both are unique

Platform : Javascript

If u have an iterative way which you think is effecient please let me know that too, am interested. I just want the most efficient way possible. — Jinendra Khobare
– Jinendra Khobare, Commented Feb 14, 2014 at 21:09
serial:DAS248 is invalid unless DAS248 references something.... I'd imagine it's supposed to be a string literal, i.e.: serial:"DAS248". — canon
– canon, Commented Feb 14, 2014 at 21:22
yea those are strings i evaluate while pushing, let me edit those, thanks — Jinendra Khobare
– Jinendra Khobare, Commented Feb 14, 2014 at 21:23

Community · Accepted Answer · 2017-05-23 11:49:48Z

5

Assuming you don't have any repeated serial numbers, you could make serialNos an object instead of an array:

serialNos = {
    'DAS245': {
        id: 12,
        serial: 'DAS245'
    }
    // etc
};

This way, you can just delete the property using delete serialNos['DAS245'];.

With this method, there's no "sorting" this object, but when you iterate through it, you can use this solution.

edited May 23, 2017 at 11:49

CommunityBot

11 silver badge

answered Feb 14, 2014 at 21:07

Travesty3

14.5k7 gold badges64 silver badges103 bronze badges

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

4 Comments

Jinendra Khobare Over a year ago

the ids are unique. but while pushing am facing difficulties syntax errors. Can u tell me how to push them? if i have ids and serials in variables say serialID and serialVal?

scrappedcola Over a year ago

@JinendraKhobare you don't use push with objects. Start by learning about the basics of the language: developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/…

Jinendra Khobare Over a year ago

Okie, if not push then how do i add objects dynamically in the array?

Travesty3 Over a year ago

@JinendraKhobare: serialNos['DAS245'] = {id: 12, serial: 'DAS245'};

reergymerej · Accepted Answer · 2014-02-18 17:02:26Z

1

If you can use an object from the get go, that would be best (see @Travesty3 's answer). If you are stuck with your array, do something like this. You'll incur cost of indexing the array's objects only once rather than each time you want to find something.

var serialNos = [
  { id: 12, serial: 'DAS245' },
  { id: 13, serial: 'DAS246' },
  { id: 13, cereal: 'DAS249' },
  { id: 15, serial: 'DAS247' }
];

var moBetter = indexByKey(serialNos, 'serial');
console.log(moBetter.DAS246);

/**
* Given an array of objects, each with a unique attribute,
* this will return an object, indexed by those keys.
* @param {Object[]} arrOfObjects
* @param {String} key The unique identifier inside each object.
* @return {Object}
*/
function indexByKey (arrOfObjects, key) {
  var index = {},
    i = 0,
    max = arrOfObjects.length;

  // iterate through the array
  for (; i < max; i++) {

    // check to see if the object has the key
    if (arrOfObjects[i].hasOwnProperty(key)) {

      // make sure we haven't already indexed this key
      if (index.hasOwnProperty(arrOfObjects[i][key])) {
        console.error('Hey, wait.  These aren\'t unique!');
      } else {

        // If all is well, index this object by the key.
        index[arrOfObjects[i][key]] = arrOfObjects[i];
      }

    } else {
      console.error('This does not have the key, so it cannot be indexed.', arrOfObjects[i]);
    }
  }

  return index;
}

edited Feb 18, 2014 at 17:02

answered Feb 14, 2014 at 21:46

reergymerej

2,4172 gold badges27 silver badges32 bronze badges

4 Comments

Travesty3 Over a year ago

My answer isn't suggesting to turn the array into an object every time you want to find something. I was suggesting to use an object instead of an array from the beginning. So you would never create an array at all.

reergymerej Over a year ago

@Travesty3, sorry for the implication that you suggested to convert to an array repeatedly. That's not what I meant. I agree that using an object is ideal and this shows how he can convert it.

Travesty3 Over a year ago

That's just the thing. This is suggesting to take the collection as an array, and convert it to an object. To clarify, I would suggest to never create an array. Build it as an object from the start, when you initially populate the collection. Side note, please don't take this in an attacking tone. You've got the checkmark, so I guess your answer is more of what the OP was looking for.

reergymerej Over a year ago

@Travesty3 I hear you. I upvoted your answer before I wrote mine. :)

Collectives™ on Stack Overflow

Removing elements from multi dimensional array using value in Javascript

2 Answers 2

4 Comments

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related