wie kann ein Thread beendet werden?

Status
Nicht offen für weitere Antworten.

sensor

Mitglied
Für die Darstellung einer Tabelle wird die Standardausgabe:

Code:
        public static void ShowGUI()
       {
    	//Schedule a job for the event-dispatching thread:
        //creating and showing this application's GUI.
        javax.swing.SwingUtilities.invokeLater(new Runnable()
        {
            public void run()
            {
                createAndShowGUI();
            }
        });
verwendet.
Wird diese Thread in einer Mainclass gestartet wird der Thread nach einmaliger Ausführung korrekt beendet.
In meinem Fall wird der Thread in einer Public class aufgerufen und läuft dann n -mal bis ein Systemevent diesen zwangweise z.B. wegen Stack Überlauf beendet.

Wie kann dieser Thread korrekt beendet werden. Irgend etwas mit Timer habe ich gefunden. Das kann aber nicht die Lösung sein. Oder ist die Konstruktion grundlegend fasch.

Anbei der komplette Code:

Code:
import java.awt.event.*;
import java.awt.Dimension; 
import java.awt.event.ActionEvent;
import java.awt.event.FocusListener;
import java.awt.event.ItemEvent;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.Color;
import java.awt.GridLayout;
import java.awt.List;
import java.awt.TextField;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
import javax.swing.*;
import java.text.ParseException;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import java.lang.String;
import java.util.Formatter;
import java.awt.*;
import javax.swing.InputVerifier;
import javax.swing.JCheckBox;
import javax.swing.JComponent;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JTextField;
import javax.swing.SwingUtilities;
import javax.swing.event.DocumentEvent;
import javax.swing.event.DocumentListener;
import javax.swing.text.BadLocationException;
import javax.swing.text.Document;
import javax.swing.text.JTextComponent;
import javax.swing.JButton; 
import javax.swing.JPanel;
import javax.swing.border.*;
import java.text.DateFormat;
import java.util.Calendar;
import java.util.EventObject;
import java.util.GregorianCalendar;
import java.util.Date;
import java.io.*;
import java.lang.Math;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

public class Fahrzeugtabelle extends JPanel implements MouseListener, KeyListener {
	
	  private  BearbeitenAltfahrzeuge zellen_inhalt=null;
	  public void setBearbeitenAltfahrzeuge (BearbeitenAltfahrzeuge zellen_inhalt) 
	  {
		  this.zellen_inhalt=zellen_inhalt; 	
	  }
 
//Variablen-----------------------------------------------------------------------	
	private boolean DEBUG = false;
	
	public	String      status="0",        
    					xdate="x",
    					blank="",
    					old_key;	
	public String       query="",
    					insert="",
    					delete="",
    					update="",
    					count="";
	static String       kdid= "",
    					kdfirma= "",
    					kdnachname= "",
    					kdvorname= "",
    					kdstrasse= "",
    					kdstrassenr= "",
    					Zelleninhalt= "";
	private String    	key= "0";
						
						
	private int			row,				//Nummer der Tabelle/Reihe
						column;				//Nummer der Tabelle/Spalte (Zellen Koordinate)
	public	TextField   txtf_annahmedatum = null;	
//	public  List        lst_laufendenummer;
//Ende Variablen----------------------------------------------------------------------	
	
	

	
	JFrame fahrzeugtabelle;
    public Fahrzeugtabelle() { 
 
       super(new GridLayout(1,0));
    
        String[] columnNames = {"Lfd.Nummer",
                                "Datum",
                                "Kennzeichen",
                                "Fahrgestellnummer",
                                "Anlieferername",
                                "Modell",
                                "Status"
                                };
        

        	
       	CheckInput check=new CheckInput();  // Classen-Aufruf 
       	ConnectDb db = new ConnectDb();     // Classen-Aufruf, Datenbank MYSQL Anbindung  
/*        	
      	//ERZEUGEN DES ASSIGNICON_OBJEKTS
        AssignIcon Icon = new AssignIcon();
        //AUFRUF DER METHODE ZUM EINFÜGEN DES BILDES, DIESER METHODE WIRD DAS FRAME ÜBERGEBEN
        Icon.AssignIcon(fahrzeugtabelle);   	
*/
        

//Datenfelder LFD-Nummer,Anlieferer Name, Annahme Datum, Kennzeichen, Fahrgestellnummer, Modell aus der Datenbank holen und in das Feldobjekt data übertragen       
         String query="SELECT lfdidchar,lfdannahmedatum,lfd_letztamtkennzeichen,lfd_fahrzeugidentnr,lfd_anlieferername,lfd_modell,lfd_status" +
   		   "  FROM tbl_vgaf where lfdidchar like '%'";
         String count="SELECT COUNT(*) FROM tbl_vgaf where lfdidchar like '%'";     
    	System.out.println("query : "+query);
    	System.out.println("count : "+count);	
    	Object[][] data = null;    	   	    
    	data = ConnectDb.Query_tbl_vgaf(query,count);
 

//    	 Tabelle: table  aufbauen        
        final JTable table = new JTable(data, columnNames);   					//Name ist Table, data sind row- und column-daten, columnnames sind als String vorgegeben
        table.setPreferredScrollableViewportSize(new Dimension(1000, 100));  	// X,Y-Dimension der Tabelle festleggen
        table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);

        table.addMouseListener(new MouseAdapter() { 							// Mouse Listener hinzufügen welcher den Wert bei Mousklick aus dem Feld holt.
        public void mouseClicked(MouseEvent e) {
//        	System.out.println("Mouseklick");
        	int i = table.getSelectedRow();										// Die Reihe auslesen
        	int j = table.getSelectedColumn();									// die Spalte auslesen
        	Zelleninhalt = String.valueOf(table.getValueAt(i, j));				// Umwandel des Objekts in eine String, Inhalt der Zelle steht in table.getValueAt(i, j)
        	System.out.println("Mouseklick Zelleninhalt: "+Zelleninhalt+" Zeile-Nr.: "+i+" "+"Spaltennummer: "+j);
        	if (j==0)   // 0 = Laufende Nummer
        	{
        		BearbeitenAltfahrzeuge bearbeitenAltfahrzeuge = new BearbeitenAltfahrzeuge();	// Bereitstellung des aufrufenden Objektes für die Datenkommunikation
        	}
        	
        }
        });
        		
        
        if (DEBUG) {
            table.addMouseListener(new MouseAdapter() {
               public void mouseClicked(MouseEvent e)
               {
                   printDebugData(table);
                }
            });
        }

        //Create the scroll pane and add the table to it.
        JScrollPane scrollPane = new JScrollPane(table);

        //Add the scroll pane to this panel.
        add(scrollPane);

        ShowGUI();
        
    } //Ende public Fahrzeugtabelle()

    private void printDebugData(JTable table) {
        int numRows = table.getRowCount();
        int numCols = table.getColumnCount();
        javax.swing.table.TableModel model = table.getModel();

        System.out.println("Value of data: ");
        for (int i=0; i < numRows; i++) {
            System.out.print("    row " + i + ":");
            for (int j=0; j < numCols; j++) {
                System.out.print("  " + model.getValueAt(i, j));
            }
            System.out.println("private void printDebugData(JTable table)");
        }
        System.out.println("--------------------------");
       
    }

    /**
     * Create the GUI and show it.  For thread safety,
     * this method should be invoked from the
     * event-dispatching thread.
     */
    private static void createAndShowGUI() {
        //Create and set up the window.
        JFrame tabelle = new JFrame("Alle bisher angenommene Fahrzeuge");
        tabelle.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        //Create and set up the content pane.
        Fahrzeugtabelle newContentPane = new Fahrzeugtabelle();
        newContentPane.setOpaque(true); //content panes must be opaque
        tabelle.setContentPane(newContentPane);

        //Display the window.
        tabelle.pack();
        tabelle.setVisible(true);
    }
//  ########### public void mouseClicked(MouseEvent arg0)#####################
//  Methode Feld aus Tabelle auslesen
    
    public void keyReleased(KeyEvent arg0) // Methode keyReleased
    {
    	System.out.println("keyReleased");
 
    	{
    		
    	}
     };
    public void mouseClicked(MouseEvent e)
    {
    	if(e.toString()!="" || e.toString()!=null)
    	{
    		Zelleninhalt = e.toString();
    		System.out.println("mouseClicked Zelleninhalt: "+Zelleninhalt);
    	}
    		else
    		{
    			Zelleninhalt = e.toString();
    			System.out.println("mouseClicked Zelleninhalt leer: "+Zelleninhalt);
    		}
    }
    
    public void mousePressed(MouseEvent e) {	//Muss sein wegen Mouselistener
    	System.out.println("MousePressed");
    	
     }

     public void mouseReleased(MouseEvent e) { //Muss sein wegen Mouselistener
    	 System.out.println("MouseReleased");
     	
     }

     public void mouseEntered(MouseEvent e) {	//Muss sein wegen Mouselistener
    	 System.out.println("MouseEntered");
     	
     }

     public void mouseExited(MouseEvent e) {	//Muss sein wegen Mouselistener
    	 System.out.println("MouseExited");
     	
     }
    
 //   Aufruf des Thread ------------------------------------------------------
     public static void ShowGUI()
       {
    	//Schedule a job for the event-dispatching thread:
        //creating and showing this application's GUI.
        javax.swing.SwingUtilities.invokeLater(new Runnable()
        {
            public void run()
            {
                createAndShowGUI();
            }
        });
//  Ende Thread----------------------------------------------------------------------       
    }

	public void keyPressed(KeyEvent arg0)
	{
		System.out.println("keyPressed");
		
	}

	public void keyTyped(KeyEvent arg0)
	{
		System.out.println("keyTyped");
		
	}

}

