Sitelet https://web.archive.org/web/20201103184416/https://github.com/TheAlgorithms/Java/pull/1396
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

Add javadoc comments and Cleanup DynamicArray.java #1396

Merged
merged 2 commits into from Aug 13, 2020

Conversation

@rbshealy
Copy link
Contributor

@rbshealy rbshealy commented Aug 7, 2020 •

Javadoc comments allow the user to understand the purpose of each method within the DynamicArray class.

cleanup within the class covers:
Condense imports
Initialize fields in constructor
Change binary shifting to simple multiplication reassignment for better understanding
Add brackets to multi-line if statements
Change name of size() method to getSize() to reflect purpose of the method
remove extra white-space
remove 2 lines of unnecessary commented out code

closes #1400

rbshealy added 2 commits Aug 7, 2020
@rbshealy
Copy link
Contributor Author

@rbshealy rbshealy commented Aug 12, 2020

@StepfenShawn Could you kindly review this?

@StepfenShawn StepfenShawn merged commit ab88fed into TheAlgorithms:master Aug 13, 2020
@StepfenShawn
Copy link
Member

@StepfenShawn StepfenShawn commented Aug 13, 2020

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.