|
public Iterator<Integer> getTroco() { |
|
return null; |
|
} |
Descrição:
O método getTroco() retorna null em vez de um iterador vazio, o que pode induzir a NullPointerException quando o chamador tentar iterar sobre o retorno.
Impacto:
- Introduz risco de falhas em tempo de execução.
- Viola boas práticas de design: métodos que retornam coleções devem preferir retornar coleções vazias em vez de null.
Ações sugeridas:
- Retornar Collections.emptyIterator() quando não houver troco.
- Documentar o comportamento esperado no método.
Source-Code-Inspection/Source Code Inspection/src/br/calebe/ticketmachine/core/TicketMachine.java
Lines 39 to 41 in 56b0f5f
Descrição:
O método getTroco() retorna null em vez de um iterador vazio, o que pode induzir a NullPointerException quando o chamador tentar iterar sobre o retorno.
Impacto:
Ações sugeridas: