paso de un DefaultTableModel a otro

[lucho248]

Hola comunidad, tengo el siguiente problema.
He leido en este foro y en internet, y los distintos ejemplos no me han ayudado con el tema puntual que tengo..

Tengo el siguiente esquema:
tengo un JFrame en el cual es para tickar las compras (supermercado), y como suele pasar que viene un cliente, pasa algunos articulos, los pasa, y se va a buscar otros...

mientras tanto para seguir cobrando al siguiente, lo que hago es "guardar", la lista de articulos comprados en un boton (con su objeto de DefaulTableModel asociado).
Puedo guardar hasta 10 ticket a la vez.

paso cod de lo que tengo:

Código Declaracion: :

public DefaultTableModel model;
public static DefaultTableModel mM1, mM2, mM3, mM4, mM5, mM6, mM7, mM8, mM9, mM10;
private List<DefaultTableModel> listModel;
private JTable tabla;
private JButton btnM1, btnM2, btnM3, btnM4, btnM5, btnM6, btnM7, btnM8, btnM9, btnM10;
 

Código metodos:

listModel = new ArrayList<DefaultTableModel>();
 
/*la declaracion de todos los botones es la misma*/
btnM1 = new JButton();      
        btnM1.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                botonTicket(btnM1, mM1); //evento para asignar los modelos
            }
        });
        btnM1.setFont(new Font("Tahoma", Font.PLAIN, 30));
        btnM1.setBounds(744, 20, 80, 50);
        contentPane.add(btnM1);
 

Si el boton tiene o no asignado un ticket, le asigno el texto de: "Mn", donde "n" es la cantidad de ticket ya guardados en lista.

Código Java:

public void botonTicket(JButton boton, DefaultTableModel modN){
        DefaultTableModel auxModel, aux;
        try {
            auxModel = (DefaultTableModel) tabla.getModel();
            boolean celdaVacia = (auxModel.getValueAt(0, 0).equals(null) || auxModel.getValueAt(0, 0).equals(""));
            
                listModel.add(auxModel);    //agrego el modelo al boton
                
                if (boton.getText().length() > 0)
                {
                    initTabla(modN); //esta en el boton
                    modN = auxModel;
                    listModel.remove(modN); //quito el modelo del boton                                 
                }else
                { 
                    
                    modelos(auxModel, modN);
                    initTabla(null);
                }
                boton.setText("M"+listModel.size());            
            
        } catch (Exception e) {
            //quito modN de la lista y borro el boton
            if(boton.getText().length() > 0){
                listModel.remove(modN); //quito el modelo del boton                                 
                initTabla(modN); //esta en el boton                     
                boton.setText("");
            }           
            
            e.printStackTrace();
            
        }
        
    }
 
    public void modelos(DefaultTableModel modViejo, DefaultTableModel modNuevo) {
       for (int i=0;i<modViejo.getRowCount(); i++) {
            Object fila [] = new Object [modViejo.getColumnCount()];
            for (int j=0; j<modViejo.getColumnCount(); j++){
                fila[j] = modViejo.getValueAt(i,j);
            }
            modNuevo.addRow(fila);
        }        
    }
 
public void initTabla(TableModel mod){
        if (mod == null){
            model = new DefaultTableModel(
                    new Object[][] {
                            {null, null, null, null, null, null, null},
                            {null, null, null, null, null, null, null},
                            {null, null, null, null, null, null, null},
                            {null, null, null, null, null, null, null},
                        },
                        new String[] {
                            "C\u00F3digo", "Detalle", "Cantidad", "Imp.Interno", "Pr.Unit.", "Total", "SN"
                        }
                    ) {
                        Class[] columnTypes = new Class[] {
                            String.class, String.class, String.class, String.class, String.class, String.class, String.class
                        };
                        public Class getColumnClass(int columnIndex) {
                            return columnTypes[columnIndex];
                        }
                    };
            tabla.setModel(model);
            
            tabla.getColumnModel().getColumn(0).setMinWidth(50);
            tabla.getColumnModel().getColumn(1).setPreferredWidth(100);
            tabla.getColumnModel().getColumn(1).setMinWidth(60);
            tabla.getColumnModel().getColumn(3).setMinWidth(30);
            tabla.getColumnModel().getColumn(4).setMinWidth(30);
        }else{
            int fila = mod.getRowCount();
            int col = mod.getColumnCount();
            
            for (int x = 0; x < fila; x++){
                for (int y = 0; y < col; y++){
                    
                    model.setValueAt(mod.getValueAt(x, y), x, y );
                }
            }
//          model = md;
            tabla.setModel(model);
        }   
        
    }
 

