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.

Sign up for GitHub

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

Jump to bottom

[W4.6b][W15-A2]Ho Bing Xuan #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[W4.6b][W15-A2]Ho Bing Xuan #26

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
118 changes: 58 additions & 60 deletions src/seedu/addressbook/AddressBook.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,14 +14,7 @@
import java.nio.file.InvalidPathException;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
import java.util.Collections;
import java.util.HashSet;
import java.util.Optional;
import java.util.Scanner;
import java.util.Set;
import java.util.*;

/*
* NOTE : =============================================================
Expand Down Expand Up @@ -160,6 +153,12 @@ public class AddressBook {
*/
private static final char INPUT_COMMENT_MARKER = '#';

private static final String NAME = "name";
private static final String PHONE = "phone";
private static final String EMAIL = "email";

private enum PersonInfo {NAME, PHONE, EMAIL};

/*
* This variable is declared for the whole class (instead of declaring it
* inside the readUserCommand() method to facilitate automated testing using
Expand All @@ -181,14 +180,14 @@ public class AddressBook {
/**
* List of all persons in the address book.
*/
private static final ArrayList<String[]> ALL_PERSONS = new ArrayList<>();
private static final ArrayList<HashMap> ALL_PERSONS = new ArrayList<>();

/**
* Stores the most recent list of persons shown to the user as a result of a user command.
* This is a subset of the full list. Deleting persons in the pull list does not delete
* those persons from this list.
*/
private static ArrayList<String[]> latestPersonListingView = getAllPersonsInAddressBook(); // initial view is of all
private static ArrayList<HashMap> latestPersonListingView = getAllPersonsInAddressBook(); // initial view is of all

/**
* The path to the file used for storing person data.
Expand Down Expand Up @@ -417,15 +416,15 @@ private static String getMessageForInvalidCommandInput(String userCommand, Strin
*/
private static String executeAddPerson(String commandArgs) {
// try decoding a person from the raw args
final Optional<String[]> decodeResult = decodePersonFromString(commandArgs);
final Optional<HashMap> decodeResult = decodePersonFromString(commandArgs);

// checks if args are valid (decode result will not be present if the person is invalid)
if (!decodeResult.isPresent()) {
return getMessageForInvalidCommandInput(COMMAND_ADD_WORD, getUsageInfoForAddCommand());
}

// add the person as specified
final String[] personToAdd = decodeResult.get();
final HashMap personToAdd = decodeResult.get();
addPersonToAddressBook(personToAdd);
return getMessageForSuccessfulAddPerson(personToAdd);
}
Expand All @@ -437,7 +436,7 @@ private static String executeAddPerson(String commandArgs) {
* @param addedPerson person who was successfully added
* @return successful add person feedback message
*/
private static String getMessageForSuccessfulAddPerson(String[] addedPerson) {
private static String getMessageForSuccessfulAddPerson(HashMap addedPerson) {
return String.format(MESSAGE_ADDED,
getNameFromPerson(addedPerson), getPhoneFromPerson(addedPerson), getEmailFromPerson(addedPerson));
}
Expand All @@ -451,7 +450,7 @@ private static String getMessageForSuccessfulAddPerson(String[] addedPerson) {
*/
private static String executeFindPersons(String commandArgs) {
final Set<String> keywords = extractKeywordsFromFindPersonArgs(commandArgs);
final ArrayList<String[]> personsFound = getPersonsWithNameContainingAnyKeyword(keywords);
final ArrayList<HashMap> personsFound = getPersonsWithNameContainingAnyKeyword(keywords);
showToUser(personsFound);
return getMessageForPersonsDisplayedSummary(personsFound);
}
Expand All @@ -462,7 +461,7 @@ private static String executeFindPersons(String commandArgs) {
* @param personsDisplayed used to generate summary
* @return summary message for persons displayed
*/
private static String getMessageForPersonsDisplayedSummary(ArrayList<String[]> personsDisplayed) {
private static String getMessageForPersonsDisplayedSummary(ArrayList<HashMap> personsDisplayed) {
return String.format(MESSAGE_PERSONS_FOUND_OVERVIEW, personsDisplayed.size());
}

Expand All @@ -482,9 +481,9 @@ private static Set<String> extractKeywordsFromFindPersonArgs(String findPersonCo
* @param keywords for searching
* @return list of persons in full model with name containing some of the keywords
*/
private static ArrayList<String[]> getPersonsWithNameContainingAnyKeyword(Collection<String> keywords) {
final ArrayList<String[]> matchedPersons = new ArrayList<>();
for (String[] person : getAllPersonsInAddressBook()) {
private static ArrayList<HashMap> getPersonsWithNameContainingAnyKeyword(Collection<String> keywords) {
final ArrayList<HashMap> matchedPersons = new ArrayList<>();
for (HashMap person : getAllPersonsInAddressBook()) {
final Set<String> wordsInName = new HashSet<>(splitByWhitespace(getNameFromPerson(person)));
if (!Collections.disjoint(wordsInName, keywords)) {
matchedPersons.add(person);
Expand All @@ -507,7 +506,7 @@ private static String executeDeletePerson(String commandArgs) {
if (!isDisplayIndexValidForLastPersonListingView(targetVisibleIndex)) {
return MESSAGE_INVALID_PERSON_DISPLAYED_INDEX;
}
final String[] targetInModel = getPersonByLastVisibleIndex(targetVisibleIndex);
final HashMap targetInModel = getPersonByLastVisibleIndex(targetVisibleIndex);
return deletePersonFromAddressBook(targetInModel) ? getMessageForSuccessfulDelete(targetInModel) // success
: MESSAGE_PERSON_NOT_IN_ADDRESSBOOK; // not found
}
Expand Down Expand Up @@ -554,7 +553,7 @@ private static boolean isDisplayIndexValidForLastPersonListingView(int index) {
* @param deletedPerson successfully deleted
* @return successful delete person feedback message
*/
private static String getMessageForSuccessfulDelete(String[] deletedPerson) {
private static String getMessageForSuccessfulDelete(HashMap deletedPerson) {
return String.format(MESSAGE_DELETE_PERSON_SUCCESS, getMessageForFormattedPersonData(deletedPerson));
}

Expand All @@ -574,7 +573,7 @@ private static String executeClearAddressBook() {
* @return feedback display message for the operation result
*/
private static String executeListAllPersonsInAddressBook() {
ArrayList<String[]> toBeDisplayed = getAllPersonsInAddressBook();
ArrayList<HashMap> toBeDisplayed = getAllPersonsInAddressBook();
showToUser(toBeDisplayed);
return getMessageForPersonsDisplayedSummary(toBeDisplayed);
}
Expand Down Expand Up @@ -629,7 +628,7 @@ private static void showToUser(String... message) {
* The list will be indexed, starting from 1.
*
*/
private static void showToUser(ArrayList<String[]> persons) {
private static void showToUser(ArrayList<HashMap> persons) {
String listAsString = getDisplayString(persons);
showToUser(listAsString);
updateLatestViewedPersonListing(persons);
Expand All @@ -638,10 +637,10 @@ private static void showToUser(ArrayList<String[]> persons) {
/**
* Returns the display string representation of the list of persons.
*/
private static String getDisplayString(ArrayList<String[]> persons) {
private static String getDisplayString(ArrayList<HashMap> persons) {
final StringBuilder messageAccumulator = new StringBuilder();
for (int i = 0; i < persons.size(); i++) {
final String[] person = persons.get(i);
final HashMap person = persons.get(i);
final int displayIndex = i + DISPLAYED_INDEX_OFFSET;
messageAccumulator.append('\t')
.append(getIndexedPersonListElementMessage(displayIndex, person))
Expand All @@ -657,7 +656,7 @@ private static String getDisplayString(ArrayList<String[]> persons) {
* @param person to show
* @return formatted listing message with index
*/
private static String getIndexedPersonListElementMessage(int visibleIndex, String[] person) {
private static String getIndexedPersonListElementMessage(int visibleIndex, HashMap person) {
return String.format(MESSAGE_DISPLAY_LIST_ELEMENT_INDEX, visibleIndex) + getMessageForFormattedPersonData(person);
}

Expand All @@ -667,7 +666,7 @@ private static String getIndexedPersonListElementMessage(int visibleIndex, Strin
* @param person to show
* @return formatted message showing internal state
*/
private static String getMessageForFormattedPersonData(String[] person) {
private static String getMessageForFormattedPersonData(HashMap person) {
return String.format(MESSAGE_DISPLAY_PERSON_DATA,
getNameFromPerson(person), getPhoneFromPerson(person), getEmailFromPerson(person));
}
Expand All @@ -677,7 +676,7 @@ private static String getMessageForFormattedPersonData(String[] person) {
*
* @param newListing the new listing of persons
*/
private static void updateLatestViewedPersonListing(ArrayList<String[]> newListing) {
private static void updateLatestViewedPersonListing(ArrayList<HashMap> newListing) {
// clone to insulate from future changes to arg list
latestPersonListingView = new ArrayList<>(newListing);
}
Expand All @@ -688,7 +687,7 @@ private static void updateLatestViewedPersonListing(ArrayList<String[]> newListi
* @param lastVisibleIndex displayed index from last shown person listing
* @return the actual person object in the last shown person listing
*/
private static String[] getPersonByLastVisibleIndex(int lastVisibleIndex) {
private static HashMap getPersonByLastVisibleIndex(int lastVisibleIndex) {
return latestPersonListingView.get(lastVisibleIndex - DISPLAYED_INDEX_OFFSET);
}

Expand Down Expand Up @@ -728,8 +727,8 @@ private static void createFileIfMissing(String filePath) {
* @param filePath file to load from
* @return the list of decoded persons
*/
private static ArrayList<String[]> loadPersonsFromFile(String filePath) {
final Optional<ArrayList<String[]>> successfullyDecoded = decodePersonsFromStrings(getLinesInFile(filePath));
private static ArrayList<HashMap> loadPersonsFromFile(String filePath) {
final Optional<ArrayList<HashMap>> successfullyDecoded = decodePersonsFromStrings(getLinesInFile(filePath));
if (!successfullyDecoded.isPresent()) {
showToUser(MESSAGE_INVALID_STORAGE_FILE_CONTENT);
exitProgram();
Expand Down Expand Up @@ -760,7 +759,7 @@ private static ArrayList<String> getLinesInFile(String filePath) {
*
* @param filePath file for saving
*/
private static void savePersonsToFile(ArrayList<String[]> persons, String filePath) {
private static void savePersonsToFile(ArrayList<HashMap> persons, String filePath) {
final ArrayList<String> linesToWrite = encodePersonsToStrings(persons);
try {
Files.write(Paths.get(storageFilePath), linesToWrite);
Expand All @@ -782,7 +781,7 @@ private static void savePersonsToFile(ArrayList<String[]> persons, String filePa
*
* @param person to add
*/
private static void addPersonToAddressBook(String[] person) {
private static void addPersonToAddressBook(HashMap person) {
ALL_PERSONS.add(person);
savePersonsToFile(getAllPersonsInAddressBook(), storageFilePath);
}
Expand All @@ -793,7 +792,7 @@ private static void addPersonToAddressBook(String[] person) {
* @param exactPerson the actual person inside the address book (exactPerson == the person to delete in the full list)
* @return true if the given person was found and deleted in the model
*/
private static boolean deletePersonFromAddressBook(String[] exactPerson) {
private static boolean deletePersonFromAddressBook(HashMap exactPerson) {
final boolean changed = ALL_PERSONS.remove(exactPerson);
if (changed) {
savePersonsToFile(getAllPersonsInAddressBook(), storageFilePath);
Expand All @@ -804,7 +803,7 @@ private static boolean deletePersonFromAddressBook(String[] exactPerson) {
/**
* Returns all persons in the address book
*/
private static ArrayList<String[]> getAllPersonsInAddressBook() {
private static ArrayList<HashMap> getAllPersonsInAddressBook() {
return ALL_PERSONS;
}

Expand All @@ -821,7 +820,7 @@ private static void clearAddressBook() {
*
* @param persons list of persons to initialise the model with
*/
private static void initialiseAddressBookModel(ArrayList<String[]> persons) {
private static void initialiseAddressBookModel(ArrayList<HashMap> persons) {
ALL_PERSONS.clear();
ALL_PERSONS.addAll(persons);
}
Expand All @@ -838,26 +837,26 @@ private static void initialiseAddressBookModel(ArrayList<String[]> persons) {
*
* @param person whose name you want
*/
private static String getNameFromPerson(String[] person) {
return person[PERSON_DATA_INDEX_NAME];
private static String getNameFromPerson(HashMap<PersonInfo, String> person) {
return person.get(PersonInfo.NAME);
}

/**
* Returns given person's phone number
*
* @param person whose phone number you want
*/
private static String getPhoneFromPerson(String[] person) {
return person[PERSON_DATA_INDEX_PHONE];
private static String getPhoneFromPerson(HashMap person) {
return (String) person.get(PersonInfo.PHONE);
}

/**
* Returns given person's email
*
* @param person whose email you want
*/
private static String getEmailFromPerson(String[] person) {
return person[PERSON_DATA_INDEX_EMAIL];
private static String getEmailFromPerson(HashMap person) {
return (String) person.get(PersonInfo.EMAIL);
}

/**
Expand All @@ -868,11 +867,11 @@ private static String getEmailFromPerson(String[] person) {
* @param email without data prefix
* @return constructed person
*/
private static String[] makePersonFromData(String name, String phone, String email) {
final String[] person = new String[PERSON_DATA_COUNT];
person[PERSON_DATA_INDEX_NAME] = name;
person[PERSON_DATA_INDEX_PHONE] = phone;
person[PERSON_DATA_INDEX_EMAIL] = email;
private static HashMap<PersonInfo, String> makePersonFromData(String name, String phone, String email) {
final HashMap<PersonInfo, String> person = new HashMap<>();
person.put(PersonInfo.NAME, name);
person.put(PersonInfo.EMAIL, email);
person.put(PersonInfo.PHONE, phone);
return person;
}

Expand All @@ -882,7 +881,7 @@ private static String[] makePersonFromData(String name, String phone, String ema
* @param person to be encoded
* @return encoded string
*/
private static String encodePersonToString(String[] person) {
private static String encodePersonToString(HashMap person) {
return String.format(PERSON_STRING_REPRESENTATION,
getNameFromPerson(person), getPhoneFromPerson(person), getEmailFromPerson(person));
}
Expand All @@ -893,11 +892,10 @@ private static String encodePersonToString(String[] person) {
* @param persons to be encoded
* @return encoded strings
*/
private static ArrayList<String> encodePersonsToStrings(ArrayList<String[]> persons) {
private static ArrayList<String> encodePersonsToStrings(ArrayList<HashMap> persons) {
final ArrayList<String> encoded = new ArrayList<>();
for (String[] person : persons) {
encoded.add(encodePersonToString(person));
}
for(HashMap h: persons)
encoded.add(encodePersonToString(h));
return encoded;
}

Expand All @@ -915,12 +913,12 @@ private static ArrayList<String> encodePersonsToStrings(ArrayList<String[]> pers
* @return if cannot decode: empty Optional
* else: Optional containing decoded person
*/
private static Optional<String[]> decodePersonFromString(String encoded) {
private static Optional<HashMap> decodePersonFromString(String encoded) {
// check that we can extract the parts of a person from the encoded string
if (!isPersonDataExtractableFrom(encoded)) {
return Optional.empty();
}
final String[] decodedPerson = makePersonFromData(
final HashMap<PersonInfo, String> decodedPerson = makePersonFromData(
extractNameFromPersonString(encoded),
extractPhoneFromPersonString(encoded),
extractEmailFromPersonString(encoded)
Expand All @@ -936,10 +934,10 @@ private static Optional<String[]> decodePersonFromString(String encoded) {
* @return if cannot decode any: empty Optional
* else: Optional containing decoded persons
*/
private static Optional<ArrayList<String[]>> decodePersonsFromStrings(ArrayList<String> encodedPersons) {
final ArrayList<String[]> decodedPersons = new ArrayList<>();
private static Optional<ArrayList<HashMap>> decodePersonsFromStrings(ArrayList<String> encodedPersons) {
final ArrayList<HashMap> decodedPersons = new ArrayList<>();
for (String encodedPerson : encodedPersons) {
final Optional<String[]> decodedPerson = decodePersonFromString(encodedPerson);
final Optional<HashMap> decodedPerson = decodePersonFromString(encodedPerson);
if (!decodedPerson.isPresent()) {
return Optional.empty();
}
Expand Down Expand Up @@ -1028,10 +1026,10 @@ private static String extractEmailFromPersonString(String encoded) {
*
* @param person String array representing the person (used in internal data)
*/
private static boolean isPersonDataValid(String[] person) {
return isPersonNameValid(person[PERSON_DATA_INDEX_NAME])
&& isPersonPhoneValid(person[PERSON_DATA_INDEX_PHONE])
&& isPersonEmailValid(person[PERSON_DATA_INDEX_EMAIL]);
private static boolean isPersonDataValid(HashMap<PersonInfo, String> person) {
return isPersonNameValid(person.get(PersonInfo.NAME))
&& isPersonPhoneValid(person.get(PersonInfo.PHONE))
&& isPersonEmailValid(person.get(PersonInfo.EMAIL));
}

/*
Expand Down