O SonarLint para Eclipse, acusa o erro:
Refactor this method to reduce its Cognitive Complexity from 64 to the 15 allowed.
Reescreva esse metodo para reduzir sua complexidade cognitiva de 64 para 15 permitidos
Ou seja, meu método tem muitos if
s e else
s, muitos pontos de decisão e eu preciso diminuir isso.
Em um método de com regras de negócio isso até que é "fácil", reescrevo o método separando em vários outros métodos pequenos e "JUnitizáveis".
O problema é o método equals()
, possuo alguns objetos com vários campos, e o método equals()
precisa verificar a similaridade de todos esses campos, cheque de null
, de equals
e ==
para primitivos.
Esse meu equals()
já é um método com contexto simples, ele só faz uma tarefa, já é testável por JUnit, porém possui diversos pontos de decisão dentro.
O que fazer nesse caso? Criar vários métodos, um para testar null
, um para testar equals()
, um para testar primitivos e juntar todos no equals()
principal? Tem alguma abordagem melhor?
OBS: Sei que posso juntar vários if
s em uma única linha, mas isso atrapalha muito a visualização por um ser humano e estou tentando evitar essa prática.
Exemplo, para essa classe o SonarLint está acusando que ela possui complexidade de 34
:
package testes;
import java.math.BigDecimal;
public class EntidadeGrande {
private int codUser;
private String descUser;
private long idUser;
private BigDecimal valueX;
private int scoreUser;
private int scoreStore;
private int cdLogo;
private String nmMother;
private String sgEstado;
private String dsBandeira;
private String codCupom;
/* UM MONTE DE GET E SET AQUI, HASHCODE TAMBÉM */
@Override
public boolean equals(Object o) {
if (o == null || o.getClass() != getClass()) {
return false;
}
final EntidadeGrande e = (EntidadeGrande) o;
if (getCodUser() != e.getCodUser()
|| getIdUser() != e.getIdUser()
|| getScoreUser() != e.getScoreUser()
|| getScoreStore() != e.getScoreStore()
|| getCdLogo() != e.getCdLogo()) {
return false;
}
if (getValueX() == null && e.getValueX() != null
|| getValueX() != null && e.getValueX() == null) {
return false;
}
if (!getValueX().equals(e.getValueX())) {
return false;
}
if (getDescUser() != null && e.getDescUser() == null
|| getDescUser() == null && e.getDescUser() != null) {
return false;
}
if (getNmMother() != null && e.getNmMother() == null
|| getNmMother() == null && e.getNmMother() != null) {
return false;
}
if (getSgEstado() != null && e.getSgEstado() == null
|| getSgEstado() == null && e.getSgEstado() != null) {
return false;
}
if (getDsBandeira() != null && e.getDsBandeira() == null
|| getDsBandeira() == null && e.getDsBandeira() != null) {
return false;
}
if (getCodCupom() != null && e.getCodCupom() == null
|| getCodCupom() == null && e.getCodCupom() != null) {
return false;
}
if (!getDescUser().equals(e.getDescUser())) {
return false;
}
if (!getNmMother().equals(e.getNmMother())) {
return false;
}
if (!getSgEstado().equals(e.getSgEstado())) {
return false;
}
if (!getDsBandeira().equals(e.getDsBandeira())) {
return false;
}
if (!getCodCupom().equals(e.getCodCupom())) {
return false;
}
return true;
}
}