Vorab vielen Dank wenn Ihr eine Lösung gefunden habt.

MfG

sensor
 
Von echten Threads, die zu beenden waeren, finde ich in deinem Code nichts. Du versuchst die Initialisierung an den EventQueue des GUI Threads delegieren, das wars schon.

Allerding sehe ich ein klassische Endlosschleife:

1. die Methode ShowGUI() sagt dem EventQueue er soll einmal createAndShowGUI() aufrufen wenn er mit alle Events fertig ist
2. die Methode createAndShowGUI() legt ein JFrame an und eine Fahrzeugtabelle
3. der Constructor von Fahrzeugtabelle ruft ganz am Ende ShowGUI() auf -> wir setzen bei Puink 1 wieder fort

Bist Du Dir sicher dass Du weisst was Du tust? Wahrscheinlich ist Punkt 3 der Fehler.
 
Irgendwo in deinem Programmablauf kommt der Punkt, an dem Du ein JFrame mit einer Fahrzeugtabelle aufbauen und darstellen möchtest (beispielsweise in der Methode public static void main(String[]) deiner Main Klasse). An der Stelle rufst Du Fahrzeugtabelle.ShowGUI(); auf. Den Aufruf von ShowGUI(); in deinem Code Zeile 169 musst Du entfernen.

