VitoriaPreviato / Source-Code-Inspection

0 stars 0 forks source link

Iteração do método hasNext() da classe TrocoIterator incorreta #12

Closed VitoriaPreviato closed 8 years ago

VitoriaPreviato commented 8 years ago

Descrição: Na estrutura de loop for do método hasNext() da classe TrocoIterator, o i é incrementado de forma que no corpo do for ocorrem tentativas de acesso à posições que não existem no array, ou seja, fora de seu escopo. Classificação: Computação Severidade: Média

VitoriaPreviato commented 8 years ago

Issue resolvido. Correção feita: A variável i do for do método foi alterada, inicializando agora com valor 5 em vez de 6, e sendo decrementada conforme as iterações ocorrem em vez de incrementada. Dessa forma, todas as posições do array são acessadas, do final para o começo, ao invés de ocorrerem tentativas de acesso à posições fora do escopo do array.