Abordagem com Map
Já que o seu propósito é setar uma cor diferente para cada região, então:
private static final Map<String, String> regioes = new HashMap<>(27);
private static final Map<String, Color> cores = new HashMap<>(27);
static {
String[] norte = {"AM", "AP", "AC", "RO", "RR", "PA", "TO"};
String[] sul = {"PR", "SC", "RS"};
String[] sudeste = {"SP", "MG", "RJ", "ES"};
String[] nordeste = {"BA", "SE", "AL", "PE", "PB", "RN", "CE", "PI", "MA"};
String[] centroOeste = {"MS", "MT", "GO", "DF"};
for (String s : norte) regioes.put(s, "norte");
for (String s : sul) regioes.put(s, "sul");
for (String s : sudeste ) regioes.put(s, "sudeste");
for (String s : nordeste ) regioes.put(s, "nordeste");
for (String s : centroOeste ) regioes.put(s, "centro-oeste");
cores.put("norte", Color.RED);
cores.put("sul", Color.GREEN);
cores.put("centro-oeste", Color.YELLOW);
cores.put("nordeste", Color.BLUE);
cores.put("sudeste", Color.ORANGE);
}
private static Color corDoEstado(String sigla) {
String regiao = regioes.get(sigla);
return cores.get(regiao);
}
E então no seu código, você só tem que fazer isso:
algumaCoisa.setCor(corDoEstado(estado));
As vantagens dessa abordagem acima é que:
- O mapeamento é construído uma única vez no carregamento da classe.
- O mapeamento é reutilizável em outros locais, e você se livra de ter que repetir esse
switch
horroroso sempre que for lidar com estados.
Abordagem orientada a objetos
Entretanto, essa abordagem ainda tem os seus problemas. O principal problema é que tudo isso se trata de programação orientada a strings, por tratar essas coisas como strings e não como objetos que deveriam ser.
Assim, o ideal é fazer algo parecido com isso:
package com.example;
import java.awt.Color;
public enum RegiaoBrasileira {
SUL(Color.GREEN),
SUDESTE(Color.ORANGE),
CENTRO_OESTE(Color.YELLOW),
NORTE(Color.RED),
NORDESTE(Color.BLUE);
private final Color cor;
private RegiaoBrasileira(Color cor) {
this.cor = cor;
}
public Color getCor() {
return cor;
}
public static List<EstadoBrasileiro> getEstados() {
return LocalizaEstados.porRegiao(this);
}
}
package com.example;
import java.util.Locale;
import static com.example.RegiaoBrasileira.*;
import com.example.util.StringUtils;
public enum EstadoBrasileiro {
RIO_GRANDE_DO_SUL("RS", SUL),
SANTA_CATARINA("SC", SUL),
PARANÁ("PR", SUL),
SÃO_PAULO("SP", SUDESTE),
RIO_DE_JANEIRO("RJ", SUDESTE),
MINAS_GERAIS("MG", SUDESTE),
ESPÍRITO_SANTO("ES", SUDESTE),
BAHIA("BA", NORDESTE),
SERGIPE("SE", NORDESTE),
ALAGOAS("AL", NORDESTE),
PERNAMBUCO("PE", NORDESTE),
PARAÍBA("PB", NORDESTE),
RIO_GRANDE_DO_NORTE("RN", NORDESTE),
CEARÁ("CE", NORDESTE),
PIAUÍ("PI", NORDESTE),
MARANHÃO("MA", NORDESTE),
PARÁ("PA", NORTE),
AMAPÁ("AP", NORTE),
ACRE("AC", NORTE),
AMAZONAS("AM", NORTE),
RONDÔNIA("RO", NORTE),
RORAIMA("RR", NORTE),
TOCANTINS("TO", NORTE),
MATO_GROSSO("MT", CENTRO_OESTE),
MATO_GROSSO_DO_SUL("MS", CENTRO_OESTE),
GOIÁS("GO", CENTRO_OESTE),
DISTRITO_FEDERAL("DF", CENTRO_OESTE),
private final String nome;
private final String sigla;
private final RegiaoBrasileira regiao;
private EstadoBrasileiro(String sigla, RegiaoBrasileira regiao) {
this.sigla = sigla;
this.regiao = regiao;
this.nome = StringUtils.toTitleCase(name().toLowerCase(Locale.ROOT).replace("_", " "));
}
public String getNome() {
return nome;
}
public String getSigla() {
return sigla;
}
public RegiaoBrasileira getRegiao() {
return regiao;
}
}
package com.example;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
class LocalizaEstados {
private static final Map<String, EstadoBrasileiro> SIGLAS_POR_ESTADO = new HashMap<>(27);
private static final Map<RegiaoBrasileira, List<EstadoBrasileiro>> ESTADO_POR_REGIAO = new HashMap<>(27);
static {
for (RegiaoBrasileira s : RegiaoBrasileira.values()) {
ESTADO_POR_REGIAO.put(s, new ArrayList<>(9));
}
for (EstadoBrasileiro s : EstadoBrasileiro.values()) {
SIGLAS_POR_ESTADO.put(s.sigla, s);
ESTADO_POR_REGIAO.get(s.getRegiao()).add(s);
}
for (RegiaoBrasileira s : RegiaoBrasileira.values()) {
ESTADO_POR_REGIAO.put(s, Collections.unmodifiableList(ESTADO_POR_REGIAO.get(s));
}
}
private LocalizaEstados() {}
static EstadoBrasileiro porSigla(String sigla) {
return SIGLAS_POR_ESTADO.get(sigla);
}
static List<EstadoBrasileiro> porRegiao(RegiaoBrasileira regiao) {
return ESTADO_POR_REGIAO.get(regiao);
}
}
package com.example.util;
class StringUtils {
// Fonte: https://stackoverflow.com/a/1086134/540552
public static String toTitleCase(String input) {
StringBuilder titleCase = new StringBuilder();
boolean nextTitleCase = true;
for (char c : input.toCharArray()) {
if (Character.isSpaceChar(c)) {
nextTitleCase = true;
} else if (nextTitleCase) {
c = Character.toTitleCase(c);
nextTitleCase = false;
}
titleCase.append(c);
}
return titleCase.toString();
}
}
Pode parecer um grande trabalho, mas:
Você separa os conceitos de estados e regiões brasileiras de outras partes do código.
Tudo isso é reutilizável.
Se precisar alterar conceitos de estados e regiões brasileiras (por exemplo, acrescentar o conceito de capitais de cada estado), não é difícil alterar o enum.
Conceitos de estados, regiões brasileiras e cores correspondentes deixam de poluir outras partes do código.
Conclusão
Utilizar o switch
é uma coisa horrorosa. Ele tende a poluir o código deixando-o confuso e difícil de ser mantido. O polimorfismo e o tabelamento adequado de dados em estruturas de dados adequadas são alternativas melhores a ele na maioria dos casos.
É verdade que isso tudo parece ser mais complicado do que usar um switch
. O problema é que raramente você acaba usando apenas um e sim que você acaba usando dois, três, cinco, vinte vezes esses switch
es espalhados em um monte de lugares do código.
Por isso, atente-se sempre ao caso do problema XY.
switch
é uma má escolha. Mas para saber o que seria melhor no seu caso, você teria que dizer o que você quer fazer com esses estados.enum Regiao
comNORDESTE
,SUDESTE
, etc). Me parece mais limpo tratar 5 regiões do que umswitch
com todos os estados + DF.if
não tornaria a execução mais lenta que a funçãoswitch
? Oswitch
possui a possibilidade de frear (break) o código caso o valor corresponda, já oif
causaria um processamento desnecessário, pois se encontrei o que preciso não teria necessidade de continuar a processar. Ou oif
também possui um break internamente?