Code

[W2.4b][W3.3][W09-A4] Charles Goh#12

Code

Open

charlesgohck wants to merge 25 commits intonus-cs2103-AY1718S1:masternus-cs2103-AY1718S1/addressbook-level1:masterfrom charlesgohck:masterCopy head branch name to clipboard

+187-125

Commits

Commits on Aug 23, 2017

• [DS] Changed String references for person to Strings rather than integers

  

  Charles Goh

  authored and

  Charles Goh

  committed
  d9e2984

• [PERSON][DS] Changed person from String Array to HashMap <String, String>

  

  Charles Goh

  authored and

  Charles Goh

  committed
  3bcf337

• [PERSON][DS] Replaced String[] person paramters with HashMap equivalent

  

  Charles Goh

  authored and

  Charles Goh

  committed
  beb1006

• [PERSON][DS] Changed stray String[] person to HashMap equivalent

  

  Charles Goh

  authored and

  Charles Goh

  committed
  03d811c

• [INDEX][HASH] Changed index to hash property, i.e. DATA_INDEX to PROPERTY

  

  Charles Goh

  authored and

  Charles Goh

  committed
  6be9c03

• [ALL_PERSONS] Edited ALL_PERSONS generic type to accept HashMap

  

  Charles Goh

  authored and

  Charles Goh

  committed
  c9f4935

• [DEBUG] Changed DS of addedPerson and deletedPerson to accept HashMap param

  

  Charles Goh

  authored and

  Charles Goh

  committed
  4258f8f

• [FIX][ARRAYLIST][HASHMAP] Changed Arraylist to contain HashMaps intead of String[]s

  

  Charles Goh

  authored and

  Charles Goh

  committed
  13901c6

• [FIX] Fixed data entry to HashMap for makePersonFromData function

  

  Charles Goh

  authored and

  Charles Goh

  committed
  35037f2

• [FIX] Fixed data retrieval from HashMap in isPersonDataValid function

  

  Charles Goh

  authored and

  Charles Goh

  committed
  69516d7

• [HASHMAP][GET][FIX] Changed person info getters for HashMap

  

  Charles Goh

  authored and

  Charles Goh

  committed
  78653f6

• [IMPORTS] Imported all of Java Util instead of individual calls

  

  Charles Goh

  authored and

  Charles Goh

  committed
  6522a14

• [FIX][DS] Replaced more ArrayList person calls with HashMap equivalent

  

  Charles Goh

  authored and

  Charles Goh

  committed
  c3c5529

• [DEMO][VARARGS] Demoed non varargs showToUser version (commented out)

  

  Charles Goh

  authored and

  Charles Goh

  committed
  8d4ef1b

Commits on Aug 27, 2017

• [REFACTOR] Reversed conditional for two functions (loadPersonsFromFile and decodePersonsFromStrings)

  

  Charles Goh

  authored and

  Charles Goh

  committed
  ae52457

• [REFACTOR] Rolled back decodePersonsFromStrings and replaced hardcoded boolean statements in isPersonDataExtractableFrom

  

  Charles Goh

  authored and

  Charles Goh

  committed
  53d6fdd

• [INDENTATION] Fixed indentation for switch statement

  

  Charles Goh

  authored and

  Charles Goh

  committed
  d44f01d

Commits on Aug 29, 2017

• [COMMAND][SORT] Set up case and static declarations for sortby command

  

  Charles Goh

  authored and

  Charles Goh

  committed
  6a2be7d

• [COMMAND][SORT] Changed scope setup for sort to name asc and desc only

  

  Charles Goh

  authored and

  Charles Goh

  committed
  8772fae

• [FIX][COMMAND][SORT] Fixed differing static variable

  

  Charles Goh

  authored and

  Charles Goh

  committed
  7687907

• [COMMAND][SORTBYNAME] Implemented Brute Force Sort by Name command

  

  Charles Goh

  authored and

  Charles Goh

  committed
  c8e3030

• [FIX] Fixed accidental deletion of conditional statement that broke runtime

  

  Charles Goh

  authored and

  Charles Goh

  committed
  244667e

• [FIX][COMMAND][SORTBYNAME] Fixed wrong variable that cause CLI to crash

  

  Charles Goh

  authored and

  Charles Goh

  committed
  b2dd079

• [DOCS][COMMAND][SORT] Added documentation for name sort

  

  Charles Goh

  authored and

  Charles Goh

  committed
  d678c97

Commits on Aug 30, 2017

• [FIX][COMMAND][SORT] Fixed COMMAND_SORT_EXAMPLE constant variable

  

  Charles Goh

  committed
  316a8f7