Java jtable aktualisieren *Update -> neues Prob*

Hey, ich hab nen Problem mit Java und google konnte mir da nicht helfen.

Ich habe eine Tabelle, welche bei der initialisierung ihre Daten aus einem 2-dimdensionalen Array erhält, klappt alles ohne Probleme.

Hier der Erzeugungsprozess, wobei manager.gettable() eben das 2-dimensionale Array zurückgibt.

Code:

Manager manager = new Manager();
private String[] columnNames = {"Filme", "Rating", "Genre"};
dtm = new DefaultTableModel(manager.gettable(), columnNames);
table = new JTable(dtm);
getContentPane().add(new JScrollPane(table), BorderLayout.CENTER);

das klappt wunderbar.

In der Klasse Manager habe ich eine Methode, welche die Tabelle sortiert. Nun möchte ich einfach, dass die Tabelle sozusagen neu gezeichnet wird.
Bei google kommt dann als lösung immer sowas raus:

Code:

dtm.fireTableDataChanged()
table.repaint(); 

das funktioniert aber nicht.
Hat wer eine Idee? Mach schon seit Stunden rum, bekomms aber einfach nicht hin :/

tia

Ich kann jetzt auch total falsch liegen aber ich würde mal sagen, dass deiner Table komplett egal ist was du mit deinem Array im Manager machst.

Du übergibst ja lediglich das Array an das DefaultTableModel. Was du dann im Manager machst ist nur im Manager.

das Problem ist aber, dass die "refresh" bzw repaint methode vom table nicht hergeht und mal beim Manager fragt ob sich etwas geändert hat.

Schon mal daran gedacht es mit einem Model zu lösen?

ok ..

ungefähr so schaut ein Model aus:

Code:

public class MyTableModel extends AbstractTableModel {

	private String[] head = {"Name", "Größe", "Pfad", "Datum", "Lesen", "Schreiben", 
	private Manager manager;

        public MyTableModel(Manager manager){
                this.manager = manager;
        }

	public int getColumnCount() {
		return head.length;
	}

	public int getRowCount() {
		return manager.gettable().length();
	}
	
        public String getColumnName(int col) {
                return head[col];
        }

	public Object getValueAt(int row, int col) {
		return manager.gettable()[row][col];
	}

        public void sort(){
                manager.sort();
                fireTableDataChanged();
        }
}

und dann einfach beim Erstellen der Table übergeben.

Manager blalba = new Manager();
MyTableModel mtm = new MyTableModel(blalba);
JTable t = new JTable(mtm);

oder

t.setModel(mtm);

und dann einfach

mtm.sort();


Schöne Übersicht: http://download.oracle.com/javase/t...ents/table.html

Update!

Leider hab ich wieder ein Problem mit den Tabellen :/

Da es wohl etwas komplexer ist, erkläre ich hier meine Gedanken und unten folgen die kompletten Klassen, welche relevant sind, damit man es besser nachvollziehen kann.

Ich bin dabei, nen MovieManager zu basteln. Ich hab noch nicht so viel Übung, was "größere" Projekte in Java angeht, deswegen tue ich mich wohl bei einigen Sachen relativ schwer.

Zu den Klassen:
Die Klasse Manager ist praktisch die Controllklasse für alles, was im Hintergrund abgeht, hier eigentlich nur zwei Arrays.
Da ich nicht immer die Arrays hin und her schieben möchte, sind die static und falls die GUI-Klassen was ändern müssen, erzeugen sie einfach ein neues Manager-Objekt und bearbeiten die Arrays mit set- und get-Methoden(ist das sinnvoll?).

Die Klasse Mainwindow erzeugt das Frame, in welchem sich alles abspielt.
Damit nicht alles unübersichtlich wird und man es in Zukunft besser erweitern kann, habe ich die GUI-Elemente in andere Klassen ausgelagert (ist das sinnvoll? Oder wie macht man es am besten?).

Es gibt momentan 3 relevante GUI-Elemente:
- das Menü oben
- die Toolbar
- die Tabelle
In der Klasse Mainwindow passiert sonst eigentlich nichts. Die ActionEvents laufen jeweils in den ausgegliederten Klassen ab(geht bestimmt auch eleganter, für Input wäre ich dankbar!).

Vorher hat alles funktioniert, aber ich habe die ganze Zeit Objekte und Arrays hin und her geschoben (als Parameter für Konstruktoren), bis es irgendwann einfach total unübersichtlich und einfach "unsauber" war.
Deswegen will ich es nun anders lösen.

erst einmal die Klassen.

Mainwindow:

Code:

package main;

import java.awt.BorderLayout;
import java.awt.GridLayout;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;

import main.ui.Menu;
import main.ui.Table;
import main.ui.Toolbar;


public class Mainwindow extends JFrame {
	private static final long serialVersionUID = 1L;

	private JPanel south = new JPanel();

	private JLabel anzahlderFilme;
	
	
	/**
	 * Konstruktor
	 */
	public Mainwindow (Manager manager) {
		super("Movie Manager");
		setResizable(true);
		setLayout(new BorderLayout());
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
		
		// Menü
		setJMenuBar(new Menu());

		// Toolbar(north)
		add(new Toolbar(), BorderLayout.NORTH);

		
		// Tabelle (center)
		add(new JScrollPane(new Table ()), BorderLayout.CENTER);

		
		// Anzeige (south) -> WIRD NOCH NICHT GEUPDATED! <-
		south.setLayout(new GridLayout (1, 1));
		anzahlderFilme = new JLabel ("Anzahl der Filme:  " + manager.getlistlength());
		south.add(anzahlderFilme);
		add(south, BorderLayout.SOUTH);
		
		// packen und sichbtbar machen
		pack();
		setVisible(true);		
	}
	
	/**
	 * Erzeugt das Fenster
	 * @param args
	 */
	public static void main(String[] args) {
		Manager manager = new Manager();
		manager.importlist();
		Mainwindow m = new Mainwindow(manager);
	}

}

Menu:

Code:

package main.ui;

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

import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;

import main.Manager;


public class Menu extends JMenuBar implements ActionListener {
	private static final long serialVersionUID = 1L;

	private JMenuItem add, del, sortbyname, sortbyimdbrating, sortbymmrating, 
	sortbygenre, ueber, changename, changeimdbrating, changemmrating, 
	changegenre;


	/**
	 * Konstruktor
	 */
	public Menu () {
		
		// Menü "Datei"
		JMenu datei = new JMenu("Datei");
		add = new JMenuItem ("Film hinzufügen");
		add.addActionListener(this);
		datei.add(add);
		del = new JMenuItem ("Film löschen");
		del.addActionListener(this);
		datei.add(del);
		this.add(datei);
		
		// Menü "Bearbeiten"
		JMenu bearbeiten = new JMenu ("Bearbeiten");
		changename = new JMenuItem ("Filmnamen ändern");
		changename.addActionListener(this);
		bearbeiten.add(changename);
		changeimdbrating = new JMenuItem ("imdb.com-Bewertung ändern");
		changeimdbrating.addActionListener(this);
		bearbeiten.add(changeimdbrating);
		changemmrating = new JMenuItem ("moviemaze.de-Bewertung ändern");
		changemmrating.addActionListener(this);
		bearbeiten.add(changemmrating);
		changegenre = new JMenuItem ("Filmgenre ändern");
		changegenre.addActionListener(this);
		bearbeiten.add(changegenre);
		this.add(bearbeiten);
		
		// Menü "Sortieren"
		JMenu sortieren = new JMenu ("Sortieren");
		sortbyname = new JMenuItem ("Nach Namen sortieren");
		sortbyname.addActionListener(this);
		sortieren.add(sortbyname);
		sortbyimdbrating = new JMenuItem ("Nach imdb.com-Bewertung sortieren");
		sortbyimdbrating.addActionListener(this);
		sortieren.add(sortbyimdbrating);
		sortbymmrating = new JMenuItem ("Nach moviemaze.de-Bewertung sortieren");
		sortbymmrating.addActionListener(this);
		sortieren.add(sortbymmrating);
		sortbygenre = new JMenuItem ("Nach Genre sortieren");
		sortbygenre.addActionListener(this);
		sortieren.add(sortbygenre);
		this.add(sortieren);
		
		// Menü "Hilfe"
		JMenu hilfe = new JMenu ("Hilfe");
		ueber = new JMenuItem ("Über MovieManager");
		ueber.addActionListener(this);
		hilfe.add(ueber);
		this.add(hilfe);
	}

	/**
	 * ActionEvents
	 */
	public void actionPerformed(ActionEvent e) {
		if (e.getSource() == add) {
			String name = JOptionPane.showInputDialog("Filmname");
			if (name.length() == 0) {
				JOptionPane.showMessageDialog(null, "Dateinamen angeben.");
			} else {
				Manager manager = new Manager();
				manager.add(name);
				Table table = new Table();
				table.refresh();			
			}
		}

		if (e.getSource() == del) {
		}

		if (e.getSource() == sortbyname) {
			Manager manager = new Manager();
			manager.sortlistbyname();
		}

		if (e.getSource() == sortbyimdbrating) {
			Manager manager = new Manager();
			manager.sortlistbyimdbrating();
		}
		
		if (e.getSource() == sortbymmrating) {
			Manager manager = new Manager();
			manager.sortlistbymmrating();
		}

		if (e.getSource() == sortbygenre) {
			Manager manager = new Manager();
			manager.sortlistbygenre();
		}

	}

}

Toolbar:

Code:

package main.ui;

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

import javax.swing.JButton;
import javax.swing.JOptionPane;
import javax.swing.JToolBar;

import main.Manager;


public class Toolbar extends JToolBar implements ActionListener {
	private static final long serialVersionUID = 1L;
		
	private JButton addMovie;
	private JButton deleteMovie;
	private JButton sortMoviesbyName;
	private JButton sortMoviesbyGenre;
	private JButton sortMoviesbyRating;
	
	
	/**
	 * Konstruktor
	 */
	public Toolbar () {	
		addMovie = new JButton ("add");
		addMovie.addActionListener(this);
		deleteMovie = new JButton ("delete");
		deleteMovie.addActionListener(this);
		sortMoviesbyName = new JButton ("sort by Name");
		sortMoviesbyName.addActionListener(this);
		sortMoviesbyRating = new JButton ("sort by Rating");
		sortMoviesbyRating.addActionListener(this);
		sortMoviesbyGenre = new JButton ("sort by Genre");
		sortMoviesbyGenre.addActionListener(this);
		
		this.add(addMovie);
		this.add(deleteMovie);
		this.add(sortMoviesbyName);
		this.add(sortMoviesbyRating);
		this.add(sortMoviesbyGenre);
	}
	
	/**
	 * ActionEvents
	 */
	public void actionPerformed(ActionEvent e) {
		if (e.getSource() == addMovie) {
			String name = JOptionPane.showInputDialog("Filmname");
			if (name.length() == 0) {
				JOptionPane.showMessageDialog(null, "Dateinamen angeben.");
			} else {
				Manager manager = new Manager();
				manager.add(name);
				Table table = new Table();
				table.refresh();			
			}
		}

		if (e.getSource() == deleteMovie) {
		}

		if (e.getSource() == sortMoviesbyName) {
		}

		if (e.getSource() == sortMoviesbyGenre) {
		}

		if (e.getSource() == sortMoviesbyRating) {
		}
	}
	
}

Table:

Code:

package main.ui;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;

import main.Manager;


public class Table extends JTable implements ActionListener {
	private static final long serialVersionUID = 1L;
		
	private static Object[] columnNames = {"Filme", "imdb.com-Rating", "moviemaze.de-Rating", "Genre"};
	private static DefaultTableModel dtm;
	
	private static Manager manager;
	
	
	/**
	 * Konstruktor
	 * @param manager
	 */
	public Table () {
		manager = new Manager();
		dtm = new DefaultTableModel(manager.gettable(), columnNames);
		this.setModel(dtm);
	}
	
	/**
	 * ActionEvent
	 */
	public void actionPerformed(ActionEvent e) {
		// TODO Auto-generated method stub
	}
	
	/**
	 * Zeichnet die Tabelle neu
	 */
	public void refresh () {
		manager.convert();
		dtm.setRowCount(manager.getlistlength());
		dtm.fireTableStructureChanged();
		dtm.setDataVector(manager.gettable(), columnNames);
		dtm.fireTableDataChanged();
	}

}

Manager (hier ist auch viel in andere Klassen ausgelagert, aber die anderen Klassen sollten eigentlich keine Rolle spielen. Falls ja, poste ich sie natürlich gerne):

Code:

package main;

import main.system.Movie;
import main.system.Port;
import main.system.Sort;
import main.system.Search;


public class Manager {
	static private Movie[] list = new Movie[0];
	static private Object[][] table;

	/**
	 * Konstruktor: erstellt beim Aufruf eine neue Movieliste (array)
	 */
	public Manager () {
		convert();
	}

	/**
	 * Gibt das Movieobjekt an der Stelle "index" im Array zurück
	 * @param index int: index-Zahl
	 * @return Movieobjekt
	 */
	public Movie getmovie (int index) {
		return list[index];
	}

	/**
	 * Gibt die Anzahl der Filme in der Movieliste zurück
	 * @return int: Anzahl der Filme in der Movieliste
	 */
	public int getlistlength () {
		return list.length;
	}
	/**
	 * Gibt die Tabelle "table" zurück
	 * @return String[][]:zweidimensionales Array der Movieliste
	 */
	public Object[][] gettable () {
		return table;
	}

	/**
	 * Konvertiert die Liste in ein zweidimensionales Array (für JTable)
	 */
	public void convert() {
		table = new String[list.length][4];
		for (int i = 0; i < list.length; i++) {
			table[i][0] = getmovie(i).getname();
			table[i][1] = getmovie(i).getimdbrating();
			table[i][2] = getmovie(i).getmmrating();
			table[i][3] = getmovie(i).getgenre();
		}		
	}

	/**
	 * Fügt einen neuen Film in die Movieliste hinzu
	 * @param name String: Name des Films, welcher hinzugefügt werden soll
	 */
	public void add (String name) {
		Movie[] temp = new Movie[list.length + 1];

		// alte Liste kopieren
		for (int i = 0; i < list.length; i++) {
			temp[i] = list[i];
		}

		// neues Element hinzufügen
		temp[temp.length-1] = new Movie(name);

		// zu liste referenzieren
		list = temp;
	}

	/**
	 * Löscht einen Film aus der Movieliste
	 * @param index int: Index-Zahl des Films im Array, welcher gelöscht 
	 * werden soll
	 */
	public void delete (int index) {
		Movie[] temp = new Movie[list.length - 1];

		// alte Liste bis zu "index" kopieren
		for (int i = 0; i < index; i++) {
			temp[i] = list[i];
		}

		// kopiere alles, was nach dem zu löschendem Element kommt, weiter in
		// die Liste
		for (int i = index; i < temp.length; i++) {
			temp[i] = list[i+1];
		}

		// zu liste referenzieren
		list = temp;
	}

	/**
	 * Sortiert die Liste alphabetisch nach den Namen der Filme
	 */
	public void sortlistbyname () {
		Sort sort = new Sort();
		list = sort.sortlistbyname(list);
	}

	/**
	 * Sortiert die Liste alphabetisch nach dem Genre der Filme
	 */
	public void sortlistbygenre () {
		Sort sort = new Sort();
		list = sort.sortlistbygenre(list);
	}

	/**
	 * Sortiert die Liste nach dem imdb.com-Rating der Filme
	 */
	public void sortlistbyimdbrating () {
		Sort sort = new Sort();
		list = sort.sortlistbyimdbrating(list);
	}
	
	/**
	 * Sortiert die Liste nach dem moviemazde.de-Rating der Filme
	 */
	public void sortlistbymmrating () {
		Sort sort = new Sort();
		list = sort.sortlistbymmrating(list);
	}

	/**
	 * Sucht in der Movieliste einen Film (vollständiger Name notwendig)
	 * @param name String: Name des Films, welcher gesucht werden soll
	 * @return int: Index-Zahl im Array des Films. -1, falls Film nicht gefunden
	 */
	public int searchbyfullname (String name) {
		Search search = new Search();
		return search.searchbyfullname(list, name);
	}

	/**
	 * Exportiert die Movieliste in eine Datei (vorher wird sie nach Namen 
	 * sortiert)
	 */
	public void exportlist () {
		sortlistbyname();

		Port export = new Port();
		export.exportlist(list);
	}

