1

Estou com alguns problemas ao atualizar o JDialog criado através do WindowsBuilder no Eclipse (não seria por conta disto).

Basicamente eu tenho uma tela de Alterar Cadastros do meu mini-sistema financeiro, eu carrego as informações na tela do usuário logado e exibo os demais em um JComboBox.

Criei um ActionListener para que quando o JComboBox for modificado, ele valide se o usuário selecionado é diferente do que está mostrando atualmente. Se for, ele carrega as novas informações.

Abaixo um trecho de código tentando explicar:

import javax.swing.JComboBox;
import javax.swing.JDialog;
import javax.swing.JPanel;

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.ParseException;
import java.util.ArrayList;

import javax.swing.JTextField;
import javax.swing.border.EmptyBorder;

import br.com.base_classes_paper.cadastroBase;
import br.com.classes_paper.cadastroActions;


public class altCadastroUsuario1 extends JDialog {

    private final JPanel contentPanel = new JPanel();
    private JTextField txtLogin;
    private JTextField passSenha;
    public cadastroBase valCadBas;
    public ArrayList<cadastroBase> cadTotal;

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        try {
            altCadastroUsuario1 dialog = new altCadastroUsuario1();
            dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
            dialog.setVisible(true);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
    public altCadastroUsuario1() {

        setTitle("PaperSys");
        setBounds(100, 100, 800, 600);
        getContentPane().setLayout(new BorderLayout());
        contentPanel.setBorder(new EmptyBorder(5, 5, 5, 5));
        getContentPane().add(contentPanel, BorderLayout.CENTER);
        contentPanel.setLayout(null);

        txtLogin = new JTextField();
        txtLogin.setBounds(106, 165, 260, 20);
        txtLogin.setText(valCadBas.getLogin());
        contentPanel.add(txtLogin);
        txtLogin.setColumns(10);

        passSenha = new JTextField();
        passSenha.setBounds(468, 165, 270, 20);
        passSenha.setText(valCadBas.getPass());
        contentPanel.add(passSenha);
        passSenha.setColumns(10);

        JComboBox jComBox_NomeUsuar = new JComboBox();

        for (cadastroBase load : cadTotal){
            jComBox_NomeUsuar.addItem(load.login);
        }

        jComBox_NomeUsuar.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                String selected = (String) jComBox_NomeUsuar.getSelectedItem();
                if (selected.equalsIgnoreCase(txtLogin.getText())){
                    for (cadastroBase load : cadTotal){
                        if (selected.equalsIgnoreCase(load.nomes)){
                            valCadBas.setLogin(load.login);
                            valCadBas.setPass(load.pass);
                            recarregarTela();
                        }
                    }
                }

            }

        });
        jComBox_NomeUsuar.setBounds(539, 282, 199, 20);
        contentPanel.add(jComBox_NomeUsuar);

    }
    protected void recarregarTela() {
        //altCadastroUsuario.this.removeAll();
        altCadastroUsuario1.this.repaint();
        altCadastroUsuario1.this.revalidate();
        //contentPanel.repaint();
        //contentPanel.revalidate();
    }

}

No caso, meu método faria mais ou menos o que está ai em cima, porém com alguns campos extras e métodos internos que rodam em outras classes. Mas do jeito que está, o mesmo não funciona.

Para ver se consigo me comunicar melhor: tenho a tela (gerada pelo windows builder do eclipse) que inicialmente exibe os dados do usuário que se logou no sistema (vamos supor login e senha) via get/set e as demais informações de todos os usuários do qual ele tem acesso salvo em um array carregando apenas os logins no jcombobox, dai quando ele modifica alguma casa no combobox eu valido se o que ele muda no jcombobox é diferente do que esta mostrando na tela, se sim eu carrego as informações deste usuário no get/set e tento dar o refresh da pagina pelos métodos revalidate() e repaint() pelo qual não atualizam o cadastro (o java compilando executar ambos mas nada acontece a seguir). Caso necessitam do código inteiro eu disponho dele amanhã pois hoje não me encontro em casa.

2
  • 2
    Olá Giovan, bem vindo ao Stack Overflow em português. Para que possamos te ajudar precisamos de um contexto maior sobre o que está acontecendo e o que não funciona. Crie um mvce e começamos a trabalhar daí. Commented 15/04/2015 às 2:00
  • 1
    Esse código nem roda, com erro de sintaxe. Tente testar o código postado antes de criar a questão. Commented 15/04/2015 às 2:09

1 Resposta 1

1

O melhor que consegui foi isso. Eu não sei o que ele deveria fazer exatamente ao recarregar a tela, portanto coloquei lá um System.out.println para demonstrar que o método foi chamado. Sempre que eu seleciono na JComboBox um nome igual ao da JTextField, esta função é chamada e o texto é escrito no console.

Eis o código:

