Construtor da classe Troco com lógica desnecessária e algoritmo que não condiz com expectativas

VitoriaPreviato / Source-Code-Inspection

0 stars 0 forks source link

Construtor da classe Troco com lógica desnecessária e algoritmo que não condiz com expectativas #10

Closed VitoriaPreviato closed 8 years ago

VitoriaPreviato commented 8 years ago

Descrição: Os loops realizados funcionam da maneira errada e são desnecessários, e o método no geral não produz o resultado esperado. O algoritmo poderia ter sido implementado de uma maneira muito mais eficaz e sucinta. Classificação: Desempenho, Excesso, Dados Severidade: Alta

LucasPedroLP commented 8 years ago

Issue resolvido. Correção feita: O código foi totalmente refatorado, removendo os while desnecessários e os substituindo por cálculos simplificados. Além disso a variável count teve seu nome alterado para _quantnotas devido as mudanças no código.