	/**
	 * Importiert die Movieliste aus einer Datei 
	 */
	public void importlist () {
		Port port = new Port();
		list = port.importlist();
	}



}

Nun funktioniert das Tabellen neuzeichnen nicht mehr (ka wieso).
Wenn ich über System.out.println() von dtm zB die Anzahl der Zeilen ausgebe nach dem hinzufügen eines Films, stimmts (scheint also, als ob er alles richtig gemacht hätte). Aber irgendwie will er im Frame die Tablle nicht neu zeichnen, die ändert sich einfach nicht.
Beim initialisieren macht er alles richtig (also import aus einer Datei). Hier passt die Tabelle. Nur ändert sich dieser Zustand nie :/

Ich wäre über Feedback zum Code (auch wenn es nicht zur Lösung meines Problems hilft) und allg Tipps sehr dankbar.
Über die Lösung des Problems natürlich auch

tia Denne

momentan habe ich nur das Event "Film hinzufügen" vollständig implementiert und dort wird am schluss die reflesh()-Methode in Table aufgerufen. Also ich gehe schon davon aus, dass sie aufgerufen werden sollte.

Ich vermute den Fehler an 2 Stelle:
- das mit der Objekterzeugung und den statischen Klassenattributen funktioniert nicht so, wie ich es mir vorstelle
- die reflesh()-Methode in der Klasse Table ist nicht ganz richtig.

Aber wie gesagt, ich habe kaum bis gar keine Erfahrung mit größeren Projekten und ich verzweifel schon den ganzen Tag daran, wie ich es am besten mit OO-Design umsetze.

@ Model statt Manager: Du meinst bei der Tabelle mit Models arbeiten, statt das über den Manager und dem Array zu machen? Versteh es nicht ganz, sorry.

@ Collection: Ich kenne hier leider nur die LinkedList. Als ich mit der Programmierung angefangen habe, war ich mir nicht sicher, was besser ist. Dadurch, dass Filme hinzugefügt und gelöscht werden, ist die LinkedList was Dynamik angeht natürlich besser (wobei es bei einem Array jetzt nicht so der Aufwand ist imho).
Ich habe mich dann aber für ein Array entschieden, weil ich über die index-Zahl sofort auf jedes Array zugreifen kann, ohne immer durch die Liste laufen zu müssen.
Zum Datenmodel:
Das Array besteht aus Movieobjekten, welche so aussehen:

Code:

package main.system;

public class Movie {
	// Name des Films
	private String name;
	// Genre des Films
	private String genre = "kein Genre gesetzt";
	// Rating der Seite imdb.com
	private String imdbrating = "kein Rating gesetzt";
	// Rating der Seite moviemaze.de
	private String mmrating = "kein Rating gesetzt";
	// url zu einer Seite (zb imdb.com)
	private String url;
	
	/**
	 * Konstruktor
	 * @param name String: Name des Filmes, welcher hinzugefügt werden soll
	 */
	public Movie (String name) {
		this.name = name;
	}
	
	public void setname (String name) {
		this.name = name;
	}
	
	public String getname() {
		return this.name;
	}
	
	public void setgenre (String genre) {
		this.genre = genre;
	}
	
	public String getgenre() {
		return this.genre;
	}

	public void setimdbrating (String rating) {
		this.imdbrating = rating;
	}
	
	public String getimdbrating() {
		return this.imdbrating;
	}
	
	public void setmmrating (String rating) {
		this.mmrating = rating;
	}
	
	public String getmmrating() {
		return this.mmrating;
	}
	
	public void seturl (String url) {
		this.url = url;
	}
	
	public String geturl() {
		return this.url;
	}
}

@ JMenu, JTable und JToolBar: Ja, einfacher ist es, wenn ich es alles in eine GUI-Klasse haue (ich denke, dann würd das mit dem refresh auch funktionieren). Aber wenn ich alles schön erweiter (das momentan ist ja nur das Grundgerüst) hat die Klasse am Schluss 1k+ Zeilen und dann wirds doch sehr unübersichtlich, deswegen wollte ich es in andere Klassen auslagern.
Hat hier wer Erfahrung, wie man das am besten macht? Oder wirklich einfach alles in eine Klasse? :/

@ MVC: Ja, habe ich schon von gehört, habe es aber nie richtig "gelernt".
Grob habe ich es mir so vorgestellt: Manager = Controller, Mainwindow = View und Manager ist dann wohl auch das Model :/
Nicht perfekt, aber wie mach ich es in meinem speziellen Fall besser?

Thx für das Feedback, sowas brauche ich! Bitte mehr davon

@ LinkedList: Hast schon Recht, werde es dann mal umbauen.

@ GUI: Auch hier wird wohl alles abgeändert werden (weil ich so momentan eh nicht weiter komme wegen meinem refresh()-Problem)

Thx für den Input, werde mich dann mal ranmachen, alles umzubauen.
Werde hier dann das Ergebnis posten (vorher eventuell Fragen haben, falls ich wieder feststecke ), ob sich mein Problem erledigt hat etc.

lg Denne