Article View: es.comp.lenguajes.java
Article #508Re: Sugerencia en Codigo
From: Ricardo Palomares Martinez
Date: Wed, 27 Jun 2007 17:27
Date: Wed, 27 Jun 2007 17:27
90 lines
3465 bytes
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!newsReferences:
<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