Gostaria de saber como otimizar uma grande cadeia de ifs de forma que o código fique mais eficiente e mais legível.
Minha aplicação Android usa a versão 2.3.3 (API 10), nesta aplicação tenho uma activity responsável por exibir as informações da conta do usuário, onde o mesmo pode editar essas informações e atualizar. Essa activity contém 8 campos ao lado de campo tem 1 botão "Editar" e um botão "OK" escondido, se o botão "Editar" for clicado então a aplicação esconde o "Editar" e exibe o "OK" no mesmo lugar. Dessa forma tenho ao total 16 botões, 8 "Editar" e 8 "OK". Que deixaram meu método onClick(View v) com 16 ifs.
Como posso melhorar isso e deixar mais eficiente e mais legível? Sei que posso usar o switch que seria quase a mesma coisa.
O código abaixo tem o método com apenas 2 dos 16 botões para simplificar, já que os outros botões seguem a mesma lógica.
public void onClick(View v) {
// TODO Auto-generated method stub
if(v.getId() == R.id.btneditar1) {
camponome.setSelectAllOnFocus(true);
camponome.setFocusable(true);
camponome.setFocusableInTouchMode(true);
camponome.requestFocus();
imm.showSoftInput(camponome, InputMethodManager.SHOW_FORCED);
btneditar1.setVisibility(View.GONE);
btnok1.setVisibility(View.VISIBLE);
}
if(v.getId() == R.id.btnok1) {
CampoValido = ValidaCampo.validateNotNull(camponome, "Campos sem dados são inválidos!");
if(CampoValido) {
CampoValido = ValidaCampo.validateNotEqual(camponome, bnome);
if(CampoValido) {
final String nome = camponome.getText().toString().trim();
atualizar.execute(usuario, nome, null, null, null, null, null, null);
txtnomebd.setText(nome);
}
camponome.setFocusable(false);
camponome.setFocusableInTouchMode(false);
camponome.requestFocus();
btnok1.setVisibility(View.GONE);
btneditar1.setVisibility(View.VISIBLE);
}
}
if(v.getId() == R.id.btneditar2) {
camposobrenome.setSelectAllOnFocus(true);
camposobrenome.setFocusable(true);
camposobrenome.setFocusableInTouchMode(true);
camposobrenome.requestFocus();
imm.showSoftInput(camposobrenome, InputMethodManager.SHOW_FORCED);
btneditar2.setVisibility(View.GONE);
btnok2.setVisibility(View.VISIBLE);
}
if(v.getId() == R.id.btnok2) {
CampoValido = ValidaCampo.validateNotNull(camposobrenome, "Campos sem dados são inválidos!");
if(CampoValido) {
CampoValido = ValidaCampo.validateNotEqual(camposobrenome, bsobrenome);
if(CampoValido) {
final String sobrenome = camposobrenome.getText().toString().trim();
atualizar.execute(usuario, null, sobrenome, null, null, null, null, null);
txtsobrenomebd.setText(sobrenome);
}
camposobrenome.setFocusable(false);
camposobrenome.setFocusableInTouchMode(false);
camposobrenome.requestFocus();
btnok2.setVisibility(View.GONE);
btneditar2.setVisibility(View.VISIBLE);
}
}
}
findViewById(id).setOnClickListener(new View.OnClickListener() { ... seu código ...} );
, que eu considero mais legível. De qualquer forma, seria um grande avanço separar esses corpos dos ifs em suas devidas funções privadas adequadamente nomeadas. Quanto ao desempenho, a não ser que você tenha notado alguma lentidão nessa parte do código, não se preocupe com isso precipitadamente.