Code smells são um dos maiores vilões nos nosso código, onde muita vezes dificultam o entendimento do código, proporcionam armadilhas nas implementações e frequentemente ocasionam bugs em produção, com isso, essa apresentação tem como objetivo mostrar os benefícios de produzir códigos limpos através de boas práticas como SOLID, clean code, entre outras, que promovem hábitos de desenvolvimento para não cairmos nessas armadilhas causadas pelos code smells.

1. COMBATENDO CODE SMELLS EM
APLICAÇÕES JAVA
Trilha Java
Emmanuel Neri
@emmanuelnerii

2. EMMANUEL NERI
‣ Mestre em Desenvolvimento de Tecnologia
‣ Desenvolvedor (Java) desde 2010
‣ Professor na FACEC
‣ Developer na Jaya Tech

3. CODE SMELLS
CODE SMELLS

4. CULTURAL
Reuso
Código limpo
Refactoring
SOLID

5. PRATICAR

6. POR ONDE COMEÇAR?

7. LEGIBILIDADE
‣ Nome claros
‣ Nome que revelem seu propósito
‣ Nomes pronunciáveis
‣ Nomes passíveis de busca

8. LEITURA DO CÓDIGO
public BigDecimal calculate(List<Order> list) {
BigDecimal value = BigDecimal.ZERO;
for (Order x : list) {
for (OrderItem y : x.getItems()) {
if (y.getValue() != null) {
value = value.add(y.getValue());
}
}
}
return value.divide(BigDecimal.valueOf(list.size()));
}

9. LEITURA DO CÓDIGO
public BigDecimal calculate(List<Order> list) {
BigDecimal value = BigDecimal.ZERO;
for (Order x : list) {
for (OrderItem y : x.getItems()) {
if (y.getValue() != null) {
value = value.add(y.getValue());
}
}
}
return value.divide(BigDecimal.valueOf(list.size()));

10. LEITURA DO CÓDIGO
public BigDecimal average(List<Order> orders) {
BigDecimal totalValue = BigDecimal.ZERO;
for (Order order : orders) {
for (OrderItem orderItem : order.getItems()) {
if (orderItem.getValue() != null) {
totalValue = totalValue.add(orderItem.getValue());
}
}
}
BigDecimal orderQuantity = BigDecimal.valueOf(orders.size());
return totalValue.divide(orderQuantity);
}

11. LEITURA DO CÓDIGO
public BigDecimal average(List<Order> orders) {
final BigDecimal totalValue = orders.stream()
.map(Order::getTotalValue)
.reduce(BigDecimal.ZERO, BigDecimal::add);
final BigDecimal orderQuantity = BigDecimal
.valueOf(orders.size());
return totalValue.divide(orderQuantity);
}

12. DATA CLASS
public class Company {
private String name;
private String document;
public String getName() {
return name;
}
public void setName(String name) {
this.name = name;
}
public String getDocument() {
return document;
}
public void setDocument(String document) {
this.document = document;
}
}

13. DATA CLASS
@Data
public class Company {
private String name;
private String document;
@Getter
@Setter
public class Company {
private String name;
private String document;

14. DATA CLASS
public class Company {
private String name;
private String document;
private Company parent;
private Type type = Type.MASTER;
public boolean validate() {
return validateName() && validateDocument();
}
private boolean validateName() {}
private boolean validateDocument() {}
public void addParentCompany(final Company company) {
this.parent = company;
this.type = Type.BRANCH;
}
public void removeParent() {
this.parent = null;
this.type = Type.MASTER;
}

15. DATA CLASS
@Value
public class CompanyDTO {
private String name;
private String document;
}
public class CompanyDTO {
private final String name;
private final String document;
public CompanyDTO(String name, String document) {…}
public String getName() {…}
public String getDocument() {…}
}

16. TEMPORARY FIELD
public class Account {
private BigDecimal value;
@Transient
private BigDecimal tax;
public BigDecimal getValue() {
return value.multiply(tax);
}
public void setTax(BigDecimal tax) {
this.tax = tax;
}
}

17. TEMPORARY FIELD
public class Account {
private BigDecimal value;
private BigDecimal tax;
public Account(BigDecimal value, BigDecimal tax) {
this.value = value;
this.tax = tax;
}
public BigDecimal getValue() {
return value.multiply(tax);
}
}
public BigDecimal calculateValue(BigDecimal tax) {
return value.multiply(tax);
}

18. LONG PARAMETER LIST
public List<Account> find(LocalDate initDate, LocalDate endDate,
String clienteName, String identifier,
String description) {
}
public List<Account> find(AccountSearchDTO accountSearch) {
}

19. ‣ Divergent change
‣ Shotgun Surgery
CHANGE PREVENTERS

20. OPEN/CLOSE PRINCIPLE
"Software entities (classes, modules, functions, etc.) should be
open for extension, but closed for modification"
public BigDecimal getValue() {
if(orderType == OrderType.CALL) {
return unitValue.multiply(duration);
} else if(orderType == OrderType.DATA) {
return unitValue.multiply(bytes);
} else {
return unitValue.multiply(quantity);
}
}

21. public enum OrderType {
CALL {
@Override
public BigDecimal calcularValorTotal(Order order) {
return order.getUnitValue() * order.getDuracao();
}
},
DATA {
@Override
public BigDecimal calcularValorTotal(Order order) {
return order.getUnitValue() * order.getBytes();
}
},
…
public abstract BigDecimal calculate(Order order);
}
public BigDecimal getTotal() {
return orderType.calculate(this);
}
OPEN/CLOSE PRINCIPLE

22. LARGE CLASS / LONG METHOD

23. DUPLICATION
public class OrderService {
public BigDecimal discount(Order order) {
....
}
}
public class OrderItemService {
public BigDecimal discount(OrderItem item) {
....
}
}
public class SalesOrder {
public BigDecimal discount(Order order) {
....
}
}

24. OUTROS
‣ Inappropriate Intimacy
‣ Feature envy
‣ Lazy class
‣ Switch Statements
‣ Refused bequest
‣ Data clumps
‣ Dead code
‣ ….

25. COMBATENDO
SOLID
DRY
KISS
YAGNI
- Single responsibility Principle
- Open / Close Principle
- Liskov Substitution Principle
- Interface Segregation Principle
- Dependency Inversion Principle
- Don’t Repeat Yourself
- Keep it small and simple
- You Ain't Gonna Need
Clean code
Refactoring
- Extract method
- Extract class
- …

26. FERRAMENTAS
‣ SonarQube
‣ FindBugs
‣ IDEs

27. LEITURA

28. “VOCÊ SEMPRE DEVE DEIXAR O LUGAR
MAIS LIMPO DO QUE ENCONTROU”
emmanuelnerisouza@gmail.com
@emmanuelnerii
www.linkedin.com/in/emmanuelnerisouza
www.emmanuelneri.com.br
OBRIGADO!