Ver Mensaje Individual
  #3 (permalink)  
Antiguo 25/04/2019, 09:38
CalgaryCorpus
 
Fecha de Ingreso: junio-2008
Ubicación: Seattle, USA
Mensajes: 733
Antigüedad: 15 años, 10 meses
Puntos: 61
Respuesta: Personalizando JList

No tengo idea de como resolver lo especifico que preguntas, pero como el codigo esta aqui para ser sometido al escrutinio, te sugiero que mejores el codigo reduciendo la redundancia del mismo. Mas facil comprender el codigo si hay menos que leer.

- Elimina los "this." innecesarios
- Elimina los casts especificando el tipo correcto en los parametros, en vez de pasar un tipo generico y luego hacer casts por cada vez que aparece el parametro.
- En vez de tener varias veces las mismas instrucciones con distintos valores (por ejemplo cuando se establece el color en getListCellRendererComponent, haz la logica para decidir respecto al color solamente y luego cambia el color 1 vez con el color que se eligio en la logica.
- Parece mas facil de leer si el codigo esta todo en espan~ol o todo en ingles, la mezcla no parece bien. Ejemplo

Registro r : p.getRegistros().getChildrens().

A proposito de ingles, si es que decides dejarlo asi, children ya es plural, no requiere la s.
- getBottom() y getRight() parecen copia una de la otra, son super parecidas, y ademas, no se usan en ninguna parte. Por que no eliminarlas? O bien sacar factor comun y eliminar la redundancia?
- En estas lineas
lmedico.setText(data.getConsulta().getMedico().get Nombre() + " " + data.getConsulta().getMedico().getApellido());
lpaciente.setText(data.getConsulta().getPaciente() .getNombre() + " " + data.getConsulta().getPaciente().getApellido());

puedes definir una variable para data.getConsulta() y luego usarla 4 veces.

Última edición por CalgaryCorpus; 25/04/2019 a las 09:49