Redesigned in builder pattern

Author: ul-jf

src/main/java/io/ipgeolocation/api/TimezoneParams.java

@@ -1,69 +1,114 @@
 package io.ipgeolocation.api;
 
+import io.ipgeolocation.api.exceptions.IPGeolocationError;
+import java.math.BigDecimal;
+
 public class TimezoneParams {
-    private String timezone;
-    private String ip;
-    private String lang;
-    private Double latitude;
-    private Double longitude;
-    private String location;
+  private final String timeZone;
+  private final String ipAddress;
+  private final BigDecimal latitude;
+  private final BigDecimal longitude;
+  private final String location;
+  private final String lang;
 
-    public TimezoneParams() {
-        this.timezone = "";
-        this.ip = "";
-        this.lang = "en";
-        this.location = "";
-        this.latitude = 1000.00;
-        this.longitude = 1000.00;
-    }
+  private TimezoneParams(
+      String timeZone,
+      String ipAddress,
+      BigDecimal latitude,
+      BigDecimal longitude,
+      String location,
+      String lang) {
+    this.timeZone = timeZone;
+    this.ipAddress = ipAddress;
+    this.latitude = latitude;
+    this.longitude = longitude;
+    this.location = location;
+    this.lang = lang;
+  }
 
-    public void setTimezone(String timezone) {
-        this.timezone = Strings.nullToEmpty(timezone);
-    }
+  public String getTimeZone() {
+    return timeZone;
+  }
 
-    public String getTimezone() {
-        return this.timezone;
-    }
+  public String getIPAddress() {
+    return ipAddress;
+  }
 
-    public void setIPAddress(String ip) {
-        this.ip = Strings.nullToEmpty(ip);
-    }
+  public BigDecimal getLatitude() {
+    return latitude;
+  }
 
-    public String getIPAddress() {
-        return this.ip;
-    }
+  public BigDecimal getLongitude() {
+    return longitude;
+  }
 
-    public void setLang(String lang) {
-        this.lang = lang;
-    }
+  public String getLocation() {
+    return location;
+  }
 
-    public String getLang() {
-        return this.lang;
+  public String getLang() {
+    return lang;
+  }
+
+  public static TimezoneParamsBuilder builder() {
+    return new TimezoneParamsBuilder();
+  }
+
+  public static class TimezoneParamsBuilder {
+    private String timeZone;
+    private String ipAddress;
+    private BigDecimal latitude;
+    private BigDecimal longitude;
+    private String location;
+    private String lang;
+
+    private TimezoneParamsBuilder() {
+      timeZone = "";
+      ipAddress = "";
+      latitude = null;
+      longitude = null;
+      location = "";
+      lang = "en";
     }
 
-    public void setCoordinates(Double latitude, Double longitude) {
-        if ((latitude >= -90 && latitude <= 90) && (longitude >= -180 && longitude <= 180)) {
-            this.latitude = latitude;
-            this.longitude = longitude;
-        } else {
-            throw new IllegalArgumentException("Coordinate value is out of range!");
-        }
+    public TimezoneParamsBuilder withTimeZone(String timeZone) {
+      this.timeZone = timeZone;
+      return this;
     }
 
-    public Double getLatitude() {
-        return this.latitude;
+    public TimezoneParamsBuilder withIPAddress(String ipAddress) {
+      this.ipAddress = ipAddress;
+      return this;
     }
 
-    public Double getLongitude() {
-        return this.longitude;
+    public TimezoneParamsBuilder withCoordinates(BigDecimal latitude, BigDecimal longitude) {
+      if (latitude.compareTo(BigDecimal.valueOf(-90.0)) < 0
+          || latitude.compareTo(BigDecimal.valueOf(90.0)) > 0) {
+        throw new IPGeolocationError("'latitude' must be between -90.0 and 90.0");
+      }
+
+      if (longitude.compareTo(BigDecimal.valueOf(-180.0)) < 0
+          || longitude.compareTo(BigDecimal.valueOf(180.0)) > 0) {
+        throw new IPGeolocationError("'longitude' must be between -180.0 and 180.0");
+      }
+
+      this.latitude = latitude;
+      this.longitude = longitude;
+      return this;
     }
 
-    public String getLocation() {
-        return location;
+    public TimezoneParamsBuilder withLocation(String location) {
+      this.location = location;
+      return this;
     }
 
-    public void setLocation(String location) {
-        this.location = location;
+    public TimezoneParamsBuilder withLang(String lang) {
+      this.lang = lang;
+      return this;
     }
 
+    public TimezoneParams build() {
+      return new TimezoneParams(timeZone, ipAddress, latitude, longitude, location, lang);
+    }
+  }
 }