In TreeMap's [Symbol.iterator]:
public [Symbol.iterator] ():Iterator<K> { return this.entrySet[Symbol.iterator](); }

The type of this.entrySet is a function, not an object of ImmutableEntrySetForTreeMap. So I think it should be defined like this:

public [Symbol.iterator]():Iterator<MapEntry<K, V>> { return this.entrySet()[Symbol.iterator](); }

Or there will get a ts(7053) error

I created a CompositeCollection class, and I'd like to see a CompositeList and a CompositeMap class as well. I'm looking for a contributor to develop these classes

1. Queue and PriorityQueue
2. SkipListMap and SkipListSet
3. More documentation and examples

and tag code when that is done

MultiMaps and MultiSets are the next implementable piece once the core Set and Map implementations are completed

Okay, this is a new feature and partially a rant.

Have you ever passed in a map or a set or something into a method and when the method is done your map has been messed with and it causes a bug in your application? And you're justing staring incredulously at your screen wondering why somebody felt it necessary to mess with your map just because you passed it to a method? Causing you a bug that you had to hunt down because somebody felt it was okay to unannounced mess with your collection?

Yeah, me too.

Thus the ImmutableSet and ImmutableMap were born. I couldn't add these to the Java collections framework because Set and Map extend them, but now I can :)

Now you can define a method as taking in an ImmutableSet that only shows the read only methods and know confidently that when you call that method your Set or Map will not be cleared out by the method.

Okay, it doesnt solve the problem of somebody changing state on the objects in the Set or Map, but it's an improvement and I'll take it :)

For some odd reason, Jasmine 3.x.x causes random test failures. I rolled back the code to 2.9.x and will take a look. I would greatly appreciate the help of some very smart person who would love to become an open source contributor :)

The devDependency @types/jasmine was updated from 3.3.16 to 3.4.0.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

@types/jasmine is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Your Greenkeeper Bot 🌴

Need to create some test cases too :)

Version 10 of Node.js (code name Dubnium) has been released! 🎊

To see what happens to your code in Node.js 10, Greenkeeper has created a branch with the following changes:

• Added the new Node.js version to your .travis.yml

If you’re interested in upgrading this repo to Node.js 10, you can open a PR with these changes. Please note that this issue is just intended as a friendly reminder and the PR as a possible starting point for getting your code running on Node.js 10.

Your Greenkeeper Bot 🌴

Queue and Deque interfaces and implementations for each

If you've got Node 10, you're been made very sad if you try to build this framework - it won't work.

I've never been a great fan of gulp, it works but I never got it building without deploying the tests and putting the test code into the code coverage tool.

Or should I say it worked until I moved to Node 10. Now it makes me sad. Gulp 4 has been in development for 2 years and doesnt appear that it will ever complete.

Please replace the build process with something that works on both macs and pcs and works on node 10. Gulp not required.

Priority Queue - done
Ease of Use for the entire framework

They are defining their own Map and Set classes, so I'm going to rename in advance to prevent problems later.

The ones in this library will become JMap and JSet.

I will also add some of their methods into JMap and JSet so JS & TS developers can use these classes and feel at home too.

Reminder to myself to get this done

A very talented volunteer reached out to me and volunteered to write the Queue interface and the PriorityQueue class.

I've opened this issue to make sure he gets appropriately thanked and very clearly recognized as being a great programmer.

Possibly a version 5 or 6 deliverable.

This class would combine the Navigable features of a SkipListMap/Set with the raw get performance of a HashMap to make a class that the Java Collections Framework never had.

Something that does a good long explanation of what we're all about here with demos for each class and some example code for people to lift.

Have you considered porting the EvictingQueue class from Guava?

This is a request for any would be contributors to improve my build process.
It works, but isn't perfect.
It packages the testing code in with the distributions.
As a side effect, my code coverage numbers are off since it's examining test code as well as source code.

Thank you in advance for fixing the build process!

I am looking for somebody to port the LinkedHashSet class from the Java Collections Framework into this project.

https://docs.oracle.com/javase/8/docs/api/java/util/LinkedHashSet.html

In TreeSet.ts:

/**

• Adds the specified element to this set if it is not already present.

• @param {K} element element to be added to this set

• @return {boolean} true if this set did not already contain the specified element
  */
  public add (element:K) : boolean {
  const tmp:number = this.datastore.put(element, 1);
  if (tmp === null) {
  return false;
  }

  return true;
  }

“true if this set did not already contain the specified element”

This add() function always returns false when the element is added successfully except when it returns number 1 if the element existed in the map and the older element replaced which means add() function returns true in this case.

Currently the collections do their default behavior which is not helpful for debugging

Hi, first of all, I would like to thank you for your great effort to bring together such a nice collection.

However, I am trying to use TreeMap as follows:

public filterMap: TreeMap<FieldDescription, LinkedList<FieldFilter>> = new TreeMap<FieldDescription, LinkedList<FieldFilter>>(???);

but I don't know what to pass in instead of ???

I looked at the demo, since they were both strings, you have passed in: Collections.getStringComparator()

but that gives an error in my case since they are custom classes.

see test coverage for HashMap:

expect (sourceMap.put ("A", "B")).toEqual(undefined);

for HashSet:

expect (set2.add (product1)).toEqual (true);

should HashMap return true on successfully adding a new Map key/value pair?

I am looking for somebody to port the LinkedHashSet and LinkedHashMap classes from the Java Collections Framework into this project.

https://docs.oracle.com/javase/8/docs/api/java/util/LinkedHashSet.html
https://docs.oracle.com/javase/8/docs/api/java/util/LinkedHashMap.html

This is a great contribution request for a creative person with good graphics skills.

This project needs a logo.

Similar to TestMap.spec.ts and TestSet.spec.ts but written to ensure consistency of Navigable behavior across implementations

Okay, I realize that as I write this the framework has 90% code coverage, and I'm asking for contributors to add more unit tests. I realize that most people would be beyond glad with 90% code coverage. This framework is intended to be embedded into many many other projects, and I would appreciate any pull requests that would capture additional conditions not currently tested.

Unfortunately, tests just show that a line of code was tested, they don't prove that the code does what you really wanted it to do in the first place.

Any assistance would be greatly appreciated.