Nesse método de refatoracao o ClientModel está fazendo algumas outras funções além do que deveria, pensei em duas delas:
ClienteModel está lidando diretamente com a lógica de assinaturas, o que poderia ser delegado a uma nova classe
ClienteModel está mexendo com lógicas do cashback em si, acredito que poderiam ser incorporadas funções dessa na classe cashback localizada na branch refactor/venda-model-cashback.
Para entender mais sobre o assunto recomendo ver o link abaixo:
Nesse método de refatoracao o ClientModel está fazendo algumas outras funções além do que deveria, pensei em duas delas:
Para entender mais sobre o assunto recomendo ver o link abaixo:
https://refactoring.guru/pt-br/extract-class