Java - Opinión sobre estructura de código

   
Vista:

Opinión sobre estructura de código

Publicado por Omar (1 intervención) el 06/07/2017 17:11:00
Hola estoy empezando en esto de la programación de JAVA y mi instrucción ha sido un poco autodidacta, recientemente hice una interfaz de usuario pequeña, aún no hace nada, la vi e intenté repetirla de la manera que creí al momento, pero sería de gran ayuda si alguien más experimentado me pudiera dar su opinión respecto al código como tal, como pudiera optimizarlo, que está bien, en que puedo mejorar, si es legible fácilmente, etc, gracias. El código consiste de tres ficheros, uno que tiene únicamente el método main donde ejecuto el frame, otro que crea el frame y otro que crea la lámina que va en el frame.

El código es el siguiente, dejé un gran espacio entre el código de cada fichero, (al final se aprecia una clase interna que implementa ActionListener porque planeo agregarle eventos):



1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
package ejerciciodialogo;
 
public class FicheroPrincipal {
 
 public static void main(String[] args) {
  // TODO Auto-generated method stub
 
  muestraMarco(miMarco);
 }
 
private static FicheroMarco muestraMarco(FicheroMarco miMarco){
 
 return miMarco;
 
}
 
 private static FicheroMarco miMarco=new FicheroMarco();
}



1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
package ejerciciodialogo;
 
import java.awt.BorderLayout;
 
import javax.swing.JFrame;
 
public class FicheroMarco extends JFrame{
 
  public FicheroMarco(){
   setSize(750,360);
   setTitle("SegundaActividad");
   setLocationRelativeTo(null);
   setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
   add(VentSup, BorderLayout.NORTH);
   add(VentInf, BorderLayout.SOUTH);
   setVisible(true);
  }
 
  private FicheroLamina VentSup=new FicheroLamina(1);
  private FicheroLamina VentInf=new FicheroLamina();
 
 }



1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
package ejerciciodialogo;
 
import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.BorderFactory;
import javax.swing.Box;
import javax.swing.ButtonGroup;
import javax.swing.JButton;
import javax.swing.JPanel;
import javax.swing.JRadioButton;
import javax.swing.border.LineBorder;
import javax.swing.border.TitledBorder;
 
public class FicheroLamina extends JPanel{
 
  public FicheroLamina(int uno){
   setLayout(LayoutSup);
 
 
   GrupoUno.add(Mensaje);
   GrupoUno.add(Confirmar);
   GrupoUno.add(Opcion);
   GrupoUno.add(Entrada);
   CajaUno.add(Mensaje);
   CajaUno.add(Confirmar);
   CajaUno.add(Opcion);
   CajaUno.add(Entrada);
   CajaUno.setBorder(BordeCajaUno);
   add(CajaUno);
 
   //-------------------
   GrupoDos.add(MenError);
   GrupoDos.add(MenInformacion);
   GrupoDos.add(MenAlerta);
   GrupoDos.add(MenPregunta);
   GrupoDos.add(MenPlano);
   CajaDos.add(MenError);
   CajaDos.add(MenInformacion);
   CajaDos.add(MenAlerta);
   CajaDos.add(MenPregunta);
   CajaDos.add(MenPlano);
   CajaDos.setBorder(BordeCajaDos);
   add(CajaDos);
 
   //---------------------
   GrupoTres.add(Cadena);
   GrupoTres.add(Icono);
   GrupoTres.add(Componente);
   GrupoTres.add(Otros);
   GrupoTres.add(Array);
   CajaTres.add(Cadena);
   CajaTres.add(Icono);
   CajaTres.add(Componente);
   CajaTres.add(Otros);
   CajaTres.add(Array);
   CajaTres.setBorder(BordeCajaTres);
   add(CajaTres);
 
   //---------------------
   GrupoCuatro.add(OpcionDefecto);
   GrupoCuatro.add(OpcionSiNo);
   GrupoCuatro.add(OpcionSiNoCancelar);
   GrupoCuatro.add(OpcionOkCancelar);
   CajaCuatro.add(OpcionDefecto);
   CajaCuatro.add(OpcionSiNo);
   CajaCuatro.add(OpcionSiNoCancelar);
   CajaCuatro.add(OpcionOkCancelar);
   CajaCuatro.setBorder(BordeCajaCuatro);
   add(CajaCuatro);
 
   //---------------------
   GrupoCinco.add(ArrayString);
   GrupoCinco.add(ArrayIcono);
   GrupoCinco.add(ArrayObjeto);
   CajaCinco.add(ArrayString);
   CajaCinco.add(ArrayIcono);
   CajaCinco.add(ArrayObjeto);
   CajaCinco.setBorder(BordeCajaCinco);
   add(CajaCinco);
 
   //---------------------
   GrupoSeis.add(CampTexto);
   GrupoSeis.add(CampCombo);
   CajaSeis.add(CampTexto);
   CajaSeis.add(CampCombo);
   CajaSeis.setBorder(BordeCajaSeis);
   add(CajaSeis);
  }
 
