-
Notifications
You must be signed in to change notification settings - Fork 27
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
Attempted to complete DoublyLinkedList #29
Open
ghost
wants to merge
16
commits into
GreenRiverCollege-SDEV333:main
Choose a base branch
from
JEller98:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… removeBack() and addBack() methods.
…t.java and LinkedIntList.java. Also implemented the removeFront() and removeBack() methods for ArrayIntList.java.
…tains(), indexOf(), isEmpty(), and size().
… not functioning the way they should. Also attempted to re-implement the add() method, as for some reason it appears that it was never properly put in.
…ould; these include addBack(), add(), removeFront(), removeBack(), remove(), get(), contains(), indexOf(), isEmpty(), size(), and clear(). Code is completely untested, no time to write test cases for any of these.
…ntains(), indexOf(), isEmpty(), size(), clear() and iterator() methods; also touched up addBack() to be more robust.
…op of the file. Also created a series of tests for DoublyLinkedIntList.java, contained in the file DoublyLinkedTests.java. That being said, these tests themselves are completely untested; the test file would not run in my IDE and I'm unsure why.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello there Ken, I just finished writing the methods for DoublyLinkedList, along with test cases for them. That being said, the test file doesn't seem to be runnable currently and I have run out of time to complete this assignment.
The ArrayIntList is finished and has tests as well, LinkedIntList is finished but does not have tests.
ArrayIntList's tests don't all pass and aren't very robust.
I apologize for the sloppy work and will be better applying myself to this class in the coming weeks.