el problema es el siguiente, cuando hace click en un boton, "creo" asigna bien lo que esta en la tabla (articulos cargados para tickar), al modelo del boton, pero...

Cuando hace click en el boton, para "volcar" lo que se encuentra en ese modelo correspondiente, no lo esta asignado al modelo de la tabla. No me lo carga o muestra.

Ya no tengo idea que probar porque hace dias que estoy dando vuelta con este tema.

Puede ser que no me este guardando el model de la tabla en el modelo correspondiente del boton (el que paso por parametro)?

Cualquier ayuda sera bienvenida.
Si hay alguna forma de pasar el codigo de la clase, me avisan porque tiene unas mil lineas.

Saludos.

[HackmanC]

Hola,

Posiblemente,

Iniciado por lucho248

[HIGHLIGHT="Java"]
public void botonTicket(JButton boton, DefaultTableModel modN){
...
modN = auxModel;
...
Puede ser que no me este guardando el model de la tabla en el modelo correspondiente del boton (el que paso por parametro)?
...

Posiblemente no se está guardando, los parámetros en el lenguaje Java se pasan por valor en todos los casos. En la línea donde dice modN = auxModel no estás modificando la variable estática mM1, como seguramente piensas.

Un par de detalles adicionales que posiblemente te sean de utilidad, el lenguaje Java es mucho mas extenso y no lo estás usando como se debe. Por eso estás haciendo miles de líneas para algo que seguramente tomaría mucho menos.

Es un lenguaje orientado a objetos, con referencias y garbage collector, posiblemente estás muy acostumbrado a usar lenguaje C, Pascal, Visual Basic 6 y su DataGrid u otros, pero por ejemplo, el TableModel es un objeto.

¿Para que copias todo el contenido de un TableModel a otro, celda por celda, si es tan simple guardar todo el table model y asignarselo completo de nuevo?

Al principio pensé que la variable listModel te servía para algo así, después me di cuenta que la variable no es útil en todo (lo visible) del programa.

Código Java:

TableModel MyOldTableModel = Table.getModel();
Table.setModel(MyOtherTableModel);
Table.setModel(MyOldTableModel);
 

Saludos,

[lucho248]

Hola HackmanC, lo copio al DefaultTableModel porque necesito guardar temporalmente lo tickado o leido por el colector de codigo de barra de un cliente.
Es una tienda chica, entoces se da seguido que viene un cliente, pasa algunos articulos, y vuelve a buscar otros articulos, entonces se guarda por un rato (5min, 10min, 25min), se sigue facturando al siguiente cliente en la fila, hasta que vuelve el anterior cliente con lo que fue a buscar. Se recupera lo ya leido, y se continúa con el ticket hasta cerralo.

Entonces solo quiero guardar la instancia de DefaultTableModel un momento, empezar otro, guardarlo o no si es necesario, si se repite la situacion, y recuperarlo cuando el cliente vuelve. Se entiende?

Link de imagen
si miras, donde estan los botones don los textos "M1", "M2", etc. es ahi donde voy guardando los ticket temporalmente.

[lucho248]

lo solucione momentaneamente de la siguiente manera:

Código Declaracion::

public DefaultTableModel model;
    public DefaultTableModel mM1, mM2, mM3, mM4, mM5, mM6, mM7, mM8, mM9, mM10;
    public int ticket;
    private List<String> listTicket;
 

Código Metodos:

public void botonTicket(JButton boton, DefaultTableModel modN){
        DefaultTableModel auxModel, aux = null;
        aux = modelosBotones(aux);
        try {
            auxModel = (DefaultTableModel) tabla.getModel();
            boolean celdaVaciaTabla = (auxModel.getValueAt(0, 0).equals("") || auxModel.getValueAt(0, 0).equals(null));
            boolean celdaVaciaModelo;
            if (modN != null){
                celdaVaciaModelo = (modN.getValueAt(0, 0).equals("") || modN.getValueAt(0, 0).equals(null));            
            }else{
                celdaVaciaModelo = false;
            }
            
            if (celdaVaciaTabla && (boton.getText().length() > 1))
            {   
                modelos(modN, auxModel); 
                tabla.setModel(auxModel);
                modN = new DefaultTableModel();
                modN = modelosBotones(modN);
                
                int intBoton = Integer.parseInt(boton.getText().substring(1, boton.getText().length()));
                String strBoton = boton.getText();
                boton.setText("");
                
                ticket = intBoton-1; //resto un ticket menos que el que se quito
                listTicket.remove(strBoton);
                
            }else{                                          
                    /** si los dos estan vacios o los dos tienen ticket, que no intercambie nada y no incremente el ticket*/
                    if(!celdaVaciaTabla && celdaVaciaModelo ){
                        modelos(auxModel, modN);
                        initTabla(null); 
                                                
                        boolean esta;                       
                        do{
                            ticket++;
                            esta = listTicket.contains("M"+ticket);                         
                        }while(esta);                       
                        if(!esta){
                            listTicket.add("M"+ticket);
                            boton.setText("M"+ticket);
                        }
                    }               
            }       
            
        } catch (Exception e) {
            e.printStackTrace();            
        }       
    }
    
    /** Intercambio los DefaultTableModel con este metodo
     * @param DefaultTableModel modViejo: el modelo con datos
     * @param DefaultTableModel modNuevo: el modelo en blanco, donde se copiaran los datos.
     * */
    public void modelos(DefaultTableModel modViejo, DefaultTableModel modNuevo) {
        int filas = modViejo.getRowCount();
        int col = modViejo.getColumnCount();
        int i = 0, j = 0;
        boolean BFilas = false;
       while (i<filas && !BFilas) {
           
            if (modViejo.getValueAt(i,j) == null || modViejo.getValueAt(i,j) == "")
                BFilas = true;
            else{
                while (j < col){
                    
                    modNuevo.setValueAt(modViejo.getValueAt(i,j), i,j);
                    modViejo.setValueAt("", i,j);
                    j++;
                }               
                i++;
                j = 0;
            }           
        } 
   }
 

con estas modificaciones logre que haga lo que estoy buscando.
Si alguien me puede indicar como mejorarlo, sera bien venido.
Saludos.

[HackmanC]

Hola,

Iniciado por lucho248

Código Metodos:

public void botonTicket(JButton boton, DefaultTableModel modN){
...
                modN = new DefaultTableModel();
                modN = modelosBotones(modN);
....
 

con estas modificaciones logre que haga lo que estoy buscando.
Si alguien me puede indicar como mejorarlo, sera bien venido.
...

Si ya te funciona pues todo está bien, como no sé que hay dentro del método modelosBotones() no sabría decir como lo solucionaste realmente. Aunque me imagino que de alguna forma realizaste un mapping entre modN y las variables estáticas, le pasas la variable y el busca la referencia en las estáticas, posiblemente con ==, y devuelve la estática, realmente saber que hiciste.

Iniciado por lucho248

/** Intercambio los DefaultTableModel con este metodo
* @param DefaultTableModel modViejo: el modelo con datos
* @param DefaultTableModel modNuevo: el modelo en blanco, donde se copiaran los datos.
* */
public void modelos(DefaultTableModel modViejo, DefaultTableModel modNuevo) {
int filas = modViejo.getRowCount();
int col = modViejo.getColumnCount();
int i = 0, j = 0;
boolean BFilas = false;
while (i<filas && !BFilas) {

if (modViejo.getValueAt(i,j) == null || modViejo.getValueAt(i,j) == "")
BFilas = true;
else{
while (j < col){

modNuevo.setValueAt(modViejo.getValueAt(i,j), i,j);
modViejo.setValueAt("", i,j);
j++;
}
i++;
j = 0;
}
}
}
[/HIGHLIGHT]

Si alguien me puede indicar como mejorarlo, sera bien venido.

Esa parte no comprendo por qué es necesaria, con lo que expliqué no es necesario copiar un modelo sobre otro. Al fin y al cabo mM1, es una instancia de un TableModel. Tampoco creo que sea un asunto sumamente importante, si a ti te funciona pues ya estás hecho.

Conceptos que posiblemente te sean de utilidad,

1. Heredar los botones (JButton) que incluya en su definición un modelo TableModel, puesto que cada botón tiene un TableModel asociado. (Evitas los 'Array Paralelos' o el mapping que estás usando). Para eso tienes que saber como funciona la herencia.

2. No copiar los TableModel uno sobre otro, usar las referencias, puesto que al fin y al cabo son TableModel's. Técnicamente un Model no debe mantener un estado (Stateless), así que copiar uno sobre otro me parece innecesario.

3. Heredar los TableModel (AbstractTableModel) por uno tuyo que ya tenga los títulos de las columnas y tenga métodos utilitarios como isEmpty(), que evidentemente lo necesitas y lo estas usando regularmente. Usando herencia y reutilizar código, no copiar y pegar el mismo código por todo el programa (esto no es PHP, es broma, es broma), por ejemplo, isEmpty():

(auxModel.getValueAt(0, 0).equals("") || auxModel.getValueAt(0, 0).equals(null));

Y así otra serie de cosas, pero eso ya te queda de tarea si lo quieres hacer,

Saludos,

[lucho248]

Hola HackmanC, tenes razon en todo lo que decis, justamente por eso que comentas de copiar los contenidos, recurri a esa alternativa porque asignado los objetos, no los guardaba cuando volvia a llamar al metodo.

Código Java:

modN = new DefaultTableModel();
                modN = modelosBotones(modN);
 

probando hacer:
1) - new DefaultTableModel();............ o
2) - modelosBotones(modN); no terminaba de mantener estas asignaciones que pretendia hacer.

