🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: es.comp.lenguajes.java
Article #508

Re: Sugerencia en Codigo

#508
From: Ricardo Palomares Martinez
Date: Wed, 27 Jun 2007 17:27
90 lines
3465 bytes
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?
Technical Details
Message-ID: <b8jal4-kag.ln1@Father.family>
Path: arc1-ca.pugleaf.net!g2news1.google.com!news4.google.com!feeder1-2.proxad.net!proxad.net!feeder1-1.proxad.net!club-internet.fr!feedme-small.clubint.net!news.mixmin.net!aioe.org!Father.family!news
References: <1182962978.195273.306190@o61g2000hsh.googlegroups.com>

Show Full Headers
Complete Article Headers
X-Google-Thread: 1116f3,3a8d7fc3e117f6ca
X-Google-Attributes: gid1116f3,public,usenet
X-Google-Language: SPANISH,ASCII
Path: arc1-ca.pugleaf.net!g2news1.google.com!news4.google.com!feeder1-2.proxad.net!proxad.net!feeder1-1.proxad.net!club-internet.fr!feedme-small.clubint.net!news.mixmin.net!aioe.org!Father.family!news
From: Ricardo Palomares Martinez <rpm.PUBLI@iespana.es>
Newsgroups: es.comp.lenguajes.java
Subject: Re: Sugerencia en Codigo
Date: Wed, 27 Jun 2007 19:27:07 +0200
Organization: Hogar, dulce hogar
Lines: 90
Message-ID: <b8jal4-kag.ln1@Father.family>
References: <1182962978.195273.306190@o61g2000hsh.googlegroups.com>
NNTP-Posting-Host: BiBNQGstmkdW0YIxF5BZkg.user.aioe.org
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
X-Complaints-To: abuse@aioe.org
User-Agent: Mozilla/5.0 (X11; U; Linux i686; es-ES; rv:1.8.1.2) Gecko/20070221 SeaMonkey/1.1.1 Mnenhy/0.7.4.0