Die Initialisierung der GUI über den EventQueue erachte ich zwar nicht als notwendig, aber auch nicht als schädlich.

PS: ShowGUI() am besten auch klein schreiben, da es eine Methode ist - sieht einfach besser aus.
 
Danke für die Antwort,
wenn ich den Aufruf herausnehme, ist zwar die Schleife unterbrochen, aber meiner Tabelle wird nicht angezeigt, weil der Konstruktur (Fahrzeugtabelle) zu Ende ist und verlassen wird.
Der Konstruktur Fahrzeugtabelle() wird von einem anderen public class via

Code:
         	  pos=noname.indexOf(subname211);
//          ************************************* Objekt erzeugen* Class-Aufruf       		  
     		  if (pos >= 0)                      // >=0: gefunden -1: nicht gefunden	
     		  {   
     		  Fahrzeugtabelle fahrzeugtabelle = new Fahrzeugtabelle(); 
     		  setposminuseins();                  //  Methode setposminuseins();
     		  }
aufgerufen.

Irgendwie habe ich mit der Aufrufs-Kommunikation von Methoden und Konstrukturen zwischen den Classen noch Verständnisprobleme.
Wenn ich das ganze aus einer Mainclass aufrufe, funktionierts problemlos. Leider kann ich in der aufrufenden Klasse keinen "Fahrzeugtabelle.(String[] args)" absetzen, da "args" unbekannt ist.

