[W6.2i][F09-4] Rachel Lee

docs/DeveloperGuide.adoc

@@ -47,9 +47,10 @@ Priorities: High (must have) - `* * \*`, Medium (nice to have) - `* \*`, Low (un
 |===========================================================================================================================================
 |Priority |As a ... |I want to ... |So that I can...
 |`* * *` |new user |see usage instructions |refer to instructions when I forget how to use the App
-|`* * *` |user |add a new person |
+|`* * *` |user |add a new person | add details of people
 |`* * *` |user |delete a person |remove entries that I no longer need
 |`* * *` |user |find a person by name |locate details of persons without having to go through the entire list
+|`* * *` |user |edit details of a person |update details or make small changes without having to remove and re-add the person 
 |`* *` |user |hide <<private-contact-detail, private contact details>> by default |minimize chance of someone else seeing them by accident
 |`*` |user with many persons in the address book |sort persons by name |locate a person easily
 |===========================================================================================================================================

src/seedu/addressbook/data/person/Address.java

@@ -6,7 +6,7 @@
  * Represents a Person's address in the address book.
  * Guarantees: immutable; is valid as declared in {@link #isValidAddress(String)}
  */
-public class Address {
+public class Address implements Printable {
 
     public static final String EXAMPLE = "123, some street";
     public static final String MESSAGE_ADDRESS_CONSTRAINTS = "Person addresses can be in any format";
@@ -55,4 +55,9 @@ public int hashCode() {
     public boolean isPrivate() {
         return isPrivate;
     }
+
+    @Override
+    public String getPrintableString() {
+        return "Address: " + value;
+    }
 }

src/seedu/addressbook/data/person/Email.java

@@ -6,7 +6,7 @@
  * Represents a Person's email in the address book.
  * Guarantees: immutable; is valid as declared in {@link #isValidEmail(String)}
  */
-public class Email {
+public class Email implements Printable{
 
     public static final String EXAMPLE = "[email protected]";
     public static final String MESSAGE_EMAIL_CONSTRAINTS =
@@ -58,4 +58,9 @@ public int hashCode() {
     public boolean isPrivate() {
         return isPrivate;
     }
+
+    @Override
+    public String getPrintableString() {
+        return "Email: " + value;
+    }
 }

src/seedu/addressbook/data/person/Name.java

@@ -9,7 +9,7 @@
  * Represents a Person's name in the address book.
  * Guarantees: immutable; is valid as declared in {@link #isValidName(String)}
  */
-public class Name {
+public class Name implements Printable{
 
     public static final String EXAMPLE = "John Doe";
     public static final String MESSAGE_NAME_CONSTRAINTS = "Person names should be spaces or alphanumeric characters";
@@ -61,4 +61,8 @@ public int hashCode() {
         return fullName.hashCode();
     }
 
+    @Override
+    public String getPrintableString() {
+        return "Name: " + fullName;
+    }
 }

src/seedu/addressbook/data/person/Phone.java

@@ -6,7 +6,7 @@
  * Represents a Person's phone number in the address book.
  * Guarantees: immutable; is valid as declared in {@link #isValidPhone(String)}
  */
-public class Phone {
+public class Phone implements Printable{
 
     public static final String EXAMPLE = "123456789";
     public static final String MESSAGE_PHONE_CONSTRAINTS = "Person phone numbers should only contain numbers";
@@ -56,4 +56,9 @@ public int hashCode() {
     public boolean isPrivate() {
         return isPrivate;
     }
+
+    @Override
+    public String getPrintableString() {
+        return "Phone: " + value;
+    }
 }

src/seedu/addressbook/data/person/Printable.java

@@ -0,0 +1,9 @@
+package seedu.addressbook.data.person;
+
+/**
+ * A printable interface for a Person in the addressbook.
+ * Implementations should guarantee: details are present and not null, field values are validated.
+ */
+public interface Printable {
+     String getPrintableString();
+}

src/seedu/addressbook/data/person/ReadOnlyPerson.java

@@ -8,7 +8,7 @@
  * A read-only immutable interface for a Person in the addressbook.
  * Implementations should guarantee: details are present and not null, field values are validated.
  */
-public interface ReadOnlyPerson {
+public interface ReadOnlyPerson extends Printable{
 
     Name getName();
     Phone getPhone();
@@ -33,14 +33,16 @@ default boolean isSameStateAs(ReadOnlyPerson other) {
                 && other.getAddress().equals(this.getAddress()));
     }
 
+    String getPrintableString();
+
     /**
      * Formats the person as text, showing all contact details.
      */
     default String getAsTextShowAll() {
         final StringBuilder builder = new StringBuilder();
         final String detailIsPrivate = "(private) ";
-        builder.append(getName())
-                .append(" Phone: ");
+        builder.append(getPrintableString());
+                //.append(" Phone: ");
         if (getPhone().isPrivate()) {
             builder.append(detailIsPrivate);
         }