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

HW04 - ArrayListGen #2

Closed
chulderman opened this issue Mar 15, 2016 · 5 comments
Closed

HW04 - ArrayListGen #2

chulderman opened this issue Mar 15, 2016 · 5 comments

Comments

@chulderman
Copy link
Owner

This class is responsible for generating an ArrayList of Integers; it will be filled by the int[] array generated by the ArrayGen.java class.

@chulderman
Copy link
Owner Author

Use something like this to create an iterator for your ArrayListGen.

Make sure that it conforms to this usage.

chulderman added a commit that referenced this issue Mar 18, 2016
Merging to master; Waiting for Issue #2 to resolve; Updates Issue #4 LinkedListGen
@ghost
Copy link

ghost commented Mar 19, 2016

ArrayListGen is complete, I will close this issue.

@ghost ghost closed this as completed Mar 19, 2016
@ghost ghost reopened this Mar 22, 2016
@ghost
Copy link

ghost commented Mar 22, 2016

This class is now finished and works properly so I will close this issue,

@ghost ghost closed this as completed Mar 22, 2016
@chulderman
Copy link
Owner Author

populateArrayList should return void IMHO. Access to the ArrayListGen should be explicit through an accessor method.

@chulderman
Copy link
Owner Author

Rather than look at the index and do a comparison for hasNext() maybe we should be doing a try catch to determine if there is an Integer at that index. If it doesn't throw an exception return true.

https://github.com/chulderman/punctualPears/blob/master/source/edu/oakland/helper/ArrayListGen.java#L140

chulderman added a commit that referenced this issue Mar 23, 2016
This issue was closed.
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

No branches or pull requests

1 participant