Ver Mensaje Individual
  #5 (permalink)  
Antiguo 27-06-2007, 18:27:07
Ricardo Palomares Martinez
 
Mensajes: n/a
Predeterminado Re: Sugerencia en Codigo

iktor escribió:
> Holas al Foro,
> Tengo un problemita con este codigo, quiero mostrar datos almacenados
> en un arreglo a un JTable, pero no aparece nada. Que estoy omitiendo,
> Resulta que estoy trabajando en netBeans y cuando diseño l JTable con
> el editor GUI le adiciono el codigo y si funciona, pero esta vez
> decidi hacerlo a punta de codigo y me sale vacio, ni las cabeceras
> aparecen. Gracias por su tiempo.



Bueno, lo más sencillo sería que lo hicieras con el diseñador GUI de
NetBeans y compararas el código. Verías más fácilmente qué diferencia hay.


> public class frmReporteOrdenesCompras extends JFrame{



El otro día leí en los grupos en inglés que está mal visto lo de
llamar a una variable "frmNombreQueSea". Lo de comenzar el
identificador con "frm" te delata como proveniente de Visual Basic. :-)


>
> DefaultTableModel Reg;



Por convenciones de estilo, los identificadores de propiedades y
métodos deberían comenzar siempre en minúsculas:

DefaultTableModel reg;


> public void inicializarcomp()
> {
> this.tblOrdenesCompra = new JTable();
> this.tblOrdenesCompra.setBounds(10, 30, 550, 150);


Aquí te falta lo que has puesto en InsertarLinea():

> this.tblOrdenesCompra.setModel(Reg);


Si no añades el modelo de datos al JTable durante la inicialización
del JFrame, probablemente no te va a funcionar como esperas.


>
> private void InsertarLinea()



Mejor así:

private void insertarLinea()

;-)


> for (int i=0; i<BaseDatos.ordenCompra.length; i++)
> {
> if (BaseDatos.ordenCompra[i]!=null)
> {
> Linea.add(1);
> Linea.add(BaseDatos.ordenCompra[i].getIdOC());
> Linea.add(BaseDatos.ordenCompra[i].getFechaOC());
> Linea.add(BaseDatos.ordenCompra[i].getTotalOC());
> i = BaseDatos.ordenCompra.length;
> }
> else if(BaseDatos.ordenCompra[i]==null)



Esta condición es redundante, ¿no? Si el flujo de ejecución no ha
entrado en la parte del if, es que necesariamente va a ser null, así
que ¿para qué comprobarlo?


> }
> else
> {



Además, si creas e inicializas el modelo de datos en su sitio, no
necesitarás este bloque if...else que es prácticamente redundante.

Saludos.

--
If it's true that we are here to help others,
then what exactly are the OTHERS here for?
Responder Con Cita