roanbrasil / clean-code-tdc

Não ficou claro para min o motivo do get(0) nessa linha de código.
Um outro ponto, o que aconteceria se o response.getComplexGroupList() estivesse vazio?
IMHO: O melhor seria se o ComplexDataStreamExample fosse mais rico e retornasse algo como getFirstGrouop

clean-code-tdc/src/main/java/com/tdc/cleancode/complex/after/ComplexMapperCleanCode.java

Line 22 in 27e6f07

    
           final List<Complex> complexList = response.getComplexGroupList().get(0).getComplexList();

O retorno já é um filtro é está dentro de um método privado, eu colocaria um nome mais explito o que ele faz.
Por exemplo:

.filter(byPaymentMethod(paymentMethodToSearch))

clean-code-tdc/src/main/java/com/tdc/cleancode/complex/after/ComplexMapperCleanCode.java

Line 26 in 27e6f07

.filter(complexFilterByPaymentMethod(paymentMethodToSearch))

Vc vazou o encapsulamento aqui. Acho que seria mais semântico adicionar o pagamento e lá dentro ele define o tipo.

 payment.add(debit);

clean-code-tdc/src/main/java/com/tdc/cleancode/complex/after/ComplexMapperCleanCode.java

Line 52 in 27e6f07

payment.getDebitList().add(debit);

Vc só utiliza esse método uma única vez, pq não mapea para retornar o que vc quer?

    final String paymentTypeDescription = getCurrentPaymentMethod(complex);

clean-code-tdc/src/main/java/com/tdc/cleancode/complex/after/ComplexMapperCleanCode.java

Line 37 in 27e6f07

    
           final String paymentTypeDescription = getCurrentPaymentMethod(complex).getAttributeValue();

Eu levaria essa lógica de construção toda para dentro do débito de alguma forma, por exemplo, com um builder

Debit debit = Debit.of(paymentTypeDescription, simpleValueListFromTarget);
payment.add(debit);

clean-code-tdc/src/main/java/com/tdc/cleancode/complex/after/ComplexMapperCleanCode.java

Lines 43 to 50 in 27e6f07

    
           final Debit debit = new Debit(); 
        
           debit.setPaymentTypeDescription(paymentTypeDescription); 
        
           simpleValueListFromTarget 
        
                   .forEach(simpleValue -> 
        
                       ValueType.findByName(simpleValue.getAttributeName()) 
        
                               .setDebit(debit, simpleValue.getAttributeValue()) 
        
                   );

ps: Complex mapper, confesso que senti falta do contexto, parece que é algo de pagamento, não? E se fosse PaymentDataStream? Talvez inseria um pouco de DDD e linguagem ubiqua.
Pq está muito complexo: Uma lista que mappea complexo, que retorno complexo, que tem filtro complexo.
Tá as vezes tem o simple :)

Done

	final Debit debit = new Debit();
	debit.setPaymentTypeDescription(paymentTypeDescription);

	simpleValueListFromTarget
	.forEach(simpleValue ->
	ValueType.findByName(simpleValue.getAttributeName())
	.setDebit(debit, simpleValue.getAttributeValue())
	);

Sugestões