Clase Interfaz

[Squar]

Buenas:

He de decir que yo personalmente tengo un estilo totalmente distinto de programar intefaces al tuyo, pero bueno, aún así te lo voy a sugerir a ver que te parece.

• Por un lado, la parte de general hay programadores que la meten en la misma clase del JFrame, a mi personalmente me gusta separarla como tu has hecho.
• En segundo lugar, creo que te estas complicando mucho al crear Visual, ya que realmente lo que haces es incluir dos atributos y hacer una función que introduce los componentes en los atributos. Yo cambiaría esto, ya que no te aporta nada salvo complejidad (si te das cuenta has tenido que hacer un par de switch para algo que si lo haces directo no tienes ningún problema, es decir, pasas el codigo de lo que deseas y se lo asisgnas directamente, sin saber que es cada cosa pues no necesitas modificarlo). La segunda cosa que aporta es claridad en Interfaz, que está bien, pero se puede hacer de otra manera.
• Por último, la clase Interfaz es muy escueta y está muy bien, realmente me gusta su simplicidad, pero no ha costa de añadir complejidad por otro lado. Si lo que quieres es una función para añadir para tener simplicidad en la introducción de contenido, añadela en la misma clase. Otra opción es añadir toda la información en una linea
  contenedor.add(elemento, new GridBagConstraints(0, 0, 1, 1, 0.0, 0.0, GridBagConstraints.CENTER, GridBagConstraints.NONE, new Insets(5, 5, 5, 5), 0, 0));
  contenedor.add(elemento new GridBagConstraints(gridx, gridy, gridwidth, gridheight, weightx, weighty, anchor, fill, insets, ipadx, ipady));
  Aunque he de decir que puede quedar un poco ilegible, y si alguien tiene que volver a tocar, lo mismo se confunde.

Bueno, eso es todo, espero que te sirva de algo. Saludos.

PD: Una última cosa, yo pondría el añadir componentes de General dentro del constructor de la Interfaz, pues es algo que siempre has de hacer y no tiene sentido independizarlo.