clases que implementan listener jamas por que expones funcionalidad interna es decir muestras lo que no debes a través de métodos publico que potencial mente serán llamados donde no corresponde

clases externas que implementen listener yo pienso que no es una buen idea por que lo que deberia ser externo y aparte es la acción que realiza el usuario que sea reutilizable por ejemplo si hariamos un sistema de edición de texto tendríamos crearDocumento importarDeWord importarDeXML confirmarCerrar que serian reutilisados desde los formularios así si presionas la x de la ventana ,si presionas escape  o presionas el botón cerrar rutilases confirmarCerrar que te abriria una ventana de confirmación pero este patrón introduciría a una explosión de clases que general mente no serian reutilisadas aparte de generar un poco de boilepart(el cual no es tan innecesario) así que creo que tendría mas sentido solo usarlo después de que exista la necesidad no desde el principio  otro razon para no usar este patrón extensivamente es que esos comportamientos ya estarían dentro de las clases servicios(ServicioImpresion,ServicioExportacion) y también en las de dominio


la otra opción es usar clases anónima internas son horribles gigantescas ensucian la pureza y la belleza de mi código y agrega muchas lineas de indentacion a mi clase aparte de que me obligan a que tengas mas de 300 lineas lo cual me produce un profundo vació en mi corazon y me da una sensacion de fracaso y depresión 

aunque parezca raro yo prefiero la opción de la clase anónima interna aunque mi corazon sufra  debido a que sus desventajas son miles de veces menores que las desventajas de las otras 2 opciones

si soy un tonto perfeccionista y obsesionado con el código