en cambio con
modN = modelosBotones(modN);......... con esta si me terminaba guardando los cambios en el modelo, porque? no tengo idea ja.

con modeloBotones hago esto:

Código Java:

public DefaultTableModel modelosBotones(DefaultTableModel mod){
        mod = new DefaultTableModel(
                new Object[][] {
                        {"", "", "", "", "", "", ""},
                        {"", "", "", "", "", "", ""},
                        {"", "", "", "", "", "", ""},
                        {"", "", "", "", "", "", ""},
                        
                    },
                    new String[] {
                        "C\u00F3digo", "Detalle", "Cantidad", "Imp.Interno", "Pr.Unit.", "Total", "SN"
                    }
                ){
                Class[] columnTypes = new Class[] {
                    String.class, String.class, String.class, String.class, String.class, String.class, String.class
                };
                public Class getColumnClass(int columnIndex) {
                    return columnTypes[columnIndex];
                }
            };
            return mod;
    }
 

creo un modelo vacio, nada mas.

y esta linea:
(auxModel.getValueAt(0, 0).equals("") || auxModel.getValueAt(0, 0).equals(null));
en algun momento me empezo a levantar excepciones, modificandola un poco entonces, no recuerdo porque la deje asi, y me dio resultado.

Tenes razon en todo lo que me decis, pero como me daba muchos errores, empece a hacer modificaciones, hasta que se fueron eliminando estos errores, y me termino quedando de esa manera. Como alguna vez te habra pasado ja.

