78th iteration thanks to vg

src/main/java/com/zipcodewilmington/phonebook/Person.java

@@ -1,42 +1,32 @@
 package com.zipcodewilmington.phonebook;
 
-/*
+
+import java.util.ArrayList;
 
 public class Person {
 
     //variables
 
-    /*we made these private bc we don't want the user to be able to change the data. Another reason we want to do this
-     is that you want to "hide" how you store the information from the users.
+    /**
+     * we made these private bc we don't want the user to be able to change the data. Another reason we want to do this
+     * is that you want to "hide" how you store the information from the users.
+     */
 
 
     private String name;
-    private String phoneNumber;
-
-    //constructor
-    public Person(String name, String number) {
-        this.name= name;
-        this.phoneNumber = number;
-    }
-    //"everything else"
-    public String getName() {
-        return name;
-    }
+    private ArrayList<String> phoneNumber;
 
-    public void setName(String name) {
-        this.name = name;
-    }
+    public Person (String entryName, String entryNumber) {
+        this.name = entryName;
+        this.phoneNumber = new ArrayList<String>();
+        this.phoneNumber.add(entryNumber);
 
-    public String getPhoneNumber() {
-        return phoneNumber;
     }
 
-    public void setPhoneNumber(String number) {
-        this.phoneNumber = number;
+    public String getNumbers(){
+        return phoneNumber.toString();
     }
-
-
 }
 
-*/
+
 

src/main/java/com/zipcodewilmington/phonebook/PhoneBook.java

@@ -1,90 +1,62 @@
 package com.zipcodewilmington.phonebook;
 
+import com.sun.tools.doclets.formats.html.SourceToHTMLConverter;
+
 import java.util.ArrayList;
 import java.util.Map;
 import java.util.Set;
 import java.util.TreeMap;
 
+//phoneBook class is like the cover of the book.
 public class PhoneBook {
 
-    //the below defines what kind of attribute you have, which in this case is your "contacts"
-    TreeMap<String, String> personMap;
-
-    String name = "";
-
-    //this is your constructor. whenever you see public + the classname, that is your constructor
-    //
-    // (this phoneBook constructor is like the "cover" of your phonebook (the binding and the spine). it creates an empty book.)
-
-    //this can make as many phonebook objects as you want
-    public PhoneBook() {
-
-        //when we make our phonebooks, this is the form they will take (below)
-        //
-        // this treeMap initializes the "blank pages of the book". the treemap is the structure of how your book will hold info
+    public static void main(String[] args) {
 
-        //if you were to not set this, your person map would be null(default setting)
-        this.personMap = new TreeMap<String, String>();
+        PhoneBook myPhoneBookActualObjectInstance= new PhoneBook();
+        myPhoneBookActualObjectInstance.add("vince", "1123456789");
+        myPhoneBookActualObjectInstance.add("amy", "29387429");
+        System.out.println(myPhoneBookActualObjectInstance.showNames());
     }
 
-    // this is our method below. it describes what we can do with this phonebook
 
-    //so later, when you call "phonebook.add", that's like someone literally writing a new name and number in your phonebook object
+    /*the treeMap is like the t structure with key/values. key is person's name in this example, value is the arraylist
+    which can hold multiple numbers, fields etc.
+    */
+    TreeMap<String, Person> personTreeMap = new TreeMap<String, Person>();
 
+    public PhoneBook() {
 
-    public void add(String name, String number) {
-        personMap.put(name, number);
     }
 
-    /*
-    //this method would really be better for if you need more information on your person objects (like address, etc).
-
-    public void add(Person person){
-        this.personMap.put(person.getName(), person);
+    public void add(String name, String number) {
+        Person phoneBookEntry = new Person(name, number);
+        personTreeMap.put(name, phoneBookEntry);
     }
 
-    */
 
-    public void remove(String name, String number) {
+    public void remove(String name) {
 
-        personMap.remove(name);
+        personTreeMap.remove(name);
     }
 
     public String lookup(String name) {
 
-        return personMap.get(name);
+        return personTreeMap.get(name).getNumbers();
 
     }
 
-   /* public String reverseLookup (String phoneNumber){
-
-        //this will "grab" all the listings in your book. it does NOT print them.
-        Set<Map.Entry<String, String> = personMap.entrySet();
-
-        for (Map.Entry<String, String> phoneBookEntry : entries) {
+    public String showNames() {
 
-            Person person = phoneBookEntry.getValue();
-
-            if (person.getPhoneNumber().equals(phoneNumber)){
-                return phoneBookEntry.getKey();
+        String phoneList = "";
+        Set<String> unsortedKeys = personTreeMap.keySet();
+        for (String personsName : unsortedKeys) {
+            phoneList += (personsName + "\n");
             }
-        }
 
-       return "We're sorry. The requested phone number could not be found. Have fun when you code!!!!!!!!";
+        return phoneList;
 
-    }
-
-    */
-
-    public void displayAllEntries() {
-
-        for (Map.Entry<String, String> entry : personMap.entrySet()) {
-            String name = entry.getKey();
-            String number = entry.getValue();
         }
 
     }
 
 
-}
-

src/main/java/com/zipcodewilmington/phonebook/oldPhonebookClass.java

@@ -0,0 +1,96 @@
+
+/*
+package com.zipcodewilmington.phonebook;
+
+        import java.util.ArrayList;
+        import java.util.Map;
+        import java.util.Set;
+        import java.util.TreeMap;
+
+public class oldPhonebookClass {
+
+    //the below defines what kind of attribute you have, which in this case is your "contacts"
+    TreeMap<String, String> personMap;
+
+    String name = "";
+
+    //this is your constructor. whenever you see public + the classname, that is your constructor
+    //
+    // (this phoneBook constructor is like the "cover" of your phonebook (the binding and the spine). it creates an empty book.)
+
+    //this can make as many phonebook objects as you want
+    public PhoneBook() {
+
+        //when we make our phonebooks, this is the form they will take (below)
+        //
+        // this treeMap initializes the "blank pages of the book". the treemap is the structure of how your book will hold info
+
+        //if you were to not set this, your person map would be null(default setting)
+        this.personMap = new TreeMap<String, String>();
+    }
+
+    // this is our method below. it describes what we can do with this phonebook
+
+    //so later, when you call "phonebook.add", that's like someone literally writing a new name and number in your phonebook object
+
+
+    public void add(String name, String number) {
+        Person JohnDoe = new Person(name, number);
+        personMap.put(name, number);
+    }
+
+
+
+    /*
+    //this method would really be better for if you need more information on your person objects (like address, etc).
+
+    public void add(Person person){
+        this.personMap.put(person.getName(), person);
+    }
+
+
+
+    public void remove(String name, String number) {
+
+        personMap.remove(name);
+    }
+
+    public String lookup(String name) {
+
+        return personMap.get(name);
+
+    }
+
+   /* public String reverseLookup (String phoneNumber){
+
+        //this will "grab" all the listings in your book. it does NOT print them.
+        Set<Map.Entry<String, String> = personMap.entrySet();
+
+        for (Map.Entry<String, String> phoneBookEntry : entries) {
+
+            Person person = phoneBookEntry.getValue();
+
+            if (person.getPhoneNumber().equals(phoneNumber)){
+                return phoneBookEntry.getKey();
+            }
+        }
+
+       return "We're sorry. The requested phone number could not be found. Have fun when you code!!!!!!!!";
+
+    }
+
+
+
+    public void displayAllEntries() {
+
+        for (Map.Entry<String, String> entry : personMap.entrySet()) {
+            String name = entry.getKey();
+            String number = entry.getValue();
+        }
+
+    }
+
+
+}
+
+*/

src/test/java/com/zipcodewilmington/phonebook/PhoneBookTest.java

@@ -14,8 +14,6 @@ public class PhoneBookTest {
 
     PhoneBook testPhonebook = new PhoneBook();
 
-    TreeMap<String, String> personMapTest= new TreeMap<String, String>();
-
     @Before
 
     public void setup(){
@@ -27,10 +25,10 @@ public class PhoneBookTest {
     @Test
     public void testLookup() {
 
-        personMapTest.put("JohnDoe", "1123");
+        testPhonebook.add("JohnDoe", "1123");
 
-        String expected = "JohnDoe";
-        String actual = testPhonebook.lookup(expected);
+        String expected = "1123";
+        String actual = testPhonebook.lookup("JohnDoe");
 
         Assert.assertEquals(expected, actual);