  public FicheroLamina(){
   add(BotonMostrar);
   BotonMostrar.addActionListener(new AccionBoton());
  }
 
  private GridLayout LayoutSup=new GridLayout(2,3);
  private JButton BotonMostrar=new JButton("Mostrar");
  private Box CajaUno=Box.createVerticalBox();
  private Box CajaDos=Box.createVerticalBox();
  private Box CajaTres=Box.createVerticalBox();
  private Box CajaCuatro=Box.createVerticalBox();
  private Box CajaCinco=Box.createVerticalBox();
  private Box CajaSeis=Box.createVerticalBox();
  private JRadioButton Mensaje=new JRadioButton("Mensaje",true);
  private JRadioButton Confirmar=new JRadioButton("Confirmar",false);
  private JRadioButton Opcion=new JRadioButton("Opción",false);
  private JRadioButton Entrada=new JRadioButton("Entrada",false);
  private JRadioButton MenError=new JRadioButton("Mensaje de Error",true);
  private JRadioButton MenInformacion=new JRadioButton("Mensaje de Informacón",false);
  private JRadioButton MenAlerta=new JRadioButton("Mensaje de Alerta",false);
  private JRadioButton MenPregunta=new JRadioButton("Mensaje de Pregunta",false);
  private JRadioButton MenPlano=new JRadioButton("Mensaje Plano",false);
  private JRadioButton Cadena=new JRadioButton("Cadena",true);
  private JRadioButton Icono=new JRadioButton("Icono",false);
  private JRadioButton Componente=new JRadioButton("Componente",false);
  private JRadioButton Otros=new JRadioButton("Otros",false);
  private JRadioButton Array=new JRadioButton("Array[]",false);
  private JRadioButton OpcionDefecto=new JRadioButton("Opción por Defecto",true);
  private JRadioButton OpcionSiNo=new JRadioButton("Si o No",false);
  private JRadioButton OpcionSiNoCancelar=new JRadioButton("Si, No o Cancelar",false);
  private JRadioButton OpcionOkCancelar=new JRadioButton("Ok o Cancelar",false);
  private JRadioButton ArrayString=new JRadioButton("String[]",true);
  private JRadioButton ArrayIcono=new JRadioButton("Icono[]",false);
  private JRadioButton ArrayObjeto=new JRadioButton("Objeto[]",false);
  private JRadioButton CampTexto=new JRadioButton("Campo de Texto",true);
  private JRadioButton CampCombo=new JRadioButton("Combo",false);
  private ButtonGroup GrupoUno=new ButtonGroup();
  private ButtonGroup GrupoDos=new ButtonGroup();
  private ButtonGroup GrupoTres=new ButtonGroup();
  private ButtonGroup GrupoCuatro=new ButtonGroup();
  private ButtonGroup GrupoCinco=new ButtonGroup();
  private ButtonGroup GrupoSeis=new ButtonGroup();
  private LineBorder LineaBordesDeCaja=new LineBorder(Color.BLACK);
  private TitledBorder BordeCajaUno=BorderFactory.createTitledBorder(LineaBordesDeCaja, "Tipo", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
  private TitledBorder BordeCajaDos=BorderFactory.createTitledBorder(LineaBordesDeCaja,"Tipo de Mensaje", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
  private TitledBorder BordeCajaTres=BorderFactory.createTitledBorder(LineaBordesDeCaja,"Mensaje", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
  private TitledBorder BordeCajaCuatro=BorderFactory.createTitledBorder(LineaBordesDeCaja,"Confirmar", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
  private TitledBorder BordeCajaCinco=BorderFactory.createTitledBorder(LineaBordesDeCaja,"Opción", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
  private TitledBorder BordeCajaSeis=BorderFactory.createTitledBorder(LineaBordesDeCaja,"Entrada", TitledBorder.DEFAULT_JUSTIFICATION, TitledBorder.DEFAULT_POSITION, null, Color.BLACK);
 
  class AccionBoton implements ActionListener{
 
   public void actionPerformed(ActionEvent e) {
 
 
   }
 
  }
 
 }
Valora esta pregunta
Me gusta: Está pregunta es útil y esta claraNo me gusta: Está pregunta no esta clara o no es útil
0
Responder
Revisar política de publicidad