TurfMeasurement#length method cannot handle empty lists
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Java
- Estrelas
- 438
- Forks
- 117
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
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.
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.
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.
Avaliação
- Stack de tecnologia
- java
- Domínio
- backend-api-design
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Precisa de esclarecimento
- Facilidade para iniciantes
- 35/100