diff --git a/src/main/java/com/zipcodewilmington/exceptions/InvalidPhoneNumberFormatException.java b/src/main/java/com/zipcodewilmington/exceptions/InvalidPhoneNumberFormatException.java index 5d3fb58..9c1a95a 100644 --- a/src/main/java/com/zipcodewilmington/exceptions/InvalidPhoneNumberFormatException.java +++ b/src/main/java/com/zipcodewilmington/exceptions/InvalidPhoneNumberFormatException.java @@ -4,4 +4,6 @@ * Created by leon on 5/10/17. */ // Checked Exception public final class InvalidPhoneNumberFormatException extends Exception { + + } diff --git a/src/main/java/com/zipcodewilmington/phone/PhoneNumberFactory.java b/src/main/java/com/zipcodewilmington/phone/PhoneNumberFactory.java index 36d323b..a198a8c 100644 --- a/src/main/java/com/zipcodewilmington/phone/PhoneNumberFactory.java +++ b/src/main/java/com/zipcodewilmington/phone/PhoneNumberFactory.java @@ -1,7 +1,9 @@ package com.zipcodewilmington.phone; import com.zipcodewilmington.exceptions.InvalidPhoneNumberFormatException; +import com.zipcodewilmington.tools.RandomNumberFactory; +import java.util.logging.Level; import java.util.logging.Logger; /** @@ -9,6 +11,9 @@ */ public final class PhoneNumberFactory { private static final Logger logger = Logger.getGlobal(); +// private int areaCode; +// private int centralOfficeCode; +// private int phoneLineCode; private PhoneNumberFactory() { /** This constructor is private @@ -19,15 +24,23 @@ private PhoneNumberFactory() { * @param phoneNumberCount - number of PhoneNumber objects to instantiate * @return array of randomly generated PhoneNumber objects */ //TODO - Implement logic - public static PhoneNumber[] createRandomPhoneNumberArray(int phoneNumberCount) { - return null; + public static PhoneNumber[] createRandomPhoneNumberArray(int phoneNumberCount) throws InvalidPhoneNumberFormatException{ + PhoneNumber[] phoneNumbers= new PhoneNumber[phoneNumberCount]; + for (int i = 0; i < phoneNumberCount; i++ ){ + phoneNumbers[i] = createRandomPhoneNumber(); + } + return phoneNumbers; } /** * @return an instance of PhoneNumber with randomly generated phone number value */ //TODO - Implement logic - public static PhoneNumber createRandomPhoneNumber() { - return createPhoneNumberSafely(-1, -1, -1); + public static PhoneNumber createRandomPhoneNumber() throws InvalidPhoneNumberFormatException { + Integer areaCode = RandomNumberFactory.createInteger(100, 999); + Integer centralOfficeCode = RandomNumberFactory.createInteger(100, 999); + Integer phoneLineCode = RandomNumberFactory.createInteger(1000 ,9999); + + return createPhoneNumberSafely(areaCode, centralOfficeCode, phoneLineCode); } @@ -37,8 +50,15 @@ public static PhoneNumber createRandomPhoneNumber() { * @param phoneLineCode - 4 digit code * @return a new phone number object */ //TODO - if input is valid, return respective PhoneNumber object, else return null - public static PhoneNumber createPhoneNumberSafely(int areaCode, int centralOfficeCode, int phoneLineCode) { - return createPhoneNumber(null); + public static PhoneNumber createPhoneNumberSafely(int areaCode, int centralOfficeCode, int phoneLineCode) throws InvalidPhoneNumberFormatException{ + StringBuilder sb = new StringBuilder(); + String correctPhoneFormat = sb.append("(").append(areaCode).append(")-").append(centralOfficeCode).append("-").append(phoneLineCode).toString(); + + try { return createPhoneNumber(correctPhoneFormat);} + catch (InvalidPhoneNumberFormatException e) { + logger.log(Level.INFO, String.format("%s, is not a valid phone number", correctPhoneFormat)); + return null; + } } /** @@ -46,7 +66,9 @@ public static PhoneNumber createPhoneNumberSafely(int areaCode, int centralOffic * @return a new phone number object * @throws InvalidPhoneNumberFormatException - thrown if phoneNumberString does not match acceptable format */ // TODO - Add throws statement to method signature - public static PhoneNumber createPhoneNumber(String phoneNumberString) { - return null; + public static PhoneNumber createPhoneNumber(String phoneNumberString) throws InvalidPhoneNumberFormatException { + PhoneNumber phoneNumber = new PhoneNumber(phoneNumberString); + logger.log(Level.INFO, String.format("Attempting to create a new PhoneNumber object with a value of %s", phoneNumber)); + return phoneNumber; } } diff --git a/src/test/java/com/zipcodewilmington/PhoneNumberFactoryTest.java b/src/test/java/com/zipcodewilmington/PhoneNumberFactoryTest.java index a607c50..1c3f22f 100644 --- a/src/test/java/com/zipcodewilmington/PhoneNumberFactoryTest.java +++ b/src/test/java/com/zipcodewilmington/PhoneNumberFactoryTest.java @@ -21,7 +21,7 @@ public void testInvalidPhoneNumberFormatException() throws InvalidPhoneNumberFor } @Test - public void testCreatePhoneNumberSafely() { + public void testCreatePhoneNumberSafely() throws InvalidPhoneNumberFormatException { // : Given int areaCode = 0; int centralOfficeCode = 0; @@ -35,7 +35,7 @@ public void testCreatePhoneNumberSafely() { } @Test - public void testGetAreaCode() { + public void testGetAreaCode() throws InvalidPhoneNumberFormatException { // : Given Integer areaCode = 302; int centralOfficeCode = 312; @@ -49,7 +49,7 @@ public void testGetAreaCode() { } @Test - public void testGetCentralOfficeCode() { + public void testGetCentralOfficeCode() throws InvalidPhoneNumberFormatException { // : Given int areaCode = 302; Integer centralOfficeCode = 312; @@ -64,7 +64,7 @@ public void testGetCentralOfficeCode() { @Test - public void testPhoneLineCode() { + public void testPhoneLineCode() throws InvalidPhoneNumberFormatException { // : Given int areaCode = 302; int centralOfficeCode = 312; @@ -78,7 +78,7 @@ public void testPhoneLineCode() { } @Test - public void testCreateRandomPhoneNumber() { + public void testCreateRandomPhoneNumber() throws InvalidPhoneNumberFormatException { for (int i = 0; i < 999; i++) { // : Given // : When