[W5.11][T12-1] Ho Phi Long #167
Conversation
Checks index and shows based on index
[James] Add edit command
Commented out first because the .bat file does not work with the login function
Gerald's branch
|
Hi @Tertium3, your pull request title is invalid. For PR sent to satisfy a learning outcome, the PR name should be in the format of For team PR, the PR name should be in the format of Please follow the above format strictly and edit your title for reprocessing. Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at nus-se-pr-bot and add a link to this PR. |
| public SortCommand() {} | ||
| public List<ReadOnlyPerson> SortList() { | ||
| //Copy the unmodifiable array to a list | ||
| List<ReadOnlyPerson> SortedList = new ArrayList<ReadOnlyPerson>(addressBook.getAllPersons().immutableListView()); |
There was a problem hiding this comment.
Check if this line crosses the 120 characters hard limit. If yes, consider breaking it into two lines.
| import java.util.ArrayList; | ||
| import java.util.Collections; | ||
| import java.util.List; | ||
|
|
There was a problem hiding this comment.
Consider adding comments for this class, clarifying what kind of "sorting" this class is supposed to do.
Sort by name