e2: hasta que no funcione no pongo trazas con Logger

Tu estás tonto (por decirlo finamente).

Las trazas son para poner a punto el programa y localizar rápidamente los fallos.

Poner trazas sólo cuando funcione comprendo que es necesario para que te aprobemos
el ejercicio, pero realmente estás perdiendo tu tiempo.

Si te sobra el tiempo, sigue buscando errores a ciegas.
Pero si tienes cosas más interesantes que hacer en la vida,
pon trazas en los programas según escribes el código
y así lo pondrás a punto mucho más rápidamente.

Para que no interfiera con las trzas que te has pedido,
a tus trazas ponles otro nombre
MI_LOG= Logger.getLogger("cifrador.MiCifrador.mis_trazas");

Y NO LAS QUITES CUANDO LO ENTREGUES.