Update SkipList. Fixed Issues #137, #138, #139, #140 #147

Gutsonok · 2020-08-05T16:01:32Z

Fixed #137 . This bug occurs when _getNextLevel() method returned 0.

Fixed #138 and #139 . This bug occurs because of skip list was сircular.

Fixed #140. This bug occurs comparing method was called for null object.

I add implementaion for public T this[T item]. I think this should be work like a standard Dictionary type.

I divided a unit test for SkipList on some tests. And added additinal tests.

An issues were first created before opening this pull request
The new code follows the contribution guidelines
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests to ensure that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

This reverts commit 9cab153.

…ring>

Gutsonok added 4 commits August 5, 2020 20:51

Fixed Issue aalhour#137 Items are not always added to a SkipList

f769be0

Fixed Issue aalhour#139 A new SkipList<int> contains 0

9cab153

Revert "Fixed Issue aalhour#139 A new SkipList<int> contains 0"

a9ce311

This reverts commit 9cab153.

Fix aalhour#138 and aalhour#139 issues. Update skipList logic

377a8d1

Gutsonok changed the title ~~Fixed Issue #137 Items are not always added to a SkipList~~ Update SkipList. Fixed Issues #137, #138, #139 Aug 8, 2020

Fix aalhour#140 Program crashes upon search for null in a SkipList<st…

321e838

…ring>

Gutsonok changed the title ~~Update SkipList. Fixed Issues #137, #138, #139~~ Update SkipList. Fixed Issues #137, #138, #139, #140 Aug 8, 2020

aalhour self-requested a review December 22, 2020 10:10

Provide feedback