Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding readme file explaining cocktailshakersort adding cocktailshake… #1119

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mohmmadAyesh
Copy link

Hi I added CocktailShakerSort Algorithm its like Bubble Sort Algorithm but it can be bidirectional in some cases it can achieve better than BubbleSort with less time Complexity on average and worst case its O(n^2) on best case its O(n) it take constant amount from Memory so its memory complexity is O(1) I Wrote A java script file for it following your pattern in writing these algorithm like extending class from Sort,etc Also make a test file CocktailShake.test.js to test it Also I added a Readme file that explaining it with references for further information Please Accept my PR.

…rsort.js contain algorithm code adding cocktailshakersort.test.js contain test case for this algorithm i ran npm run test and it work no failed tests
@mohmmadAyesh
Copy link
Author

@trekhleb is that PR awaiting review or just I need to fix this test to pass please help thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant