JTable

[schamamutz]

Hallo Leute,
ich würde gerne bei meinem Programm eine Tabelle befüllen, die ich dann anteilig berechne.

Durch "hinzufügen", möchte ich meine Tabelle mit den Daten füttern. In der Spalte 1 befinden sich die Namen, in der Spalte 2 die Zahlen. Die Spalte 2 soll dann anteilig berechnet werden, mit dem Wert von "berechne".

Zum Teil funktioniert es, aber irgendwo ist der Fehler

Im Bildchen sieht man besser was ich meine

import java.awt.Color;
import java.awt.Font;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;



import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.JTextField;
import javax.swing.table.DefaultTableModel;




public class table2 {
   
   
    public static void main(String[] args){
       
        // create JFrame and JTable
        JFrame frame = new JFrame();
        JTable table = new JTable();
       
        // create a table model and set a Column Identifiers to this model
        Object[] columns = {"Name","Kosten","Anteil"};
        DefaultTableModel model = new DefaultTableModel();
        model.setColumnIdentifiers(columns);
       
        // set the model to the table
        table.setModel(model);
       
        // Change A JTable Background Color, Font Size, Font Color, Row Height
        table.setBackground(Color.LIGHT_GRAY);
        table.setForeground(Color.black);
        Font font = new Font("",1,22);
        table.setFont(font);
        table.setRowHeight(30);
       
        // create JTextFields
        JTextField grundtf = new JTextField();
        JTextField kostentf = new JTextField();
        JTextField datumtf = new JTextField();
        JTextField grtf = new JTextField();
        JTextField anteilgrtf = new JTextField();
        JTextField test = new JTextField();
       
        // create JButtons
        JButton btnAdd = new JButton("Hinzufügen");
        JButton berechne = new JButton("Berechne");
       
        //links
        grundtf.setBounds(20, 220, 100, 25);
        kostentf.setBounds(20, 250, 100, 25);

        //rechts
        grtf.setBounds(280, 220, 100, 25);
        anteilgrtf.setBounds(280, 250, 100, 25);
        datumtf.setBounds(410, 310, 100, 25);
        test.setBounds(540, 310, 100, 25);
       
       
        btnAdd.setBounds(150, 220, 100, 25);
        berechne.setBounds(410, 220, 100, 25);
       
        // create JScrollPane
        JScrollPane pane = new JScrollPane(table);
        pane.setBounds(0, 0, 880, 200);
       
        frame.setLayout(null);
       
        frame.add(pane);
       
        // add JTextFields to the jframe
        frame.add(grundtf);
        frame.add(kostentf);
        frame.add(datumtf);
        frame.add(grtf);
        frame.add(anteilgrtf);
        frame.add(test);

        // add JButtons to the jframe
        frame.add(btnAdd);
        frame.add(berechne);
       
        // create an array of objects to set the row data
        Object[] row = new Object[3];
       
        // button add row
        btnAdd.addActionListener(new ActionListener(){

           
            public void actionPerformed(ActionEvent e) {
            
                row[0] = grundtf.getText();
                row[1] = kostentf.getText();
              
               
                // add row to the model
                model.addRow(row);
            }
        });
       
      
      [COLOR=#ff0000]  //berechne button
 
        berechne.addActionListener(new ActionListener(){
 

 
            
 
            public void actionPerformed(ActionEvent e) {
 
                
 
                 for (int clmCnt1 = model.getColumnCount(),rowCnt1 = model.getRowCount(), k = 0; k < rowCnt1; k++) {
 
                     for (int l = 0; l < clmCnt1; l++) {
 
                     
 
                        Double anteil = Double.parseDouble(anteilgrtf.getText());
 
                        Double gro = Double.parseDouble(grtf.getText());
 
                        
 
                        Object obj = model.getValueAt(k, 1);
 
                        Double wovo = Double.parseDouble(obj.toString());
 
                        
 
                        Double an = anteil/gro;
 
                        an = Math.round(100*an)/100.0;
 
                        String str1 = String.valueOf(an);
 
                        datumtf.setText(str1);
 
                        
 
                        
 
                        Double an1 = wovo*an;
 
                        an1 = Math.round(100*an1)/100.0;
 
                                        
 
                        String str = String.valueOf(an1);
 
                        row[2] = str;
 
//                        test.setText(str);
 
                        model.addRow(row);
 
                     
 
                     } 
 
                 }        [/COLOR]
               
               
               
            }
        });
       
        frame.setSize(900,400);
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
       
    }

}

Kann mir jemand helfen?

Vielen Danke, der Mutz

P.S.: Falls es das Thema schon gibt, bitte ich das zu entschuldigen, hab es dann nicht gefunden.

 

[Robat]

Und was genau ist das Problem / was funktioniert nicht?

 

[MoxxiManagarm]

Mir ist ehrlich gesagt absolut nicht klar was anteilgrtf und grtf sind. Das Gleiche gilt für viele weitere Felder. Ich empfehle sprechendere Benennung.

Ich würde meinen du willst in deiner actionPerformed über Zeilen iterieren, tatsächlich iterierst du aber über Spalten mit deinem clmCnt1. Du führst den also 3 mal aus, willst aber vermutlich 6 mal.

 

[schamamutz]

die Tabelle wird falsch befüllt. Das Ergebnis wird zwar in die richtige 3.Spalte geliefert und ist auch korrekt, doch erst ab Reihe4(wenn ich 3 Eingaben habe), mit falschen Inhalt von den Spalten 1-2 und dann auch immer 3x.
Das Bild geht von Reihe 4 los.

Vorher sah die Tabelle (nach dem "hinzufügen") wie folgt aus:

a 1
b 2
c 3

diese Tabelle soll nun mit einem Faktor(den ich über ein Textfield eingebe, das funktioniert) berechnet werden (z.B.: 0,1):
Dann sollte meine Tabelle wie folgt aussehen

a 1 0,1
b 2 0,2
c 3 0,3

Zur Zeit sieht meine Tabelle so aus:

a 1
b 2
c 3
c 3 0,1
c 3 0,1
c 3 0,1
c 3 0,2
c 3 0,2
c 3 0,2
c 3 0,3
c 3 0,3
c 3 0,3

 

[MoxxiManagarm]

Ja du iterierst über die Anzahl Spalten, nicht über die Zeilen. Du hast 3 Spalten, daher werden 3 Zeilen bearbeitet.
Ändere die for-Schleife

 

[schamamutz]

anteilgrtf und grtf sind eingabefelder, die den Faktor bestimmen sollen. Ja, keine schönen Bezeichnungen

 

[MoxxiManagarm]

Außerdem ist bestimmt nicht gewollt dass du in berechne nochmal eine Zeile hinzufügst. Du möchtest och bestimmt nur eine Zelle in der betroffenen Spalte updaten

 

[schamamutz]

ja. addRow kam mir von anfang an auch falsch vor, aber nur so hatte ich auch mal ein ergebnis

 

[schamamutz]

Vielen Dank, funktioniert jetzt. Update war das Stichwort

 

[schamamutz]

Ich hätte nochmal 2 Fragen:
1) beim berechnen des Anteiles, gibt er mir, wie gewollt, die 2 Stellen hinter dem Komma aus, doch nur, wenn die zahlen nicht 0,00 enden. wenn die zahlen auf 0,00 enden, gibt er mir nur eine Stelle hinter dem Komma wieder. Wodran liegt das?
2) Ich habe ihn jetzt die Spalten addieren lassen (funzt auch endlich ) und möchte diese in der letzten Zeile anzeigen lassen. Wie geht das? Hab es mit setValueAt(sum,-1,1) versucht(doch das ist Quatsch). Muss ich mit ?table?.lenght arbeiten?

P.S.: Hab es jetzt auch verstanden, warum er das ganze nur 3 Mal wiederholt hat

 

[MoxxiManagarm]

Wegen 2.. Du möchtest also eine Art Summenzeile haben?
Dafür ist das DefaultTableModel nicht geeignet, wenn du mich fragst.

Du könntest ein eigenes Model bauen und dafür sogar eine andere Datenstruktur verwenden als ein 2D-Array.

Hier ein Beispiel:

package examples.tablemodel;

import java.util.ArrayList;
import java.util.List;
import javax.swing.table.AbstractTableModel;


public class EntityTableModel extends AbstractTableModel {

    private List<Entity> list;
 
    public EntityTableModel() {
        list = new ArrayList<>();
    }
 
    public void addTableEntity(Entity entity) {
        list.add(entity);
        this.fireTableRowsInserted(list.size() - 1, list.size() - 1);
    }
 
    @Override
    public int getRowCount() {
        return list.size() + 1;
    }

    @Override
    public int getColumnCount() {
        return 3;
    }
 
    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
        return (columnIndex) == 2 && (rowIndex != list.size());
    }

    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {     
        if(rowIndex != list.size()) {
            Entity entity = list.get(rowIndex);
         
            switch(columnIndex) {
                case 0: return entity.getId();
                case 1: return entity.getName();
                case 2: return entity.getWeight();
                default: return null;
            }
        } else {
            switch(columnIndex) {
                case 2: double sum = 0;
                    for(Entity entity : list) sum += entity.getWeight();
                    return sum;
                default: return null;
            }
        }
    }
 
    @Override
    public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
        if(columnIndex != 2 || rowIndex == list.size()) return;
     
        list.get(rowIndex).setWeight(Double.parseDouble((String)aValue));
     
        fireTableCellUpdated(rowIndex, columnIndex);
        fireTableCellUpdated(list.size(), columnIndex);
    }
}

Spoiler: Entity

class Entity {
    private int id;
    private String name;
    private double weight;
 
    public Entity(int id, String name, double weight) {
        this.id = id;
        this.name = name;
        this.weight = weight;
    }
 
    public int getId() {
        return id;
    }
 
    public String getName() {
        return name;
    }
 
    public double getWeight() {
        return weight;
    }
 
    public void setId(int id) {
        this.id = id;
    }
 
    public void setName(String name) {
        this.name = name;
    }
 
    public void setWeight(double weight) {
        this.weight = weight;
    }
}

Spoiler: Tester

package examples.tablemodel;

import java.awt.BorderLayout;
import java.util.Comparator;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JTable;


public class Tester extends JFrame {
 
    public Tester() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     
        setLayout(new BorderLayout());
     
        EntityTableModel tm = new EntityTableModel();
        tm.addTableEntity(new Entity(1, "Anton", 73.9));
        tm.addTableEntity(new Entity(2, "Hans", 14.3));
        tm.addTableEntity(new Entity(3, "Guenther", 85.8));
     
        JTable table = new JTable(tm);
     
        add(table, BorderLayout.CENTER);
     
        pack();
    }
 
    public static void main(String[] args) {
        new Tester().setVisible(true);
    }
}

 

[VfL_Freak]

Moin,

wenn die zahlen auf 0,00 enden, gibt er mir nur eine Stelle hinter dem Komma wieder. Wodran liegt das?

weil '0,0' das gleiche ist wie '0,00' - man schreibt ja auch '25' und nicht '0025'!
Wenn Du zwingend beide Nullen als NK-Stellen sehen willst, dann musst Du das händisch formtiert ausgeben.
Siehe bspw. hier: http://javawiki.sowas.com/doku.php?id=java:number-formatter

VG Klaus

 

[schamamutz]

Danke euch für die Antworten. Die 1. Frage ist mir im Nachhinein ziemlich peinlich. Hab es auch bestimmt schon 5 mal überlesen
Ja, ich wollte eine Summenzeile und hab es jetzt auf die einfache Weise gelöst Ich wollte vorher alles in ein Object laden ... , deswegen hatte es nicht funktioniert.
Aber das Ziel wurde erreicht

@moxxi Sieht echt gut aus, doch mir ist die Schreibweise noch nicht ganz geheuer, ich muss das erstmal Step-by-Step machen, aber dahin sollte der weg gehen. Habe auch direkt versucht mich reinzufuchsen, schön das Layout geändert, Textfield und Button eingebaut. Dann wollte ich natürlich durch die Textfield Eingabe die Tabelle füttern, doch das Hinzufügen hat nicht funktioniert.
Im ActionListener habe ich "tm.addTableEntity(new Entity (str, d1, d2));"
versucht und dachte auch am Anfang, dass das klappen sollte, gemekert wurde auch nicht, ist aber nicht so

 

[MoxxiManagarm]

Sieht echt gut aus, doch mir ist die Schreibweise noch nicht ganz geheuer, ich muss das erstmal Step-by-Step machen, aber dahin sollte der weg gehen.

