Hey Leudels,

Ich schilder euch mal kurz mein Problem:

Also ich lese via Java StringTokenizer eine Tabelle aus einer .txt Datei ein und lasse diese direkt in eine Datenbank eintragen.

Das problem hierbei ist jetzt, dass diese .txt Datei ca. 5000 Datensätze enthält, und dieser ganze vorgang so fast 20 min dauert?!?!?!

Wie kann ich denn die Performance dabei erhöhen? Ist es vielleicht sinnvoll alles erstmal in einen Vektor oder ein Feld zu schreiben, und es dann von dort erst eintragen zu lassen? Oder gibt es andere Möglichkeiten?

mfg

Steve

[3qualizer]

BufferedStreamReader, StringBuffer etc... ohne zu wissen wie du ausliest bzw in den Speicher holst wirds schwierig das zu sagen
/edit: Was ich damit meine: gib uns ein bisschen Code, im Idealfall so viel das es kompilierbar ist.

import java.io.*;
import java.util.StringTokenizer;
import java.sql.*;

public class CarsReader {

//Konstruktor

MyConnection x = new MyConnection();

public CarsReader()
{

}

public void lesen(String datei) {
try
{
FileReader f = new FileReader (datei);

String alles = "";
int c;
while (( c = f.read()) != -1)
{
alles = alles + (char)c; //die alte zeile wird mit dem neuen überschrieben
}

//Zeile zerlegen
StringTokenizer st = new StringTokenizer (alles, "\n");

while(st.hasMoreTokens())
{
String zeile = st.nextToken();
StringTokenizer stto = new StringTokenizer (zeile, "\t");

int anzahl = stto.countTokens();

if(anzahl>4)
{
String s1 = stto.nextToken();
String s2 = stto.nextToken();
String s3 = stto.nextToken();
String s4 = stto.nextToken();
String s5 = stto.nextToken();

eintragen(s1,s2,s3,s4);
}
}
}
catch(Exception e)
{
System.out.println("Fehler");
}


public void eintragen(String s1,s2,s3,s4)
{
try
{

x.stmt.executeUpdate("INSERT INTO CARS (Family, Analysis) VALUES ('"+s3+"','"+s4+"')");

}

catch(SQLException ex)
{
System.out.println("F***ing Fehler!");
}

}
}



So, das mal der Code..hehe

[3qualizer]

Wie ich schon erwähnt hab ist gebuffertes lesen u.U. erheblich performanter. Ausserdem kannst du auch mit BufferedReader#readline() ganze Zeilen lesen statt dich mit den einzelnen Zeichen zumzuschlagen

/edit: s5 bleibt so wies mir scheint ungenutzt... kann also weg

Also den BufferedReader werd ich gleich mal ausprobieren..meld mich nochmal bei Problemen

Viele Dank schonmal!!!!

Hmm..
Du hast ohne es zu wollen, die langsamst mögliche Lösung gewählt..

Der Code

FileReader f = new FileReader (datei);
String alles = "";
int c;
while (( c = f.read()) != -1)
{
alles = alles + (char)c; //die alte zeile wird mit dem neuen überschrieben
}

sollte unbedingt durch folgendes ersetzt werden:

BufferedReader reader = new BufferedReader(new FileReader(datei));
String line = null;
while ( (line = reader.readLine()) != null) {

StringTokenizer stto = new StringTokenizer (line, "\t");
int anzahl = stto.countTokens();
  if(anzahl>4) {
    String s1 = stto.nextToken();
    String s2 = stto.nextToken();
    String s3 = stto.nextToken();
    String s4 = stto.nextToken();
    String s5 = stto.nextToken();
    eintragen(s1,s2,s3,s4);
  }
}

Dies sollte etwa schon doppelt so schnell sein..

Greets