addressbook-level2
addressbook-level2 copied to clipboard
se-edu
:ab::two: A Java sample application for students. An AddressBook application that uses OOP basics.
Fixes #347 This PR removes a redundant ; from the Command class.
`Arrays.asList()` can (and should) be used for creating a `List`, thus making `TestUtils.createList()` redundant.
There is an unnecessary `;` at the end of `execute()` function in Line 44 Noticed this while reviewing PR's for CS2103 https://github.com/se-edu/addressbook-level2/blob/23644b1848abac9a973179ad015e20b7292394f5/src/seedu/addressbook/commands/Command.java#L42-L44
Unit tests do not confirm that a person is added correctly in `AddressBook#addPerson()`.
Following up on https://github.com/nus-cs2103-AY1718S1/forum/issues/32. Since auto-stripping trailing spaces is a very useful feature (especially when we reach level4 and have to deal with checkstyle), perhaps a better solution would be...
Interfaces is an advanced OOP concept. It would be good if level2 refrain from using interfaces.
.txt is a legacy requirement from level 1. This address book saves data in xml format, so the file should have the extension .xml
**Prerequisite:** https://github.com/se-edu/addressbook-level4/pull/279 (further study about changes in checkstyle.xml is required) Some module names in checkstyle.xml have the `Check` suffix. For example, ``. We should change them to be ``, as...
Many of the recent contributors are not listed in the contributor list
Parts of address (block, etc.) can be nested classes of `Address` class.