Programacion Profesional Embeber codigo

[malangr]

Buenas noches a todos amigos programadores, soy nuevo y es un gusto estar aqui, tengo desde ya hace tiempo necesidad por consejos del buen programar en java
he realizado varios proyectos JSP->Servlet->Bean con mysql y a pesar de que funcionan perfectamente creo que cometo un error no se si sea muy grave

Generalmente Inserto codigo CSS/HTML dentro de Servlets o Beans
que ya en pantalla forma una interfaz para ingresar posteriores datos
sin embargo esto lo hago junto con la hoja de estilos y me genera bueno
muchisimas lineas de codigo dentro de la variable que voy a imprimir
esto como resultado me deja un Bean con Al menos 500 lineas al tener 4 subclases
o subfunciones.

La pregunta es: es sano realizar esta practica? que puedo hacer en caso de que esto no sea muy bien visto o sea de poca calidad?, cabe mencionar que no se usar ningun framework asi que es java tal cual

desde ya muchas gracias

[chuidiang]

Hola:

Las buenas costumbres de programación requieren que en lo posible las cosas vayan separadas y más los distintos "lenguajes" que uses.

css debe ir en sus ficheros .css

Los servlets/beans java no deben tener nada de texto html ni css, sólo código java.

Los ficheros .jsp son los que contienen el html y los tags propios de jsp, haciendo "include" de los ficheros .css. El fichero .jsp debe contener lo menos posible de código java ( entre <% ... %>) y debe usar en su lugar los tags específicos de jsp, ayudándote de librerías/frameworks si es necesario (como JSTL, librería básica que te permite hacer los out.println(), los if, los bucles a base de tags jsp en vez de código estilo <% for (int i=0; ... %> ).

El motivo es sencillo, aparte de la claridad, la comodidad para hacer modificaciones más adelante. Si quieres cambiar el estilo de tu página web, sólo tienes que cambiar el fichero .css en vez de ir buscando por los ficheros java/jsp, tocar código y recompilar. Si quieres cambiar el html, sólo tienes que ir al jsp en vez de meterte en el código java del servlet/bean. Si quieres cambiar la lógica de tu aplicación, sólo vas al código java y no tienes que andar tocando los jsp.

Por supuesto, las cosas casi nunca son blanco o negro, hay muchos tonos de gris. El motivo de exigir claridad en el código es para que otra gente (o tú mismo) pueda leerlo más adelante y modificarlo fácilmente. Si tu código va a ser mantenido por otra gente o visto por un profesor, debes seguir estrictamente las buenas costumbres. Si tu código es para pequeños proyectos personales, herramientas de prueba, etc, podrías relajar estas buenas costumbres por la comodidad/velocidad de codificación. Puedes ser menos estricto cuanto más seguro estés de que nunca nadie, ni tú mismo, vas a tener que mirar/modificar ese código más adelante.

Se bueno.