From f1f38d264084d5044998abb5d7587e147afdf769 Mon Sep 17 00:00:00 2001 From: Bananeweizen Date: Sun, 12 Jan 2014 08:36:10 +0100 Subject: refactoring: remove unused code --- main/res/values-cs/strings.xml | 2 -- main/res/values-da/strings.xml | 1 - main/res/values-de/strings.xml | 2 -- main/res/values-es/strings.xml | 2 -- main/res/values-fr/strings.xml | 2 -- main/res/values-hu/strings.xml | 2 -- main/res/values-it/strings.xml | 2 -- main/res/values-ja/strings.xml | 2 -- main/res/values-nb/strings.xml | 2 -- main/res/values-nl/strings.xml | 2 -- main/res/values-pl/strings.xml | 2 -- main/res/values-pt/strings.xml | 2 -- main/res/values-sk/strings.xml | 2 -- main/res/values-sl/strings.xml | 2 -- main/res/values-sv/strings.xml | 2 -- main/res/values/strings.xml | 2 -- main/src/cgeo/geocaching/CacheListActivity.java | 4 ++- main/src/cgeo/geocaching/Geocache.java | 12 +------- .../cgeo/geocaching/enumerations/StatusCode.java | 2 -- main/src/cgeo/geocaching/geopoint/Geopoint.java | 26 +++++----------- .../cgeo/geocaching/geopoint/GeopointParser.java | 36 +++------------------- .../ui/dialog/CoordinatesInputDialog.java | 4 +-- tests/src/cgeo/geocaching/CgeoApplicationTest.java | 2 +- .../cgeo/geocaching/connector/gc/GCParserTest.java | 5 +-- .../geocaching/geopoint/GeoPointParserTest.java | 16 +++++++--- .../src/cgeo/geocaching/geopoint/ViewportTest.java | 2 +- .../geocaching/sorting/DistanceComparatorTest.java | 2 +- 27 files changed, 36 insertions(+), 106 deletions(-) diff --git a/main/res/values-cs/strings.xml b/main/res/values-cs/strings.xml index 64fb827..b220580 100644 --- a/main/res/values-cs/strings.xml +++ b/main/res/values-cs/strings.xml @@ -121,7 +121,6 @@ Přeložit do angličtiny Při velkém množství textu může překládání selhat. OK - Komunikace nezačala Čtení přihlašovací obrazovky selhalo Nelze se připojit na Geocaching.com. Stránka může být nedostupná nebo je chyba v tvém připojení k Internetu Nejsou uloženy žádné přihlašovací údaje @@ -177,7 +176,6 @@ c:geo nemůže načíst data požadovaná k zalogování návštěvy. c:geo nemůže načíst data požadovaná k zalogování návštěvy. Zkouší to znovu. c:geo stále načítá data požadovaná k odeslání Logu. Prosím, počkej ještě chvíli. - c:geo nemůže odeslat Log, protože server neodpovídá. Zdá se, že tvůj Log nebyl odeslán. Prosím, zkontroluj to na Geocaching.com. Zdá se, že tvůj obrázek nebyl nahrán. Zkontroluj to, prosím, na Geocaching.com. c:geo zapomnělo hledanou adresu. diff --git a/main/res/values-da/strings.xml b/main/res/values-da/strings.xml index d784056..6b23d3d 100644 --- a/main/res/values-da/strings.xml +++ b/main/res/values-da/strings.xml @@ -53,7 +53,6 @@ Fjern Webcamfoto taget Ok - Forbindelse ikke oprettet Login-side kan ikke indlæses Kan ikke forbinde til Geocaching.com (server eller forbindelsesfejl?) Ingen login gemt diff --git a/main/res/values-de/strings.xml b/main/res/values-de/strings.xml index 062f475..86fc324 100644 --- a/main/res/values-de/strings.xml +++ b/main/res/values-de/strings.xml @@ -121,7 +121,6 @@ Übersetze in Englisch Die Übersetzung kann bei sehr langen Texten evtl. fehlschlagen. OK - Kommunikation nicht gestartet Parsing der Anmeldung gescheitert Verbindung zu Geocaching.com konnte nicht hergestellt werden (Server oder Verbindung inaktiv?) Verbindung zu Extremcaching.com konnte nicht hergestellt werden (Server oder Verbindung inaktiv?) @@ -178,7 +177,6 @@ c:geo konnte die benötigten Daten nicht laden, um den Besuch zu loggen. c:geo konnte die benötigten Daten nicht laden, um den Besuch zu loggen. Bitte erneut versuchen. c:geo lädt gerade die benötigten Daten. Bitte kurz warten. - c:geo konnte Log nicht senden, weil der Server nicht antwortete. c:geo konnte Log nicht absenden. Es scheint dass Ihr Log nicht hochgeladen werden konnte. Bitte prüfen Sie es auf Extremcaching.com nach. Es scheint dass Ihr Logfoto nicht hochgeladen werden konnte. Bitte prüfen Sie es auf Geocaching.com nach. diff --git a/main/res/values-es/strings.xml b/main/res/values-es/strings.xml index b502385..9287073 100644 --- a/main/res/values-es/strings.xml +++ b/main/res/values-es/strings.xml @@ -114,7 +114,6 @@ Descripción Redimensionamiento Aceptar - Comunicación no iniciada Error procesando la página de acceso Error conectando a Geocaching.com (¿servidor o conexión caidos?) No hay información de acceso guardada @@ -160,7 +159,6 @@ Lo siento, c:geo no puede cargar los datos necesarios para registrar la visita. Lo siento, c:geo no puede cargar los datos necesarios para registrar la visita. Intentándolo de nuevo. c:geo sigue cargando datos para registrar el escondite. Debes esperar un poquito mas. - Lo siento, c:geo no ha podido enviar el registro porque el servidor no responde. Lo siento, c:geo no ha podido enviar el registro. Lo siento, c:geo ha olvidado la dirección que buscabas. Lo siento, c:geo no puede procesar la latitud. diff --git a/main/res/values-fr/strings.xml b/main/res/values-fr/strings.xml index d99252a..8fdfd2a 100644 --- a/main/res/values-fr/strings.xml +++ b/main/res/values-fr/strings.xml @@ -121,7 +121,6 @@ Traduire en anglais La traduction d\'un texte trop long peut échouer. ok - Communication non démarrée Échec login lecture page Échec de la connexion à Geocaching.com (probléme de serveur ou de connexion?) Impossible de contacter Extremcaching.com. Le site web peut ne pas fonctionner ou votre connexion Internet peut être défectueuse. @@ -178,7 +177,6 @@ c:geo ne peut pas charger les données pour enregistrer votre visite. c:geo ne peut pas charger les données pour enregistrer votre visite. Essayer plus tard. c:geo est en train de charger les données pour enregistrer votre visite. Veuillez patienter encore un peu. - c:geo n\'a pas réussi à charger les données pour enregistrer votre visite car le serveur ne répond pas. Il semble que votre visite n\'a pas été enregistrée. Veuillez vérifier sur le site Geocaching.com. Il semble que votre visite n\'a pas été enregistrée. Veuillez vérifier sur le site Extremcaching.com. Il semble que l\'envoi de l\'image ait échoué. Veuillez vérifier sur le site Geocaching.com. diff --git a/main/res/values-hu/strings.xml b/main/res/values-hu/strings.xml index 0666d17..cfc4312 100644 --- a/main/res/values-hu/strings.xml +++ b/main/res/values-hu/strings.xml @@ -105,7 +105,6 @@ Fordítás angolra A fordítás sikertelen lehet hosszú szövegeknél. Ok - A kommunikáció nem indult el Hiba a belépőoldalon Hiba a kapcsolatban a geocaching.com-mal (a szerver vagy a kapcsolat nem működik?) Nincs mentett belépési adat @@ -157,7 +156,6 @@ Sajnáljuk, a c:geo nem tudja betölteni az adatokat amik a megtalálás bejelentéséhez szükségesek. Sajnáljuk, a c:geo nem tudja betölteni az adatokat amik a megtalálás bejelentéséhez szükségesek. Újrapróbálkozás. A c:geo még tölti az adatokat amik a megtalálás bejelentéséhez szükségesek. Kérlek várj egy kicsit tovább. - Sajnáljuk, a c:geo-nak nem sikerült hozzáadni a logot mert a szerver nem válaszol. Úgy tűnik a logot nem került hozzáadásra. Kérlek ellenőrizd a geocaching.com-on. Sajnáljuk, a c:geo elfelejtette a címet amit kerestél. Sajnáljuk, a c:geo nem tudja értelmezni a szélességet. diff --git a/main/res/values-it/strings.xml b/main/res/values-it/strings.xml index 90c9549..8f26c2c 100644 --- a/main/res/values-it/strings.xml +++ b/main/res/values-it/strings.xml @@ -121,7 +121,6 @@ Traduci in inglese La traduzione può fallire quando c\'è molto testo. OK - Comunicazione non avviata Errore analizzando la pagina di login Connessione fallita con Geocaching.com (server o connessione non disponibile?) Informazioni di login non memorizzate @@ -177,7 +176,6 @@ c:geo non riesce a caricare i dati necessari per salvare il log della tua visita. c:geo non riesce a caricare i dati necessari per salvare il log della tua visita. Nuovo tentativo in corso. c:geo sta ancora caricando i dati necessari per salvare il log. Per cortesia attendere ancora un pochino. - c:geo non è riuscito ad inviare il log perché il server non risponde. Sembra che il log non sia stato inviato. Prego verificare su Geocaching.com. Sembra che l\'immagine del tuo log non sia stata caricata. Si prega di controllare su Geocaching.com. c:geo ha dimenticato l\'indirizzo che vuoi trovare. diff --git a/main/res/values-ja/strings.xml b/main/res/values-ja/strings.xml index 2d79690..987391f 100644 --- a/main/res/values-ja/strings.xml +++ b/main/res/values-ja/strings.xml @@ -121,7 +121,6 @@ 英語に翻訳 文章が長すぎるので翻訳できないかもしれません。 OK - 通信を始めることができませんでした ログインページの解析に失敗しました Geocaching.comに接続できません。サーバが停止中かインターネットに未接続かもしれません。 ログイン情報が保存されていません @@ -177,7 +176,6 @@ ログを書くのに必要なデータをロードすることができません。 ログを書くのに必要なデータをロードすることができません。もう一度やってみてください。 ログを投稿するのに必要なデータをまだロード中です。もう少しお待ちください。 - サーバの反応がないのでログを投稿することができませんでした。 ログを投稿することができなかったようです。Geocaching.comで確認してください。 ログの添付画像がアップロードされなかったようです。Geocaching.comで確認してください。 c:geo forgot the address you tried to find. diff --git a/main/res/values-nb/strings.xml b/main/res/values-nb/strings.xml index 6eb61a2..d5af0cf 100644 --- a/main/res/values-nb/strings.xml +++ b/main/res/values-nb/strings.xml @@ -69,7 +69,6 @@ Logg Logg tekst Ok - Klarte ikke å koble til Klarte ikke å lese innloggings-siden Klarte ikke å koble til Geocaching.com (server eller tilkobling nede?) Ingen innloggings-opplysninger lagret @@ -103,7 +102,6 @@ Beklager, c:geo klarte ikke å laste den nødvendige informasjonen for å logge funnet. Beklager, c:geo klarte ikke å laste den nødvendige informasjonen for å logge funnet. c:geo prøver på nytt. c:geo laster fortsatt informasjonen som er nødvendig for å logge funnet. Vennligst vent. - Beklager, c:geo klarte ikke å logge funnet fordi serveren er nede. Beklager, c:geo klarte ikke å logge funnet. Beklager, c:geo glemte adressen du prøvde å finne. Beklager, c:geo klarte ikke å lese breddegraden. diff --git a/main/res/values-nl/strings.xml b/main/res/values-nl/strings.xml index 63e5f65..e1a4852 100644 --- a/main/res/values-nl/strings.xml +++ b/main/res/values-nl/strings.xml @@ -121,7 +121,6 @@ Vertaal naar engels Vertaling kan mislukken bij grote stukken tekst. Ok - Communicatie niet gestart Login pagina verwerken mislukt Verbinding met Geocaching.com mislukt (geen verbinding of server niet bereikbaar) Geen login informatie opgeslagen @@ -177,7 +176,6 @@ Sorry, c:geo kan data om bezoek te loggen niet laden. Sorry, c:geo kan data om bezoek te loggen niet laden. Bezig nogmaals te proberen. c:geo is nog steeds bezig om data te laden om bezoek te loggen. Graag nog even geduld. - Sorry, c:geo kon bezoek niet loggen omdat de server niet reageert. Sorry, het is c:geo niet gelukt het log te posten. Het lijkt erop dat uw log-foto is niet geüpload. Controleer het op Geocaching.com. Sorry, c:geo is het adres dat je zoekt vergeten. diff --git a/main/res/values-pl/strings.xml b/main/res/values-pl/strings.xml index 71340cc..90c17f7 100644 --- a/main/res/values-pl/strings.xml +++ b/main/res/values-pl/strings.xml @@ -121,7 +121,6 @@ Przetłumacz na angielski Tłumaczenie może nie udać się przy dużej ilości tekstu. OK - Komunikacja nie rozpoczeła się Nieudany odczyt strony logowania. Nieudane połączenie z Geocaching.com. Strona albo Twoje połączenie internetowe nie działa. Brak danych do logowania. @@ -177,7 +176,6 @@ c:geo nie może załadować danych wymaganych do wpisania wizyty. c:geo nie może załadować danych wymaganych do wpisania wizyty. Próbuję ponownie. Nadal trwa ładowanie danych wymaganych do wpisu w dzienniku. Proszę czekać… - c:geo nie może dokonać wpisu ponieważ serwer nie odpowiada. Wygląda na to, że Twój wpis nie został zapisany. Proszę sprawdź na Geocaching.com. Wygląda na to, że Twój obraz nie został dodany. Proszę sprawdź to na Geocaching.com. c:geo zapomniał adresu, którego szukałeś. diff --git a/main/res/values-pt/strings.xml b/main/res/values-pt/strings.xml index 67dbab2..0ecec98 100644 --- a/main/res/values-pt/strings.xml +++ b/main/res/values-pt/strings.xml @@ -121,7 +121,6 @@ Traduzir para inglês A tradução pode falhar se o texto fôr muito grande. Ok - Comunicação não iniciada Falha na análise da página de login Falha na ligação a Geocaching.com (servidor ou ligação em baixo?) Informação de login não gravada @@ -177,7 +176,6 @@ O c:geo não consegue carregar os dados necessários para registar a sua visita. O c:geo não consegue carregar os dados necessários para registar a sua visita. A tentar de novo. O c:geo ainda está a carregar dados necessários para publicar o registo. Por favor espere mais um pouco. - O c:geo falhou a publicação do registo porque o servidor não responde. O c:geo falhou a publicação do registo. Parece que a sua imagem de registo não foi enviada. Por favor verifique em Geocaching.com. O c:geo esqueceu o endereço que procura. diff --git a/main/res/values-sk/strings.xml b/main/res/values-sk/strings.xml index c8dbecc..278f5aa 100644 --- a/main/res/values-sk/strings.xml +++ b/main/res/values-sk/strings.xml @@ -105,7 +105,6 @@ Preložiť do angličtiny Pre veľké texty môže preklad zlyhať. Ok - Komunikácia nezačala. Zlyhalo čítanie prihlasovacej stránky. Zlyhalo pripojenie k serveru Geocaching.com (pripojenie, alebo server nefunkčný?) Nie sú uložené žiadne prihlasovacie údaje. @@ -157,7 +156,6 @@ c:geo nemôže načítať dáta potrebné pre zalogovanie návštevy. c:geo nemôže načítať dáta potrebné pre zalogovanie návštevy. Skúša to znovu. c:geo stále načítava dáta potrebné pre zalogovanie návštevy. Prosím, počkajte chvíľu. - c:geo nemohlo odoslať log, pretože server neodpovedá. c:geo nemohlo odoslať log. c:geo zabudlo adresu, ktorú sa pokúšate nájsť. c:geo nemôže dopočítať šírku. diff --git a/main/res/values-sl/strings.xml b/main/res/values-sl/strings.xml index ed67c5d..dce642c 100644 --- a/main/res/values-sl/strings.xml +++ b/main/res/values-sl/strings.xml @@ -121,7 +121,6 @@ Prevedi v: angleščino Prevod lahko spodleti zaradi velike količine besedila. OK - Komunikacija se ni začela Razčlenjevanje prijavne strani ni uspelo Ni bilo mogoče vzpostaviti povezave z Geocaching.com. Spletna stran ali vaša povezava z internetom je lahko nedosegljiva. Prijavni podatki niso shranjeni @@ -177,7 +176,6 @@ c:geo ni mogel naložiti podatkov za zapis obiska. c:geo ne more naložiti podatkov za zapis obiska. Ponovni poskus… c:geo še vedno nalaga podatke za zapis obiska. Prosimo počakajte še trenutek. - c:geo ni mogel objaviti zapisa, ker se strežnik ne odziva. Izgleda, kot da zapis ni bil objavljen. Preverite na Geocaching.com. Izgleda, kot da slika ni bila naložena. Preverite na Geocaching.com. c:geo je pozabil naslov, ki ste ga hoteli najti. diff --git a/main/res/values-sv/strings.xml b/main/res/values-sv/strings.xml index 155da29..6a8a200 100644 --- a/main/res/values-sv/strings.xml +++ b/main/res/values-sv/strings.xml @@ -121,7 +121,6 @@ Översätt till engelska Översättningen kan misslyckas med långa texter. Ok - Kommunikationen har inte påbörjats Misslyckades att analysera inloggningssidan Misslyckades att ansluta till Geocaching.com. Hemsidan kan vara ur funktion eller så är det problem med din anslutning till Internet. Inloggningsinformationen ej sparad @@ -177,7 +176,6 @@ Tyvärr kan c:geo inte ladda information som krävs för att logga ditt besök. Tyvärr kan c:geo inte ladda information som krävs för att logga ditt besök. Försöker igen. c:geo laddar fortfarande data för att kunna posta loggen. Vänligen vänta en stund till. - Tyvärr misslyckades c:geo att posta loggen eftersom servern inte svarade. Tyvärr misslyckades c:geo att posta loggen. Det verkar som om din bild till loggen inte kunde laddas upp. Vänligen kontrollera den på Geocaching.com. Tyvärr glömde c:geo adressen du sökte efter. diff --git a/main/res/values/strings.xml b/main/res/values/strings.xml index c4bda48..10f834d 100644 --- a/main/res/values/strings.xml +++ b/main/res/values/strings.xml @@ -145,7 +145,6 @@ OK - Communication not started Failed Login page parsing Unable to contact Geocaching.com. The website may be down or your internet connection not working. Unable to contact Extremcaching.com. The website may be down or your internet connection not working. @@ -202,7 +201,6 @@ c:geo can\'t load data required to log visit. c:geo can\'t load data required to log visit. Trying again. c:geo is still loading data required to post log. Please wait a little while longer. - c:geo failed to post log because server is not responding. It seems that your log was not posted. Please check it on Geocaching.com. It seems that your log was not posted. Please check it on Extremcaching.com. It seems that your log image was not uploaded. Please check it on Geocaching.com. diff --git a/main/src/cgeo/geocaching/CacheListActivity.java b/main/src/cgeo/geocaching/CacheListActivity.java index 9c7f692..3d62e7e 100644 --- a/main/src/cgeo/geocaching/CacheListActivity.java +++ b/main/src/cgeo/geocaching/CacheListActivity.java @@ -48,10 +48,12 @@ import cgeo.geocaching.utils.GeoDirHandler; import cgeo.geocaching.utils.Log; import ch.boye.httpclientandroidlib.HttpResponse; + import org.apache.commons.collections4.CollectionUtils; import org.apache.commons.collections4.ListUtils; import org.apache.commons.lang3.StringUtils; import org.eclipse.jdt.annotation.NonNull; + import rx.util.functions.Action1; import android.app.Activity; @@ -394,7 +396,7 @@ public class CacheListActivity extends AbstractListActivity implements FilteredA if (isInvokedFromAttachment()) { type = CacheListType.OFFLINE; if (coords == null) { - coords = new Geopoint(0.0, 0.0); + coords = Geopoint.ZERO; } } diff --git a/main/src/cgeo/geocaching/Geocache.java b/main/src/cgeo/geocaching/Geocache.java index 66d37bd..30ab55f 100644 --- a/main/src/cgeo/geocaching/Geocache.java +++ b/main/src/cgeo/geocaching/Geocache.java @@ -33,6 +33,7 @@ import cgeo.geocaching.utils.MatcherWrapper; import cgeo.geocaching.utils.UncertainProperty; import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; + import org.apache.commons.collections4.CollectionUtils; import org.apache.commons.collections4.Predicate; import org.apache.commons.lang3.BooleanUtils; @@ -1366,17 +1367,6 @@ public class Geocache implements ICache, IWaypoint { } /** - * Retrieve a given waypoint. - * - * @param index - * the index of the waypoint - * @return waypoint or null if index is out of range - */ - public Waypoint getWaypoint(final int index) { - return index >= 0 && index < waypoints.size() ? waypoints.get(index) : null; - } - - /** * Lookup a waypoint by its id. * * @param id diff --git a/main/src/cgeo/geocaching/enumerations/StatusCode.java b/main/src/cgeo/geocaching/enumerations/StatusCode.java index e1a1aaa..8bda371 100644 --- a/main/src/cgeo/geocaching/enumerations/StatusCode.java +++ b/main/src/cgeo/geocaching/enumerations/StatusCode.java @@ -6,7 +6,6 @@ import android.content.res.Resources; public enum StatusCode { - COMMUNICATION_NOT_STARTED(R.string.err_start), NO_ERROR(R.string.err_none), LOG_SAVED(R.string.info_log_saved), LOGIN_PARSE_ERROR(R.string.err_parse), @@ -24,7 +23,6 @@ public enum StatusCode { LOG_POST_ERROR(R.string.err_log_post_failed), LOG_POST_ERROR_EC(R.string.err_log_post_failed_ec), NO_LOG_TEXT(R.string.warn_log_text_fill), - NO_DATA_FROM_SERVER(R.string.err_log_failed_server), NOT_LOGGED_IN(R.string.init_login_popup_failed), LOGIMAGE_POST_ERROR(R.string.err_logimage_post_failed); diff --git a/main/src/cgeo/geocaching/geopoint/Geopoint.java b/main/src/cgeo/geocaching/geopoint/Geopoint.java index f21df01..e91a93d 100644 --- a/main/src/cgeo/geocaching/geopoint/Geopoint.java +++ b/main/src/cgeo/geocaching/geopoint/Geopoint.java @@ -4,6 +4,7 @@ import cgeo.geocaching.ICoordinates; import cgeo.geocaching.R; import org.apache.commons.lang3.StringUtils; +import org.eclipse.jdt.annotation.NonNull; import android.location.Location; import android.os.Build; @@ -11,10 +12,13 @@ import android.os.Parcel; import android.os.Parcelable; /** - * Abstraction of geographic point. + * Abstraction of geographic point. This class is immutable. */ public final class Geopoint implements ICoordinates, Parcelable { - public static final Geopoint ZERO = new Geopoint(0.0, 0.0); + /** + * Reusable default object + */ + public static final @NonNull Geopoint ZERO = new Geopoint(0.0, 0.0); private static final double DEG_TO_RAD = Math.PI / 180; private static final double RAD_TO_DEG = 180 / Math.PI; @@ -64,7 +68,6 @@ public final class Geopoint implements ICoordinates, Parcelable { * longitude string to parse * @throws Geopoint.ParseException * if any argument string cannot be parsed - * @see GeopointParser#parse(String, String) */ public Geopoint(final String latText, final String lonText) { this(GeopointParser.parseLatitude(latText), GeopointParser.parseLongitude(lonText)); @@ -273,19 +276,6 @@ public final class Geopoint implements ICoordinates, Parcelable { } /** - * Checks if given Geopoint is similar to this Geopoint with tolerance. - * - * @param gp - * Geopoint to check - * @param tolerance - * tolerance in km - * @return true if similar, false otherwise - */ - public boolean isEqualTo(Geopoint gp, double tolerance) { - return null != gp && distanceTo(gp) <= tolerance; - } - - /** * Returns formatted coordinates. * * @param format @@ -300,7 +290,7 @@ public final class Geopoint implements ICoordinates, Parcelable { /** * Returns formatted coordinates with default format. * Default format is decimalminutes, e.g. N 52° 36.123 E 010° 03.456 - * + * * @return formatted coordinates */ @Override @@ -365,7 +355,7 @@ public final class Geopoint implements ICoordinates, Parcelable { /** * Get longitude character (E or W). - * + * * @return */ public char getLonDir() { diff --git a/main/src/cgeo/geocaching/geopoint/GeopointParser.java b/main/src/cgeo/geocaching/geopoint/GeopointParser.java index c043d6f..120e02e 100644 --- a/main/src/cgeo/geocaching/geopoint/GeopointParser.java +++ b/main/src/cgeo/geocaching/geopoint/GeopointParser.java @@ -70,37 +70,11 @@ class GeopointParser { } /** - * Parses a pair of coordinates (latitude and longitude) out of a String. - * Accepts following formats and combinations of it: - * X DD - * X DD° - * X DD° MM - * X DD° MM.MMM - * X DD° MM SS - * - * as well as: - * DD.DDDDDDD - * - * Both . and , are accepted, also variable count of spaces (also 0) - * - * @param latitude - * the latitude string to parse - * @param longitude - * the longitude string to parse - * @return an Geopoint with parsed latitude and longitude - * @throws Geopoint.ParseException - * if lat or lon could not be parsed - */ - public static Geopoint parse(final String latitude, final String longitude) { - final double lat = parseLatitude(latitude); - final double lon = parseLongitude(longitude); - - return new Geopoint(lat, lon); - } - - /* - * (non JavaDoc) - * Helper for coordinates-parsing. + * Helper for coordinates-parsing + * + * @param text + * @param latlon + * @return */ private static ResultWrapper parseHelper(final String text, final LatLon latlon) { diff --git a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java index 93f50e1..651ff6e 100644 --- a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java +++ b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java @@ -54,7 +54,7 @@ public class CoordinatesInputDialog extends NoTitleDialog { } else if (geo != null && geo.getCoords() != null) { this.gp = geo.getCoords(); } else { - this.gp = new Geopoint(0.0, 0.0); + this.gp = Geopoint.ZERO; } } @@ -396,7 +396,7 @@ public class CoordinatesInputDialog extends NoTitleDialog { if (geo != null && geo.getCoords() != null) { gp = geo.getCoords(); } else { - gp = new Geopoint(0.0, 0.0); + gp = Geopoint.ZERO; } } } diff --git a/tests/src/cgeo/geocaching/CgeoApplicationTest.java b/tests/src/cgeo/geocaching/CgeoApplicationTest.java index edf7862..cd80889 100644 --- a/tests/src/cgeo/geocaching/CgeoApplicationTest.java +++ b/tests/src/cgeo/geocaching/CgeoApplicationTest.java @@ -366,7 +366,7 @@ public class CgeoApplicationTest extends CGeoTestCase { final Geocache cacheFromViewport = DataStore.loadCache(cache.getGeocode(), LoadFlags.LOAD_CACHE_OR_DB); Log.d("cgeoApplicationTest.testSearchByViewportNotLoggedIn: Coords expected = " + cache.getCoords()); Log.d("cgeoApplicationTest.testSearchByViewportNotLoggedIn: Coords actual = " + cacheFromViewport.getCoords()); - assertFalse(cache.getCoords().isEqualTo(cacheFromViewport.getCoords(), 1e-3)); + assertFalse(cache.getCoords().distanceTo(cacheFromViewport.getCoords()) <= 1e-3); // depending on the chosen strategy the coords can be reliable or not assertEquals(testStrategy == Strategy.DETAILED, cacheFromViewport.isReliableLatLon()); diff --git a/tests/src/cgeo/geocaching/connector/gc/GCParserTest.java b/tests/src/cgeo/geocaching/connector/gc/GCParserTest.java index 53c393b..1509563 100644 --- a/tests/src/cgeo/geocaching/connector/gc/GCParserTest.java +++ b/tests/src/cgeo/geocaching/connector/gc/GCParserTest.java @@ -178,9 +178,10 @@ public class GCParserTest extends AbstractResourceInstrumentationTestCase { cache.setPersonalNote(note); cache.setWaypoints(new ArrayList(), false); cache.parseWaypointsFromNote(); - assertEquals(expected.length, cache.getWaypoints().size()); + final List waypoints = cache.getWaypoints(); + assertEquals(expected.length, waypoints.size()); for (int i = 0; i < expected.length; i++) { - assertTrue(expected[i].equals(cache.getWaypoint(i).getCoords())); + assertTrue(expected[i].equals(waypoints.get(i).getCoords())); } } diff --git a/tests/src/cgeo/geocaching/geopoint/GeoPointParserTest.java b/tests/src/cgeo/geocaching/geopoint/GeoPointParserTest.java index 1de2e26..28dbe77 100644 --- a/tests/src/cgeo/geocaching/geopoint/GeoPointParserTest.java +++ b/tests/src/cgeo/geocaching/geopoint/GeoPointParserTest.java @@ -19,7 +19,13 @@ public class GeoPointParserTest extends AndroidTestCase { public static void testFullCoordinates() { final Geopoint goal = new Geopoint(refLatitude, refLongitude); - assertTrue(goal.isEqualTo(GeopointParser.parse("N 49° 56.031 | E 8° 38.564"), 1e-6)); + assertEquals(goal, GeopointParser.parse("N 49° 56.031 | E 8° 38.564"), 1e-6); + } + + private static void assertEquals(final Geopoint expected, Geopoint actual, double tolerance) { + assertNotNull(expected); + assertNotNull(actual); + assertTrue(expected.distanceTo(actual) <= tolerance); } public static void testCoordinateMissingPart() { @@ -48,18 +54,18 @@ public class GeoPointParserTest extends AndroidTestCase { public static void testVariousFormats() { final Geopoint goal1 = GeopointParser.parse("N 49° 43' 57\" | E 2 12' 35"); final Geopoint goal2 = GeopointParser.parse("N 49 43.95 E2°12.5833333333"); - assertTrue(goal1.isEqualTo(goal2, 1e-6)); + assertEquals(goal1, goal2, 1e-6); } public static void testParseOurOwnSeparator() { final Geopoint separator = GeopointParser.parse("N 49° 43' 57\"" + Formatter.SEPARATOR + "E 2 12' 35"); final Geopoint noSeparator = GeopointParser.parse("N 49 43.95 E2°12.5833333333"); - assertTrue(separator.isEqualTo(noSeparator, 1e-6)); + assertEquals(separator, noSeparator, 1e-6); } public static void testInSentence() { final Geopoint p1 = GeopointParser.parse("Station3: N51 21.523 / E07 02.680"); - final Geopoint p2 = GeopointParser.parse("N51 21.523", "E07 02.680"); + final Geopoint p2 = GeopointParser.parse("N51 21.523 E07 02.680"); assertNotNull(p1); assertNotNull(p2); assertEquals(p1, p2); @@ -106,6 +112,6 @@ public class GeoPointParserTest extends AndroidTestCase { } public static void testEquatorMeridian() { - assertEquals(new Geopoint(0, 0), GeopointParser.parse("00° 00.000 00° 00.000")); + assertEquals(Geopoint.ZERO, GeopointParser.parse("00° 00.000 00° 00.000")); } } diff --git a/tests/src/cgeo/geocaching/geopoint/ViewportTest.java b/tests/src/cgeo/geocaching/geopoint/ViewportTest.java index da7d58f..11f78ad 100644 --- a/tests/src/cgeo/geocaching/geopoint/ViewportTest.java +++ b/tests/src/cgeo/geocaching/geopoint/ViewportTest.java @@ -52,7 +52,7 @@ public class ViewportTest extends AndroidTestCase { public static void testInViewport() { assertFalse(vpRef.contains(new Geopoint(-2.0, -2.0))); assertFalse(vpRef.contains(new Geopoint(4.0, 4.0))); - assertTrue(vpRef.contains(new Geopoint(0.0, 0.0))); + assertTrue(vpRef.contains(Geopoint.ZERO)); assertTrue(vpRef.contains(new Geopoint(-1.0, -2.0))); assertTrue(vpRef.contains(new Geopoint(3.0, 4.0))); } diff --git a/tests/src/cgeo/geocaching/sorting/DistanceComparatorTest.java b/tests/src/cgeo/geocaching/sorting/DistanceComparatorTest.java index 2db3f59..0cfee49 100644 --- a/tests/src/cgeo/geocaching/sorting/DistanceComparatorTest.java +++ b/tests/src/cgeo/geocaching/sorting/DistanceComparatorTest.java @@ -20,7 +20,7 @@ public class DistanceComparatorTest extends AndroidTestCase { } caches.add(cache); } - Collections.sort(caches, new DistanceComparator(new Geopoint(0, 0), caches)); + Collections.sort(caches, new DistanceComparator(Geopoint.ZERO, caches)); } } -- cgit v1.1