Gracias.
Saludos.-

[HackmanC]

Hola,

Iniciado por lucho248

... Tenes razon en todo lo que me decis, pero como me daba muchos errores, empece a hacer modificaciones, hasta que se fueron eliminando estos errores, y me termino quedando de esa manera. Como alguna vez te habra pasado ja. ...

Comparto mucho tu opinión, además no vi el mensaje que pusiste en medio de tu respuesta anterior y la mía. Eso me pasa a mi cuando alguien responde dos veces seguidas aquí en foros del web, porque modifique un poquito el aspecto del foro con GreaseMonkey y Javascript, pero eso es otra historia.

Yo soy bastante purista con los lenguajes de programación que conozco y tengo algún tiempo de experiencia con varios, uno de ellos Java.

Todos los comentarios que escribí no son con el afán de demostrar nada, ni mucho menos criticarte negativamente, el objetivo es que, conociendo bien todo eso que comenté, resultarías trabajando menos y mas rápido, lo cual desde mi punto de vista te conviene; aunque siempre exista la posibilidad que no sea así, que no te convenga.

Seguramente con el tiempo vas a comprender los motivos por los cuales te aconsejo aplicar todo lo mencionado. Por el momento me alegro mucho que te haya funcionado y espero que continúes bien.

Saludos,