Add Builder classes

Allows the creation of SearchLocationService.java and LocationService.java to be more controlled.

Fixes #13

location4j/src/main/java/com/tomaytotomato/LocationService.java

@@ -4,11 +4,9 @@
 import static java.lang.Math.sqrt;
 
 import com.tomaytotomato.loader.CountriesDataLoader;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
 import com.tomaytotomato.model.City;
 import com.tomaytotomato.model.Country;
 import com.tomaytotomato.model.State;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
 import com.tomaytotomato.text.normaliser.TextNormaliser;
 import com.tomaytotomato.usecase.FindCity;
 import com.tomaytotomato.usecase.FindCountry;
@@ -51,24 +49,7 @@ public class LocationService implements FindCountry, FindState, FindCity {
 
     private final TextNormaliser textNormaliser;
 
-    /**
-     * Default constructor, creates a LocationService class with default dependencies
-     */
-    public LocationService() {
-        this.textNormaliser = new DefaultTextNormaliser();
-        var dataLoader = new DefaultCountriesDataLoaderImpl();
-        countries = dataLoader.getCountries();
-        buildDataStructures();
-    }
-
-    public LocationService(TextNormaliser textNormaliser) {
-        this.textNormaliser = textNormaliser;
-        var dataLoader = new DefaultCountriesDataLoaderImpl();
-        countries = dataLoader.getCountries();
-        buildDataStructures();
-    }
-
-    public LocationService(TextNormaliser textNormaliser, CountriesDataLoader dataLoader) {
+    protected LocationService(TextNormaliser textNormaliser, CountriesDataLoader dataLoader) {
         this.textNormaliser = textNormaliser;
         countries = dataLoader.getCountries();
         buildDataStructures();

location4j/src/main/java/com/tomaytotomato/LocationServiceBuilder.java

@@ -0,0 +1,33 @@
+package com.tomaytotomato;
+
+import com.tomaytotomato.loader.CountriesDataLoader;
+import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
+import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
+import com.tomaytotomato.text.normaliser.TextNormaliser;
+
+public final class LocationServiceBuilder {
+
+  private TextNormaliser textNormaliser = new DefaultTextNormaliser();
+  private CountriesDataLoader countriesDataLoader = new DefaultCountriesDataLoaderImpl();
+
+  public static LocationServiceBuilder builder() {
+    return new LocationServiceBuilder();
+  }
+
+  private LocationServiceBuilder() {
+  }
+
+  public LocationServiceBuilder withCountriesDataLoader(CountriesDataLoader countriesDataLoader) {
+    this.countriesDataLoader = countriesDataLoader;
+    return this;
+  }
+
+  public LocationServiceBuilder withTextNormaliser(TextNormaliser textNormaliser) {
+    this.textNormaliser = textNormaliser;
+    return this;
+  }
+
+  public LocationService build() {
+    return new LocationService(textNormaliser, countriesDataLoader);
+  }
+}

location4j/src/main/java/com/tomaytotomato/SearchLocationService.java

@@ -1,18 +1,13 @@
 package com.tomaytotomato;
 
-import com.tomaytotomato.aliases.DefaultLocationAliases;
 import com.tomaytotomato.aliases.LocationAliases;
 import com.tomaytotomato.loader.CountriesDataLoader;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
-import com.tomaytotomato.mapper.DefaultLocationMapper;
 import com.tomaytotomato.mapper.LocationMapper;
 import com.tomaytotomato.model.City;
 import com.tomaytotomato.model.Country;
 import com.tomaytotomato.model.Location;
 import com.tomaytotomato.model.State;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
 import com.tomaytotomato.text.normaliser.TextNormaliser;
-import com.tomaytotomato.text.tokeniser.DefaultTextTokeniser;
 import com.tomaytotomato.text.tokeniser.TextTokeniser;
 import com.tomaytotomato.usecase.SearchLocation;
 import java.util.ArrayList;
@@ -51,17 +46,7 @@ public class SearchLocationService implements SearchLocation {
   private final LocationMapper locationMapper;
   private final LocationAliases locationAliases;
 
-  public SearchLocationService() {
-    textTokeniser = new DefaultTextTokeniser();
-    textNormaliser = new DefaultTextNormaliser();
-    locationMapper = new DefaultLocationMapper();
-    locationAliases = new DefaultLocationAliases();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-    countries = dataLoader.getCountries();
-    buildDataStructures();
-  }
-
-  public SearchLocationService(TextTokeniser textTokeniser, TextNormaliser textNormaliser,
+  protected SearchLocationService(TextTokeniser textTokeniser, TextNormaliser textNormaliser,
       LocationMapper locationMapper, CountriesDataLoader dataLoader,
       LocationAliases locationAliases) {
     this.textTokeniser = textTokeniser;

location4j/src/main/java/com/tomaytotomato/SearchLocationServiceBuilder.java

@@ -0,0 +1,62 @@
+package com.tomaytotomato;
+
+import com.tomaytotomato.aliases.DefaultLocationAliases;
+import com.tomaytotomato.aliases.LocationAliases;
+import com.tomaytotomato.loader.CountriesDataLoader;
+import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
+import com.tomaytotomato.mapper.DefaultLocationMapper;
+import com.tomaytotomato.mapper.LocationMapper;
+import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
+import com.tomaytotomato.text.normaliser.TextNormaliser;
+import com.tomaytotomato.text.tokeniser.DefaultTextTokeniser;
+import com.tomaytotomato.text.tokeniser.TextTokeniser;
+
+/**
+ * Allows the customisation and creation of the {@link SearchLocationService}
+ */
+public final class SearchLocationServiceBuilder {
+
+  private TextTokeniser textTokeniser = new DefaultTextTokeniser();
+  private TextNormaliser textNormaliser = new DefaultTextNormaliser();
+  private LocationMapper locationMapper = new DefaultLocationMapper();
+  private LocationAliases locationAliases = new DefaultLocationAliases();
+  private CountriesDataLoader countriesDataLoader = new DefaultCountriesDataLoaderImpl();
+
+  public static SearchLocationServiceBuilder builder() {
+    return new SearchLocationServiceBuilder();
+  }
+
+  private SearchLocationServiceBuilder() {
+  }
+
+  public SearchLocationServiceBuilder withTextTokeniser(TextTokeniser textTokeniser) {
+    this.textTokeniser = textTokeniser;
+    return this;
+  }
+
+  public SearchLocationServiceBuilder withTextNormaliser(TextNormaliser textNormaliser) {
+    this.textNormaliser = textNormaliser;
+    return this;
+  }
+
+  public SearchLocationServiceBuilder withLocationMapper(LocationMapper locationMapper) {
+    this.locationMapper = locationMapper;
+    return this;
+  }
+
+  public SearchLocationServiceBuilder withLocationAliases(LocationAliases locationAliases) {
+    this.locationAliases = locationAliases;
+    return this;
+  }
+
+  public SearchLocationServiceBuilder withCountriesDataLoader(CountriesDataLoader countriesDataLoader) {
+    this.countriesDataLoader = countriesDataLoader;
+    return this;
+  }
+
+  public SearchLocationService build() {
+    return new SearchLocationService(textTokeniser, textNormaliser, locationMapper,
+        countriesDataLoader,
+        locationAliases);
+  }
+}

location4j/src/test/java/com/tomaytotomato/usecase/FindCityTest.java

@@ -4,9 +4,7 @@
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import com.tomaytotomato.LocationService;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
+import com.tomaytotomato.LocationServiceBuilder;
 import java.math.BigDecimal;
 import jdk.jfr.Description;
 import org.junit.jupiter.api.DisplayName;
@@ -21,9 +19,8 @@ class FindCityTest {
     private final FindCity locationService;
 
     public FindCityTest() {
-        var textNormaliser = new DefaultTextNormaliser();
-        var dataLoader = new DefaultCountriesDataLoaderImpl();
-        locationService = new LocationService(textNormaliser, dataLoader);
+        locationService = LocationServiceBuilder.builder()
+            .build();
     }
 
     @Description("Find City by Lat/long with BigDecimal")

location4j/src/test/java/com/tomaytotomato/usecase/FindCountriesByStateTest.java

@@ -3,10 +3,8 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-import com.tomaytotomato.LocationService;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
+import com.tomaytotomato.LocationServiceBuilder;
 import com.tomaytotomato.model.Country;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
 import jdk.jfr.Description;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -20,9 +18,7 @@ class FindCountriesByStateTest {
   private final FindCountry locationService;
 
   public FindCountriesByStateTest() {
-    var textNormaliser = new DefaultTextNormaliser();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-    locationService = new LocationService(textNormaliser, dataLoader);
+    locationService = LocationServiceBuilder.builder().build();
   }
 
   @Description("Find All Countries By State Name, when null or blank then throw exception")

location4j/src/test/java/com/tomaytotomato/usecase/FindCountryByISOCodeTest.java

@@ -4,7 +4,7 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-import com.tomaytotomato.LocationService;
+import com.tomaytotomato.LocationServiceBuilder;
 import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
 import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
 import org.junit.jupiter.api.DisplayName;
@@ -19,9 +19,10 @@ class FindCountryByISOCodeTest {
   private final FindCountry locationService;
 
   public FindCountryByISOCodeTest() {
-    var textNormaliser = new DefaultTextNormaliser();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-    locationService = new LocationService(textNormaliser, dataLoader);
+    locationService = LocationServiceBuilder.builder()
+        .withCountriesDataLoader(new DefaultCountriesDataLoaderImpl())
+        .withTextNormaliser(new DefaultTextNormaliser())
+        .build();
   }
 
   @DisplayName("Get Country By ISO2 Code, when null or blank then throw exception")

location4j/src/test/java/com/tomaytotomato/usecase/FindCountryTest.java

@@ -4,10 +4,8 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-import com.tomaytotomato.LocationService;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
+import com.tomaytotomato.LocationServiceBuilder;
 import com.tomaytotomato.model.Country;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
 import java.util.List;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -21,9 +19,7 @@ class FindCountryTest {
   private final FindCountry locationService;
 
   public FindCountryTest() {
-    var textNormaliser = new DefaultTextNormaliser();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-    locationService = new LocationService(textNormaliser, dataLoader);
+    locationService = LocationServiceBuilder.builder().build();
   }
 
   @DisplayName("Find Country By ID, when valid and exists, then return Country")

location4j/src/test/java/com/tomaytotomato/usecase/FindStateTest.java

@@ -4,9 +4,7 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-import com.tomaytotomato.LocationService;
-import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
-import com.tomaytotomato.text.normaliser.DefaultTextNormaliser;
+import com.tomaytotomato.LocationServiceBuilder;
 import jdk.jfr.Description;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -20,9 +18,7 @@ class FindStateTest {
   private final FindState locationService;
 
   public FindStateTest() {
-    var textNormaliser = new DefaultTextNormaliser();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-    locationService = new LocationService(textNormaliser, dataLoader);
+    locationService = LocationServiceBuilder.builder().build();
   }
 
   @Description("Find State By ID, when null then throw exception")

location4j/src/test/java/com/tomaytotomato/usecase/SearchLocationTest.java

@@ -4,7 +4,7 @@
 import static org.assertj.core.api.Assertions.tuple;
 import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
 
-import com.tomaytotomato.SearchLocationService;
+import com.tomaytotomato.SearchLocationServiceBuilder;
 import com.tomaytotomato.aliases.DefaultLocationAliases;
 import com.tomaytotomato.loader.DefaultCountriesDataLoaderImpl;
 import com.tomaytotomato.mapper.DefaultLocationMapper;
@@ -23,24 +23,21 @@ class SearchLocationTest {
   private final SearchLocation searchLocationService;
 
   public SearchLocationTest() {
-    var textNormaliser = new DefaultTextNormaliser();
-    var textTokeniser = new DefaultTextTokeniser();
-    var locationMapper = new DefaultLocationMapper();
-    var locationAliases = new DefaultLocationAliases();
-    var dataLoader = new DefaultCountriesDataLoaderImpl();
-
-    searchLocationService = new SearchLocationService(textTokeniser, textNormaliser, locationMapper,
-        dataLoader, locationAliases);
+    searchLocationService = SearchLocationServiceBuilder.builder()
+        .withLocationAliases(new DefaultLocationAliases())
+        .withLocationMapper(new DefaultLocationMapper())
+        .withCountriesDataLoader(new DefaultCountriesDataLoaderImpl())
+        .withTextNormaliser(new DefaultTextNormaliser())
+        .withTextTokeniser(new DefaultTextTokeniser())
+        .build();
   }
 
   @Description("SearchLocation, when null or empty text, then throw exception")
   @Test
   void search_WhenNullOrBlank_ThenThrowException() {
 
     // When Then
-    assertThatThrownBy(() -> {
-      searchLocationService.search(null);
-    }).isInstanceOf(IllegalArgumentException.class)
+    assertThatThrownBy(() -> searchLocationService.search(null)).isInstanceOf(IllegalArgumentException.class)
         .hasMessageContaining("SearchLocation Text cannot be null or empty");
   }