'How to overload the Set Class
How would I overload the Set.prototype.add
method, for example I'm trying to do:
// allow overloading the Set().method function by passing multiple arguments
Set.prototype.add = function(x) {
for (let x of arguments) {
super.add(x); // doesn't Set act like a class?
}
return this;
}
let s = new Set([1,2,3]);
s.add(4,5,6);
console.log("Set: ", s);
// 'super' keyword unexpected here
Or do I need to alias it like this so I can call the parent via Function.call/apply
, such as:
Set.prototype.addMultiple = function(x) {
for (let x of arguments) {
Set.prototype.add.call(this, x);
}
return this;
}
let s = new Set([1,2,3]);
s.addMultiple(4,5,6);
console.log("Set: {" + Array.from(s) + "}");
Or, what about just aliasing the original method, is that considered ok? (or does Set.prototype._add = Set.prototype.add;
have unintended side effects?):
Set.prototype._add = Set.prototype.add;
Set.prototype.add = function(x) {
for (let x of arguments) {
Set.prototype._add.call(this, x);
}
return this;
}
let s = new Set([1,2,3]);
s.add(4,5,6);
console.log(`Set: {${Array.from(s)}}`);
Solution 1:[1]
You don't need all of these fancy features; just this
:
Set.prototype.addMultiple = function(x) {
for (let x of arguments) {
this.add(x);
}
return this;
}
While we're at it, let's use const
instead in the loop:
for (const x of arguments) {
And also arguments
? That's some 2010 code. Nowadays we use spread/variadic arguments:
Set.prototype.addMultiple = function (...args) {
for (const x of args) { ... }
// rest omitted
}
Keep in mind that extending/modifying native prototypes is generally frowned upon, so you're better off extending the class instead.
Solution 2:[2]
You can subclass the Set
with something like the following:
class _Set extends Set {
add(_) {
Array.from(arguments).map((x) => super.add(x));
return this;
}
}
let s = new _Set([1,2,3]);
s.add(4,5,6);
console.log(`Set: ${Array.from(s)}`);
Sources
This article follows the attribution requirements of Stack Overflow and is licensed under CC BY-SA 3.0.
Source: Stack Overflow
Solution | Source |
---|---|
Solution 1 | |
Solution 2 | David542 |