Ehrlich gesagt ist ein großes Problem, dass kaum ein Lehrgang/Lehrstuhl einmal die Verwendung von Abstract Models von Swing erklärt. Die meisten scheuen sich diese zu verwenden. Dabei kann es so viele Dinge vereinfachen, wenn man es einmal verstanden hat. Die DefaultModels sind halt begrenzt und nur für einfache Varianten ausgelegt. Zudem basieren Sie noch auf Vectoren, welche outdated sind.

Die Summenzeile mit dem AbstractTableModel funktioniert. Alternativ könntest du beim DefaultTableModel bleiben für die 1. eigentliche Tabelle und eine 2. Tabelle (auch DefaultTabelModel, bestehend aus einer Zeile = SummenZeile) so geschickt darunter platzieren, dass Sie wie zur ersten Tabelle gehörig aussieht. In diesem Fall müsste Die 2. Tabelle auf Änderungen der ersten Tabelle hören und dann die Summe neu berechnen.

 

[schamamutz]

Mein Lehrgang ist bisher nur ein Buch, das Internet und Try/Error

Hatte ich mir auch schon überlegt und ist vielleicht jetzt auch die Lösung von meinem neuen Problem, der will die Werte nicht neu berechnen. Liegt bestimmt an der geaddeten Leerreihe und Summenreihe
Danke dir

 

[MoxxiManagarm]

Die Werte werden bei mir durch 2 Sachen neu berechnet:
siehe: getValueAt für letzte Zeile Spaltenindex 2
siehe: das zweite fireTableCellUpdated in setValueAt

Das erste ist die eigentliche Berechnung und das zweite der Tabelle zusagen, dass sie das Feld neu laden muss.
In meinem Beispiel passiert das nur durch das Editieren einer Zelle. Natürlich kann es auch andere Fälle geben, z.B. das Hinzufügen/Entfernen einer kompletten Zeile. Auch hier müsstest du dann ein Event feuern, damit die Summenzeile neu berechnet wird. Ich habe das auch in meinem Beispiel vergessen (Schande auf mein Haupt). Mein Beispiel feuert beim einfügen einer Entity kein Event für die Summenzeile. Fügs bitte hinzu

Spoiler: EntityTableModel v2

package examples.tablemodel;

import java.util.ArrayList;
import java.util.List;
import javax.swing.table.AbstractTableModel;


public class EntityTableModel extends AbstractTableModel {

    private List<Entity> list;
  
    public EntityTableModel() {
        list = new ArrayList<>();
    }
  
    public void addTableEntity(Entity entity) {
        list.add(entity);
      
        // "Bescheid" Zeile wurde ergänzt
        fireTableRowsInserted(list.size() - 1, list.size() - 1);
        // "Bescheid" Summe muss dadurch neu berechnet werden
        fireTableCellUpdated(list.size(), 2);
    }
  
    @Override
    public int getRowCount() {
        return list.size() + 1;
    }

    @Override
    public int getColumnCount() {
        return 3;
    }
  
    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
        return (columnIndex) == 2 && (rowIndex != list.size());
    }

    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {      
        if(rowIndex != list.size()) {
            Entity entity = list.get(rowIndex);
          
            switch(columnIndex) {
                case 0: return entity.getId();
                case 1: return entity.getName();
                case 2: return entity.getWeight();
                default: return null;
            }
        } else {
            switch(columnIndex) {
                case 2: double sum = 0;
                    for(Entity entity : list) sum += entity.getWeight();
                    return sum;
                default: return null;
            }
        }
    }
  
    @Override
    public Class<?> getColumnClass(int index) {
        switch(index) {
            case 0: return Integer.class;
            case 1: return String.class;
            case 2: return Double.class;
            default: return null;
        }
    }
  
    @Override
    public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
        if(columnIndex != 2 || rowIndex == list.size()) return;
      
        list.get(rowIndex).setWeight((Double)aValue);
      
        // "Bescheid" Zelle wurde geändert
        fireTableCellUpdated(rowIndex, columnIndex);
        // "Bescheid" Summ muss dadurch neu berechnet werden
        fireTableCellUpdated(list.size(), columnIndex);
    }
}

 

[mihe7]

Ich kann @MoxxiManagarm nur beipflichten: in Swing war es eigentlich schon immer "Pflicht", sich LayoutManagern, dem EDT und mit der MVC-Architektur von Swing zu beschäftigen, d. h. insbesondere ListModel, TableModel und weiteren.