Alguien dispuesto a revisar codigo?

[megajandro]

yo veo lo siguiente:
1- No existe un esquema de objetos de dominio. Por ej: cliente. (Aunque los atributos parecen mas de una persona que de cliente). No existe ninguno de los conceptos de POO. En un simple ABMC(CRUD) ya los podes aplicar.
2- No hay una separacion de la presentacion (Swing) con tu capa de manejo de BD(JDBC).
Para ver mejor esto deberias entender como maneja swing el modelo MVC.
3- Es cierto lo del try, catch que dijo juanito1712. Aunque para un mejor manejo seria que entiendas que es una excepcion chequeada y cual no.
4- Tenes codigo repetido para obtener la conexion de la BD.

Y yo agregaria utilizar algun plugin para verificar bugs como (FindBugs) y un checkstyle creo que
te aniquilaria.

Saludos.