Skip to content

Commit

Permalink
Merge pull request #1044 from Violet-Bora-Lee/patch-6
Browse files Browse the repository at this point in the history
 update description regarding forEach method on Set
  • Loading branch information
iliakan authored Jun 11, 2019
2 parents 1b963bd + f752403 commit e1c0d62
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions 1-js/05-data-types/07-map-set-weakmap-weakset/article.md
Original file line number Diff line number Diff line change
Expand Up @@ -222,9 +222,9 @@ set.forEach((value, valueAgain, set) => {
});
```
Note the funny thing. The `forEach` function in the `Set` has 3 arguments: a value, then *again a value*, and then the target object. Indeed, the same value appears in the arguments twice.
Note the funny thing. The callback function passed in `forEach` has 3 arguments: a value, then *again a value*, and then the target object. Indeed, the same value appears in the arguments twice.
That's for compatibility with `Map` where `forEach` has three arguments. Looks a bit strange, for sure. But may help to replace `Map` with `Set` in certain cases with ease, and vice versa.
That's for compatibility with `Map` where the callback passed `forEach` has three arguments. Looks a bit strange, for sure. But may help to replace `Map` with `Set` in certain cases with ease, and vice versa.
The same methods `Map` has for iterators are also supported:
Expand Down

0 comments on commit e1c0d62

Please sign in to comment.