diff options
author | Anton Tananaev <anton.tananaev@gmail.com> | 2015-10-08 15:08:33 +1300 |
---|---|---|
committer | Anton Tananaev <anton.tananaev@gmail.com> | 2015-10-08 15:08:33 +1300 |
commit | eb4ee7a9d4d0f132546b164c3483d62eb9e9f0db (patch) | |
tree | 29bc56f0f67cd49bac31a50e415c614ed4b1c5dc /src/org/traccar/geocode/GoogleReverseGeocoder.java | |
parent | 320d41863864485c5675c9bf8332ff0845e5e812 (diff) | |
download | trackermap-server-eb4ee7a9d4d0f132546b164c3483d62eb9e9f0db.tar.gz trackermap-server-eb4ee7a9d4d0f132546b164c3483d62eb9e9f0db.tar.bz2 trackermap-server-eb4ee7a9d4d0f132546b164c3483d62eb9e9f0db.zip |
Clean up reported PMD issues
Diffstat (limited to 'src/org/traccar/geocode/GoogleReverseGeocoder.java')
-rw-r--r-- | src/org/traccar/geocode/GoogleReverseGeocoder.java | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/src/org/traccar/geocode/GoogleReverseGeocoder.java b/src/org/traccar/geocode/GoogleReverseGeocoder.java index ade49a38b..ca1d42000 100644 --- a/src/org/traccar/geocode/GoogleReverseGeocoder.java +++ b/src/org/traccar/geocode/GoogleReverseGeocoder.java @@ -30,7 +30,7 @@ public class GoogleReverseGeocoder extends JsonReverseGeocoder { } @Override - protected Address parseAddress(JsonObject json) { + public Address parseAddress(JsonObject json) { JsonArray results = json.getJsonArray("results"); if (!results.isEmpty()) { @@ -43,35 +43,31 @@ public class GoogleReverseGeocoder extends JsonReverseGeocoder { String value = component.getString("short_name"); - for (JsonString type : component.getJsonArray("types").getValuesAs(JsonString.class)) { + typesLoop: for (JsonString type : component.getJsonArray("types").getValuesAs(JsonString.class)) { switch (type.getString()) { case "street_number": address.setHouse(value); - break; + break typesLoop; case "route": address.setStreet(value); - break; + break typesLoop; case "locality": address.setSettlement(value); - break; + break typesLoop; case "administrative_area_level_2": address.setDistrict(value); - break; + break typesLoop; case "administrative_area_level_1": address.setState(value); - break; + break typesLoop; case "country": address.setCountry(value); - break; + break typesLoop; case "postal_code": address.setPostcode(value); - break; - default: - continue; + break typesLoop; } - - break; } } |