import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JComboBox;
import javax.swing.JDialog;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class AltCadastroUsuario extends JDialog {
    private final JPanel contentPanel;

    protected String nome;

    public static void main(String[] args) {
        EventQueue.invokeLater(AltCadastroUsuario::iniciar);
    }

    private static void iniciar() {
        try {
            AltCadastroUsuario dialog = new AltCadastroUsuario();
            dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
            dialog.setSize(400, 200);
            dialog.setVisible(true);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    public AltCadastroUsuario() {
        contentPanel = new JPanel();
        this.add(contentPanel);
        JTextField txtLogin = new JTextField();
        txtLogin.setText("Teste");
        contentPanel.add(txtLogin);

        JComboBox<String> jComBox_NomeUsuar = new JComboBox<>();
        jComBox_NomeUsuar.addItem("Pedro");
        jComBox_NomeUsuar.addItem("Carlos");
        contentPanel.add(jComBox_NomeUsuar);

        jComBox_NomeUsuar.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent arg0) {
                String selected = (String) jComBox_NomeUsuar.getSelectedItem();
                if (selected.equalsIgnoreCase(txtLogin.getText())) {
                    nome = selected;
                    recarregarTela();
                }
            }
        });
    }

    protected void recarregarTela() {
        System.out.println("Recarregando");
        //altCadastroUsuario.this.removeAll();
        AltCadastroUsuario.this.repaint();
        AltCadastroUsuario.this.revalidate();
        //contentPanel.repaint();
        //contentPanel.revalidate();
    }
}

No seu código havia os seguintes problemas:

  • Você estava usando protect ao invés de protected, isso causava um erro de compilação (tal como revisado pelo Bruno César).
  • Faltavam alguns casts ao usar o método getSelectedItem() da JComboBox. Isso também causava um erro de compilação.
  • Você não deu a classe toda, o que obviamente torna o seu código não-compilável e deixa a sua questão mais difícil de ser respondida. Demorou algum tempo para eu perceber que altCadastroUsuario era o nome da classe. No entanto, eu não deveria ter que adivinhar isso sozinho. Além disso, nomes de classes deveriam iniciar com letras maiúsculas. Também tive que deduzir que altCadastroUsuario é subclasse de JDialog.
  • Não acesse o Swing fora da thread de eventos do AWT.
  • Defina um tamanho padrão da janela antes de mostrá-la, para não vir uma janela de tamanho mínimo que sou obrigado a redimensionar para usar.
  • O campo nome contém null quando o construtor da classe é iniciado. Como resultado, a JTextField tinha um tamanho tão ridiculamente pequeno, que ela não era usável.
  • Você adicionava a JComboBox e a JTextField em um JPanel, mas não adicionava o JPanel à tela. O resultado é que vinha uma tela em branco!
  • Vale a pena você dar uma investida em colocar um layout legal na sua JPanel. Do jeito que está, fica até difícil de se utilizar.
  • Tal como já mencionei antes, não sei o que mais deveria ocorrer no recarregarTela. Coloquei lá um System.out.println só para provar que ele é chamado quando de fato deveria ser chamado.
  • Se quiser deixar mais simples, caso esteja usando o Java 8 ou superior, use um lambda:

        jComBox_NomeUsuar.addActionListener(event -> {
            String selected = (String) jComBox_NomeUsuar.getSelectedItem();
            if (selected.equalsIgnoreCase(txtLogin.getText())) {
                nome = selected;
                recarregarTela();
            }
        });
    

A pergunta foi editada. Agora a minha sugestão é trocar este trecho:

                for (cadastroBase load : cadTotal){
                    if (selected.equalsIgnoreCase(load.nomes)){
                        valCadBas.setLogin(load.login);
                        valCadBas.setPass(load.pass);
                        recarregarTela();
                    }
                }

Por isso:

                for (cadastroBase load : cadTotal) {
                    if (selected.equalsIgnoreCase(load.nomes)) {
                        valCadBas.setLogin(load.login);
                        valCadBas.setPass(load.pass);
                        txtLogin.setText(load.login);
                        passSenha.setText(load.pass); // Isso só se você achar que faz sentido colocar a senha.
                        break;
                    }
                }

E com isso, o método recarregarTela() deixa de existir.

13
  • Como você está usando method referece capture, ao invés de usar uma classe anônima no listener considere usar lambda com um handler method, acredito ficaria mais "legível". Talvez também seja bom deixar claro que está usando java 8 =) Commented 16/04/2015 às 2:18
  • 1
    @BrunoCésar Feito. Commented 16/04/2015 às 6:10
  • Prezados bom dia, Agradeço a ajuda, o código acima é apenas um esboço e desculpe não ter colocado todo, valido informações em conjunto com 5 classes nesta tela o que acaba dificultando um pouco a explicação do código, o que estou tentando fazer está no comentário de baixo para entender, mas na verdade seria um refresh na tela recarregando as informações. Commented 16/04/2015 às 13:05
  • Tenho a tela (gerada pelo windows builder do eclipse) que inicialmente exibe os dados do usuário que se logou no sistema (vamos supor login e senha) via get/set e carrego os outros usuários em um array passando para a tela apenas os logins no 'jcombobox', dai quando ele modifica alguma casa no combobox eu valido se o atual é diferente do mostrado na tela, se for eu carrego do array as informações deste usuario no get/set e tento dar o refresh da pagina com revalidate() e repaint() sem sucesso (o java compilando executa ambos os metodos mas nada acontece ao passar por eles). Commented 16/04/2015 às 13:07
  • @GiovanDias Tudo o que você tem que fazer é chamar os métodos setText (ou semelhantes) nos objetos adequados, provavelmente JLabels e JTextFields. Não deveria ser necessário revalidate() e nem repaint(). Commented 16/04/2015 às 13:10

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .