Correção de bug em CentavosInputFormatter

[ClodoaldoRibeiro]

No description provided.

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (4c8fe3f) 89.00% compared to head (2be6552) 89.02%.
Report is 1 commits behind head on master.

❗ Current head 2be6552 differs from pull request most recent head 8fab546. Consider uploading reports for the commit 8fab546 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
+ Coverage   89.00%   89.02%   +0.01%     
==========================================
  Files          29       29              
  Lines         582      583       +1     
==========================================
+ Hits          518      519       +1     
  Misses         64       64              

Files	Coverage Δ
lib/src/formatters/centavos_input_formatter.dart	88.88% <100.00%> (+0.25%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rubensdemelo]

Obrigado pelo PR, vou aprova-lo. Pode me detalhar quando este erro ocorre? Pois quero evoluir os testes unitários e detectar este tipo de situação.

[ClodoaldoRibeiro]

Sim, o erro ocorre quando o usuário usa esse formatador e o primeiro numero digitado for o zero. Com isso ele assume a formatação. Ex: 0,
Nas analises eu verifiquei que ele estaca tentando fazer um parse de 0 para inteirom retornando null, que foi nosso caso que foi tratado.

[rubensdemelo]

Mas este é o conportamento esperado. Como o usuario vai digitar, por exempli, R$ 0,09 ?

[ClodoaldoRibeiro]

Nese caso, ele digite 9, o 9 vai para o final como o último elemento, ficando 0,09. Ele não precisa digitir os zeros.

[rubensdemelo]

Beleza. Eu vou testar com calma e amanha ja faço o merge quando estiver tudo certo. Muito obrigado

[rubensdemelo]

Nese caso, ele digite 9, o 9 vai para o final como o último elemento, ficando 0,09. Ele não precisa digitir os zeros.

Neste ponto, quando o usuário digita 0, a formatação nao deveria ocorrer?

[ClodoaldoRibeiro]

Neste ponto, quando o usuário digita 0, a formatação nao deveria ocorrer?

Também podemos usar essa abordagem, mas hoje quando o primeiro número digitado for zero, nenunha formatação está sendo realizada.

// apaga o campo quando os valores foram zero.
if (numero == 0 && int.tryParse(centsValue) == 0) {
return const TextEditingValue(
text: "",
selection: TextSelection.collapsed(offset: 0),
);
}

[rubensdemelo]

Neste ponto, quando o usuário digita 0, a formatação nao deveria ocorrer?

Também podemos usar essa abordagem, mas hoje quando o primeiro número digitado for zero, nenunha formatação está sendo realizada.

// apaga o campo quando os valores foram zero. if (numero == 0 && int.tryParse(centsValue) == 0) { return const TextEditingValue( text: "", selection: TextSelection.collapsed(offset: 0), ); }

Mas no exemplo acima, todos os numeros digitados foram 0. Ainda nao ficou claro sobre esta situacao.

[ClodoaldoRibeiro]

Lembrando que o estado inicial do campo é sem formatação, apenas o campo vazio, @rubensdemelo da forma que está hoje, quando o primeiro número digitado for o zero, a formatação vai ficar conforme a imagem ou exemplo: R$ 0, ou 0,. Eu entendo que esse não seja uma formatação correta, R$ 0, não deveria ser válida.

O ajuste que realizei, caso o campo esteja vazio, sem nada para formatar, e o primeiro número digitado for o zero, então não realizo formatação alguma, ficando o campo como está, no estado de vazio.

Acredito que você esteja se questionando por que não é apresentado a formtação 0,00 ou R$ 0,00 caso o primeiro número digitado seja zero, mas não vejo muito sentido fazer uma foramtação assim. Esse deveria ser o estado inicial do campo, que hoje é mostrado como vazio, sendo semelhante aos campos do sistena bancários.

[rubensdemelo]

@ClodoaldoRibeiro

Entendo o seu ponto. Mas o comportamento de hoje, possibilita a formatação para o valores em Centavos, como R$ 0,09.

Eu vou criar uma branch nova com essa mudança e fazer os testes para visualizar melhor este caso.

[ClodoaldoRibeiro]

Beleza @rubensdemelo, dá uma olhada na ramificação que te mandei. Ela também permite fazer essa formtação.

[rubensdemelo]

[rubensdemelo]

Em ambos os casos, digitando qualquer valor maior que 0, a formatação esta correta:

Com o formatter atual, se o valor digitado for 0:

Com o seu PR, se o valor digitado for 0:

Lembro-me que a implementacao deste comportamento foi para seguir o "padrao" de alguns bancos, como BB, Itau e Nubank. Pois o formatter para centavos esta relacionado com valores monetarios.

Quando o usuario digitar 0 no teclado, este eh o valor que deve ser exibido na tela. O feedback real do que o usuario fez: Digitou 0.

Exemplo no app do BB, ao digital 9:

Exemplo no app do BB, ao digital 0:

Entendo que nao existe certo e errado neste caso mas acredito que seguir os padroes de outros aplicativos, facilita a experiencia do usuario.

[ClodoaldoRibeiro]

Verdade, vou ajustar esse comportamento para ficar de forma semelhante.