TurfMeasurement#length method cannot handle empty lists
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- java
- Ambito
- backend-api-design
Direzione di ricerca
Inizia da services-turf/src/main/java/com/mapbox/turf/TurfMeasurement.java, nei pressi del metodo length collegato, e riproduci il fallimento con una lista vuota. Controlla come il metodo gestisce le normali liste di punti e rivedi gli utilizzi vicini per stabilire il contratto previsto per gli input vuoti. Il lavoro è completato quando il comportamento è concordato, il caso della lista vuota non genera più un’eccezione imprevista e la copertura di regressione lo verifica.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The other day, I encountered what I regard as an issue in the following Code snippet: https://github.com/mapbox/mapbox-java/blob/4ae8b78746fb37b2673364bdd4c8702057f3e7f3/services-turf/src/main/java/com/mapbox/turf/TurfMeasurement.java#L227
The scenario is the following: We have a list of GPS points of which some or all are invalid which is denoted by the coordinates being (0 , 0) - which IS bad Design, but our Team cannot change this... 🙄. Thus, we filter out those points before passing it to the length method. We just encountered an IndexOutOfBoundsException when the list is empty because there is no length check on the list. I guess the fix here is relatively easy, but the question ist - what should be returned in this case. We built in a check before hand returning 0 in this particular case. Maybe even a checked exception might be a feasible idea.
- Lingua principale
- Java
- Stelle
- 438
- Fork
- 117
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di mapbox/mapbox-java
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
mapbox/mapbox-java#1640 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
mapbox/mapbox-java#1635 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
mapbox/mapbox-java#1619 ·
-
Mapbox-java incompatible with gson 2.13.0 and up - call to internal GSON classes that were renamed Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
mapbox/mapbox-java#1614 · 2 commenti · 1 reazione ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
mapbox/mapbox-java#1612 · 1 reazione ·