VitoriaPreviato / Source-Code-Inspection

0 stars 0 forks source link

Posição 1 do vetor papeisMoeda no construtor da classe Troco resetado erroneamente #9

Closed VitoriaPreviato closed 8 years ago

VitoriaPreviato commented 8 years ago

Descrição: No construtor da classe Troco são definidas as quantidades de cada papel moeda de valor diferente, porém a mesma posição do vetor (no caso a posição 1) é acessada para definir tanto as quantidades de papelMoeda 5 e papelMoeda 2, uma sobrescrevendo a outra. A quantidade de papelMoeda 2 deveria ser inserida na posição 0 do vetor, e como isso nao acontece, a primeira posição do mesmo é null. Classificação: Computação, Dados Severidade: Média

LucasPedroLP commented 8 years ago

Issue resolvido. Correção feita: o índice de papeisMoeda foi alterado na linha 40 de um para zero de forma que a quantidade de notas de cinco não será mais sobrescrita.