Hast Du noch einen Tipp
mfg
 
Code:
if (pos >= 0)                      // >=0: gefunden -1: nicht gefunden   
{   
    Fahrzeugtabelle.ShowGUI();
    setposminuseins();                  //  Methode setposminuseins();
}

sollte funktionieren. Denn an der Stelle willst du ja nicht nur eine Fahrzeugtabelle erstellen sondern eigentlich ein Fenster erstellen, das eine Fahrzeugtabelle enthält und dieses Fenster dann anzeigen. Im Gegensatz dazu ist der Constructor von Fahrzeugtabelle dazu da, ein neues Objekt Fahrzeugtabelle zu erstellen - und nicht um ein Fenster zu bauen. Deswegen sollte der Constructor kein ShowGUI() aufrufen.

desweiteren würde ich noch ändern:
Eine Fahrzeugtabelle ist ein GUI Element - wieso soll die Klasse Fahrzeugtabelle dafür verantwortlich sein, ein Fenster zu initialisieren und darzustellen? Also die Methoden ShowGUI(); und createAndShowGUI(); aus der Klasse Fahrzeugtabelle rausnehmen und in die Klasse verschieben, in der Du wirklich die gesamte GUI - das Fenster inklusive der Fahrzeugtabelle initialisieren willst.

Ansonsten solltest Du beim Aufbau objektorientiert denken. Ich weiss, das sagt sich immer so leicht, es braucht halt ein wenig Übung. Eigentlich geht es nur darum, Aufgaben und Daten logisch zu kapseln -> Deine Klasse Fahrzeugtabelle hat die Aufgabe Fahrzeugdaten aus einer Datenbank zu holen und grafisch darzustellen. Also sollten in Fahrzeugtabelle keine Methoden enthalten sein, um ein Fenster zu erzeugen. Wenn man das weiter denkt, könnte man noch sagen, dass man lieber 2 Klassen hat: 1. eine Fahrzeugtabelle um die Daten anzuzeigen und 2. eine andere Klasse FahrzeugDataCollector um die Daten aus der Datenbank zu holen. Dazu am besten noch eine Klasse Fahrzeug, um jeweils einen Datensatz zu kapseln.
Die Fahrzeugtabelle braucht sich dann nicht darum zu kümmern, wo die Daten her kommen, ob aus einer DB oder sonstwo - die kriegt einfach eine Liste von Fahrzeugen und zeigt sie an.
So etwas zu bauen scheint zwar im erstem Moment ein größerer Aufwand zu sein - mehr Klassen, eventuell noch ein eigenes Tablemodel... aber am Ende ist der Aufbau wesentlich leichter zu verstehen, zu erweitern, zu warten und wiederzuverwenden.
 
Der Aufruf funktioniert! toll, vielen Dank.

Das mit der Aufteilung in mehrere Klasssen werde ich mir zu Herzen nehmen und auch umsetzen. Es ist sehr ungewohnt, wenn man bisher prozedural gedacht hat, alles in Objekte zu stückeln.
 
Status
Nicht offen für weitere Antworten.

Neue Themen


Zurück
Oben