Publicado em: 14/04/2011 10:50:37

Bom dia pessoal.

É o seguinte, fiz aquele joguinho do Einstein em java.
Ultilizando o ItemListener para acionar um evento, tenho o seguinte código:
dono[x].addItemListener( new ItemListener() { public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) Select(); Confere(); } }); bebida[x].addItemListener( new ItemListener() { public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) Select(); Confere(); } });

E mais 3 outros JComboBox.

Minha dúvida é: tem como "enxugar" mais o código?
Tipo, vemos que o ItemListener é igual para todos, dá pra botar em um metodo para quando for adicionar no JComboBox ele seja acionado? ou algo parecido?

espero que dê pra entender

PS: dono e bebida são JComboBox

Publicado em: 14/04/2011 12:01:24

fabricioluna

Bom dia pessoal.

É o seguinte, fiz aquele joguinho do Einstein em java.
Ultilizando o ItemListener para acionar um evento, tenho o seguinte código:
dono[x].addItemListener( new ItemListener() { public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) Select(); Confere(); } }); bebida[x].addItemListener( new ItemListener() { public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) Select(); Confere(); } });

E mais 3 outros JComboBox.

Minha dúvida é: tem como "enxugar" mais o código?
Tipo, vemos que o ItemListener é igual para todos, dá pra botar em um metodo para quando for adicionar no JComboBox ele seja acionado? ou algo parecido?

espero que dê pra entender

PS: dono e bebida são JComboBox

O Listener é um simples objeto - use apenas uma instância dele com uma variável.
Mais ou menos assim (ignorei os [x]):

JComboBox dono; JComboBox bebida; ... ItemListener listener = new ItemListener() { @Override public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) { Select(); } Confere(); } }; dono.addItemListener(listener); bebida.addItemListener(listener);
[[]]
_________________
  Nome real: Carlos F. Heuberger

Removeram os meus direitos de administrador e moderador - sem aviso, pela segunda vez - contate o ombudsman (?), a equipejavafree ou a "alta gerência" se necessário - Que pena... que terminou dessa maneira!

"The mod javafree deserves, but not the one it needs right now."
--------------------
Não leio nem respondo MPs!
This posting is provided AS IS with no warranties and confers no rights.

Publicado em: 14/04/2011 12:01:24

acabei esquecendo outra possibilidade (que não é minha preferida):
faz a classe onde tem esse código implementar o listener:
public class SuaClasse implements ItemListener { private void umMetodo() { ... dono.addItemListener(this); bebida.addItemListener(this); ... } @Override public void itemStateChanged(ItemEvent e) { if(e.getStateChange() == ItemEvent.SELECTED) { Select(); } Confere(); } }
[[]]

_________________
  Nome real: Carlos F. Heuberger

Removeram os meus direitos de administrador e moderador - sem aviso, pela segunda vez - contate o ombudsman (?), a equipejavafree ou a "alta gerência" se necessário - Que pena... que terminou dessa maneira!

"The mod javafree deserves, but not the one it needs right now."
--------------------
Não leio nem respondo MPs!
This posting is provided AS IS with no warranties and confers no rights.

Publicado em: 14/04/2011 10:50:37

Brigadãaaaoo!!
Resolveu direitinho!

agora tenho outra dúvida:

Dependendo da escolha, mudará a cor do painel em que o JComboBox foi inserido.

Gostaria de saber se dá pra fazer a mesma coisa que foi feita no código que postei no inicio?

Tenho esse código:

casa[0].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[0].setBackground(cores[casa[0].getSelectedIndex()]); Select(); Confere(); } }); casa[1].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[1].setBackground(cores[casa[1].getSelectedIndex()]); Select(); Confere(); } }); casa[2].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[2].setBackground(cores[casa[2].getSelectedIndex()]); Select(); Confere(); } }); casa[3].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[3].setBackground(cores[casa[3].getSelectedIndex()]); Select(); Confere(); } }); casa[4].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[4].setBackground(cores[casa[4].getSelectedIndex()]); Select(); Confere(); } });

Estou tentando fazer um Array com 5 posições;
Dá pra diminuir o código como o de cima?

Tentei fazer assim:
if(int x = 0; x<5;x++){ casa[x].addItemListener(//compara item selecionado e edita a cor new ItemListener() { public void itemStateChanged(ItemEvent e) { // TODO Auto-generated method stub if(e.getStateChange() == ItemEvent.SELECTED) painel[x].setBackground(cores[casa[x].getSelectedIndex()]); Select(); Confere(); } }); }

dá erro nessa linha:
painel[x].setBackground(cores[casa[x].getSelectedIndex()]);

Mais uma vez.. brigadão pela força!