Aufruf von Klasse mit Runnable versteh ich nicht

[Verjigorm]

Hallo, ich habe da folgende Klasse gefunden

import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;


public abstract class Query implements Runnable
{
	private ResultSet result;
	private Connection con;
	private String sql;

	public abstract boolean readRow(ResultSet result) throws SQLException;
	
	public Query(Connection _con,String _sql){
		con = _con;
		sql = _sql;
	}

	public void run(){
		PreparedStatement st=null;
		ResultSet result=null;
		try{
		st=con.prepareStatement(sql);
		result = st.executeQuery();
		while(result.next() && readRow(result));
		
		}catch (SQLException e) {}
		
		finally{
			try {
				if(result!=null){
					result.close();
				}
			} catch (SQLException e) {}
			if(st!=null){
				try {
					st.close();
				} catch (SQLException e) {}
			}
		}
	}
	
	public int parseInt(int column) throws SQLException{
		return Integer.parseInt(result.getString(column));
	}
	
	public double parseDouble(int column) throws SQLException{
		return Double.parseDouble(result.getString(column));
	}
}

aufgerufen wird das Ding mit:

		new Query(param1, param2)
		{

			@Override
			public boolean readRow(ResultSet result) throws SQLException {
				// TODO Auto-generated method stub
				return false;
			}
			
		}.run();

Kann mir vielleicht jemand erklären, was das für nen Sinn hat?
Das ganze wird doch nicht als nebenläufiger Thread ausgeführt oder doch?
Ich dachte immer das müsste so aufgerufen werden:

		new Thread(
		new Query(param1, param2)
		{

			@Override
			public boolean readRow(ResultSet result) throws SQLException {
				// TODO Auto-generated method stub
				return false;
			}
			
		}).start();

Was ist denn nun richtig? *verwirrtbin*

mfg Verjigorm

 

[MiDniGG]

Naja. Wenn ich mich nicht täusch dann ist diese Query klasse mehr ne Schleife die en paar mal durchläuft... Ich find jetzt die Methodennamen wahl nicht so geschickt, da man eben durch run() unzo leicht auf nen Thread schließen kann...


Korrigiert mich bitte, wenn ich falsch liege!!!

 

[Verjigorm]

Die Query-Klasse implementiert doch Runnable
wieso soltle sie das, wenn sie nicht für Thread sein soll??

 

[MiDniGG]

Oh f... das hab ich übersehen... sorry
Dann bin ich jetzt auch verwirrt...

 

[Verjigorm]

Also heisst das, mein Vorgänger (von dem ist die Klasse) wollte eigentlich eine nebenläufige Abarbeitung eines Querys, hats aber verbockt?

Anders kann ich mirs nicht erklären

 

[MiDniGG]

Naja. Also wenns funktioniert, dann hat ers wohl irgendwie doch richtig gemacht...

Aber (hab nomma in der API geschaut) sollte man das doch eher so machen, wie du das geschrieben hast also den Thread-Block drum rum...

 

[Verjigorm]

Ok, ich hab die Zeile gefunden, die ich gesucht habe:

http://www.galileocomputing.de/open...10_002.htm#mj748d4f52c301c44d2d30adbcad1c9bce

10.2.2 Thread mit Runnable starten

Nun reicht es nicht aus, einfach die run()-Methode einer Klasse direkt aufzurufen. Würden wir dies tun, dann wäre nichts nebenläufig, sondern wir würden einfach eine Methode sequenziell ausführen. Damit der Programmcode parallel zur Applikation läuft, müssen wir ein Thread-Objekt mit dem Runnable verbinden und dann den Thread explizit starten. Dazu übergeben wir dem Konstruktor der Klasse Thread eine Referenz auf das Runnable-Objekt und rufen start() auf. Nachdem start() für den Thread eine Ablaufumgebung geschaffen hat, ruft es intern selbstständig die Methode run() genau einmal auf. Läuft der Thread schon, so löst ein zweiter Aufruf der start()-Methode eine IllegalThreadStateException aus.

 

[MiDniGG]

Joa steht in der API auch drin:

http://java.sun.com/j2se/1.5.0/docs/api/java/lang/Thread.html

 

[Verjigorm]

Ich habs mal als Thread gestartet, aber dann kommt keinerlei Ausgaben ... öhm ich bin immer mehr verwirrt ...

edit: ahso, alle exceptions ausgeschaltet, na da wundert mich nix

danke, ich glaube ich kriegs dann alleine hin ^^