TurfMeasurement#length method cannot handle empty lists
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Bug
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- java
- Domínio
- backend-api-design
Direção de pesquisa
Comece em services-turf/src/main/java/com/mapbox/turf/TurfMeasurement.java, próximo ao método length indicado no link, e reproduza a falha com uma lista vazia. Verifique como o método trata listas normais de pontos e revise os usos próximos para estabelecer o contrato pretendido para entradas vazias. Considera-se concluído quando o comportamento estiver acordado, o caso de lista vazia não lançar mais uma exceção inesperada e a cobertura de regressão verificar isso.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Java
- Estrelas
- 438
- Forks
- 117
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de mapbox/mapbox-java
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
mapbox/mapbox-java#1640 ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
mapbox/mapbox-java#1635 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 15/100
mapbox/mapbox-java#1619 ·
-
Mapbox-java incompatible with gson 2.13.0 and up - call to internal GSON classes that were renamedAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 48/100
mapbox/mapbox-java#1614 · 2 comentários · 1 reação ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 20/100
mapbox/mapbox-java#1612 · 1 reação ·
Todas as issues de mapbox/mapbox-java
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
liquid-java/liquidjava#321 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia
-
rename config parameterAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
OpenConext/OpenConext-access#1015 ·
Mantenedores costumam responder em até 1 dia
-
68000: FDBcc branch target is 2 bytes too low (uses inst_start+2 instead of the displacement word)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
NationalSecurityAgency/ghidra#9702 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
apache/rocketmq-dashboard#5358 ·
Mantenedores costumam responder em até 3 dias