Преглед изворни кода

Completed Exceptions&Logging

Chaitali Patel пре 7 година
родитељ
комит
8ef39565d5

+ 28
- 10
src/main/java/com/zipcodewilmington/phone/PhoneNumberFactory.java Прегледај датотеку

@@ -1,13 +1,15 @@
1 1
 package com.zipcodewilmington.phone;
2 2
 
3 3
 import com.zipcodewilmington.exceptions.InvalidPhoneNumberFormatException;
4
+import com.zipcodewilmington.tools.RandomNumberFactory;
4 5
 
5 6
 import java.util.logging.Logger;
7
+import java.util.logging.Level;
6 8
 
7 9
 /**
8 10
  * Created by leon on 5/1/17.
9 11
  */
10
-public final class PhoneNumberFactory {
12
+public final class PhoneNumberFactory extends RandomNumberFactory {
11 13
     private static final Logger logger = Logger.getGlobal();
12 14
 
13 15
     private PhoneNumberFactory() {
@@ -19,17 +21,23 @@ public final class PhoneNumberFactory {
19 21
      * @param phoneNumberCount - number of PhoneNumber objects to instantiate
20 22
      * @return array of randomly generated PhoneNumber objects
21 23
      */ //TODO - Implement logic
22
-    public static PhoneNumber[] createRandomPhoneNumberArray(int phoneNumberCount) {
23
-        return null;
24
+    public static PhoneNumber[] createRandomPhoneNumberArray(int phoneNumberCount){
25
+
26
+         PhoneNumber[] phoneNumberArray = new PhoneNumber[phoneNumberCount];
27
+        return phoneNumberArray;
28
+
24 29
     }
25 30
 
26 31
     /**
27 32
      * @return an instance of PhoneNumber with randomly generated phone number value
28 33
      */ //TODO - Implement logic
29
-    public static PhoneNumber createRandomPhoneNumber() {
30
-        return createPhoneNumberSafely(-1, -1, -1);
31
-    }
34
+    public static PhoneNumber createRandomPhoneNumber() throws InvalidPhoneNumberFormatException {
35
+        int areaCode = createInteger(100,999);
36
+        int centralOfficeCode = createInteger(100, 999);
37
+        int phoneLineCode = createInteger(1000, 9999);
32 38
 
39
+        return createPhoneNumberSafely(areaCode, centralOfficeCode, phoneLineCode);
40
+    }
33 41
 
34 42
     /**
35 43
      * @param areaCode          - 3 digit code
@@ -37,8 +45,15 @@ public final class PhoneNumberFactory {
37 45
      * @param phoneLineCode     - 4 digit code
38 46
      * @return a new phone number object
39 47
      */ //TODO - if input is valid, return respective PhoneNumber object, else return null
40
-    public static PhoneNumber createPhoneNumberSafely(int areaCode, int centralOfficeCode, int phoneLineCode) {
41
-        return createPhoneNumber(null);
48
+    public static PhoneNumber createPhoneNumberSafely(int areaCode, int centralOfficeCode, int phoneLineCode) throws InvalidPhoneNumberFormatException {
49
+        try {
50
+            return createPhoneNumber("(" + areaCode + ")-" + centralOfficeCode + "-" + phoneLineCode);
51
+        }
52
+        catch (InvalidPhoneNumberFormatException e) {
53
+            logger.log(Level.WARNING,
54
+             areaCode + centralOfficeCode +phoneLineCode + "is not valid");
55
+            return null;
56
+        }
42 57
     }
43 58
 
44 59
     /**
@@ -46,7 +61,10 @@ public final class PhoneNumberFactory {
46 61
      * @return a new phone number object
47 62
      * @throws InvalidPhoneNumberFormatException - thrown if phoneNumberString does not match acceptable format
48 63
      */ // TODO - Add throws statement to method signature
49
-    public static PhoneNumber createPhoneNumber(String phoneNumberString) {
50
-        return null;
64
+    public static PhoneNumber createPhoneNumber(String phoneNumberString) throws InvalidPhoneNumberFormatException {
65
+        logger.log(Level.INFO,"Attempting to create a new phoneNumber object with value of "+ phoneNumberString);
66
+        PhoneNumber newNum = new PhoneNumber(phoneNumberString);
67
+        return newNum;
68
+
51 69
     }
52 70
 }

+ 5
- 5
src/test/java/com/zipcodewilmington/PhoneNumberFactoryTest.java Прегледај датотеку

@@ -21,7 +21,7 @@ public class PhoneNumberFactoryTest {
21 21
     }
22 22
 
23 23
     @Test
24
-    public void testCreatePhoneNumberSafely() {
24
+    public void testCreatePhoneNumberSafely()  throws InvalidPhoneNumberFormatException{
25 25
         // : Given
26 26
         int areaCode = 0;
27 27
         int centralOfficeCode = 0;
@@ -35,7 +35,7 @@ public class PhoneNumberFactoryTest {
35 35
     }
36 36
 
37 37
     @Test
38
-    public void testGetAreaCode() {
38
+    public void testGetAreaCode()  throws InvalidPhoneNumberFormatException{
39 39
         // : Given
40 40
         Integer areaCode = 302;
41 41
         int centralOfficeCode = 312;
@@ -49,7 +49,7 @@ public class PhoneNumberFactoryTest {
49 49
     }
50 50
 
51 51
     @Test
52
-    public void testGetCentralOfficeCode() {
52
+    public void testGetCentralOfficeCode()  throws InvalidPhoneNumberFormatException{
53 53
         // : Given
54 54
         int areaCode = 302;
55 55
         Integer centralOfficeCode = 312;
@@ -64,7 +64,7 @@ public class PhoneNumberFactoryTest {
64 64
 
65 65
 
66 66
     @Test
67
-    public void testPhoneLineCode() {
67
+    public void testPhoneLineCode()  throws InvalidPhoneNumberFormatException{
68 68
         // : Given
69 69
         int areaCode = 302;
70 70
         int centralOfficeCode = 312;
@@ -78,7 +78,7 @@ public class PhoneNumberFactoryTest {
78 78
     }
79 79
 
80 80
     @Test
81
-    public void testCreateRandomPhoneNumber() {
81
+    public void testCreateRandomPhoneNumber() throws InvalidPhoneNumberFormatException {
82 82
         for (int i = 0; i < 999; i++) {
83 83
             // : Given
84 84
             // : When