Opinião!

Galera estou montando um sistema java todo na mão, ou seja, a interface gráfica tudo na raça… Até agora parece esta tudo tranquilo, porém gostaria da opinião de vc´s sobre a minha implementação…Abaixo segue o código…Gostaria q vc dissessem se tem jeito melhor ou se assim esta legal!

Obrigado!!

import java.awt.*;

/Classe que tem uma função para posicionar os elementos na tela/
class FuncoesGerais {

public static Object posicionarElemento(GridBagConstraints gbc, int x, int y, int w, int h) {

  gbc.gridx = x;
  gbc.gridy = y;
  gbc.gridwidth = w;
  gbc.gridheight = h;
  gbc.insets = new Insets(10, 0, 0, 0);

  return gbc;

}
}

/Nesta classe faço uso da classe acima/
import javax.swing.;
import javax.swing.border.
;
import java.awt.;
import java.awt.event.
;

class PainelConsig extends JPanel {

JTextField inscricao, nome, cpf;
JButton incluir, limpar;
TitledBorder bordaConsig;

public PainelConsig() {

  bordaConsig = new TitledBorder(BorderFactory.createLineBorder(new Color(153, 153, 153),2),"Incluir novo associado");
  this.setBorder(bordaConsig);
  this.setLayout(new GridBagLayout());
  GridBagConstraints gbc = new GridBagConstraints();
  gbc.fill = GridBagConstraints.NONE;
  gbc.anchor = GridBagConstraints.WEST;

  this.add(new JLabel("Inscrição: "), FuncoesGerais.posicionarElemento(gbc, 0, 0, 1, 1));

Poxa galera, nenhum opinião…?

:frowning:

Bom, ja que vc pediu tao encarecidamente, eu vou ser bem chato:

  • Nao use import blah.*: nao da pra saber de que pacote vieram as classes que vc esta importando. Uma boa IDE (Eclipse, IDEA e outras) tomam conta disso pra vc

  • O metodo estatico posicionarElemento nao precisa receber um GridBagConstants - ele pode muito bem criar um, e retornar ele

  • O retorno do metodo posicionarElemento nao e’ Object, e’ uma instancia de GridBagConstraints… trate-o como tal.

  • Nao tem JavaDoc nos metodos

  • Identacao, cade!?